Merge lp:~sdemchenko/glipper/feature-accelerators into lp:glipper
Status: | Needs review |
---|---|
Proposed branch: | lp:~sdemchenko/glipper/feature-accelerators |
Merge into: | lp:glipper |
Diff against target: |
165 lines (+95/-4) 4 files modified
glipper/AppIndicator.py (+26/-2) glipper/__init__.py (+3/-0) glipper/plugins/autopaste.py (+62/-0) glipper/plugins/snippets.py (+4/-2) |
To merge this branch: | bzr merge lp:~sdemchenko/glipper/feature-accelerators |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Laszlo Pandy | Pending | ||
Review via email: mp+118856@code.launchpad.net |
Description of the change
I have added accelerators to history items and to snippet menu and its items.
Also, a new plugin "Auto paste" can accelerate Glipper usage even further (it sends Control+V key stroke to the active window right after an item selection). Just like item accelerators, auto-pasting works both for history and snippets.
Unmerged revisions
- 121. By Stanislav Demchenko
-
Replaced hard-coded accelerator characters with their respective internationalized counterparts provided by GTK.
- 120. By Stanislav Demchenko
-
Added 'Auto paste' plugin to emulate Control+V on a history or snippet item selection.
- 119. By Stanislav Demchenko
-
Added a space between the accelerator and the label in menu item "Snippets" (for readability).
- 118. By Stanislav Demchenko
-
Restored spaces as in trunk to simplify review.
Added 1 extra space between the accelerator character and menu item label to improve readability. - 117. By Stanislav Demchenko
-
Added accelerators for menu items in history and in snippets.
> reserved_ accelerators = "0acpu" # _0. Snippets, _About, _Clear, _Preferences, Pl_ugins
This won't work. For example, in the French translation, the accelorator for Pl_ugins is "_Greffons".
The only way I see to make it work across languages is to only use numbers for the accelorators.