Add a shortcut to delete TM entries (Ctrl+Alt+Number)
ClosedPublic

Authored by sdepiets on Aug 3 2018, 3:04 AM.

Details

Summary

Add a shortcut to delete TM entries (Ctrl+Alt+Number)

FEATURE: 233144

Diff Detail

Repository
R456 Lokalize
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sdepiets requested review of this revision.Aug 3 2018, 3:04 AM
sdepiets created this revision.
sdepiets edited the summary of this revision. (Show Details)Aug 3 2018, 3:06 AM
sdepiets added a project: Localization.
sdepiets updated this revision to Diff 38993.Aug 3 2018, 3:08 AM

Remove unrelated warning

aacid added a comment.Aug 13 2018, 4:25 PM

Maybe Ctrl+Shift instead of Ctrl+Alt?

Shift is commonly used to say "the reverse", for example in kate Ctrl+D commends and Ctrl+Shift+D uncomments

Maybe Ctrl+Shift instead of Ctrl+Alt?

Shift is commonly used to say "the reverse", for example in kate Ctrl+D commends and Ctrl+Shift+D uncomments

That doesn't work with some keyboards (those that require Shift to press numbers, for instance French azerty)

aacid accepted this revision.Aug 14 2018, 8:16 AM
aacid added inline comments.
src/editortab.cpp
320

You don't need the QString::number, i18n will do that right, but i guess if the code above is doing it, you can leave it for consistency, as you prefer.

This revision is now accepted and ready to land.Aug 14 2018, 8:16 AM
sdepiets updated this revision to Diff 39826.Aug 16 2018, 3:50 AM
  • Merge branch 'master' into delete_shortcut_tm
sdepiets updated this revision to Diff 39830.Aug 16 2018, 6:24 AM

Update editorui.rc name

This revision was automatically updated to reflect the committed changes.