After creating a new snippet with an assigned keyboard shortcut, this shortcut did not work directly. This was because the snippet QActions are added to a widget by a rowsInserted() signal, but the new snippet's QAction did not exist yet at this time. This change ensures that the new snippet's QAction is created in time.
Details
Details
- Reviewers
cullmann - Group Reviewers
Kate - Commits
- R40:cd91625e8284: Snippets: Fix the keyboard shortcuts for new snippets not working
I tested it manually.
Diff Detail
Diff Detail
- Repository
- R40 Kate
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Makes sense, too. Not sure why the actions are not created on demand during the rowInserted handling, but perhaps that is intentional.
Comment Actions
Thanks for reviewing. No idea whether there is a good reason for the action not being created during the rowInserted handling; if not, that seems like the cleaner solution indeed.