Merge lp://staging/~sdemchenko/glipper/feature-accelerators into lp://staging/glipper

Proposed by Stanislav Demchenko
Status: Superseded
Proposed branch: lp://staging/~sdemchenko/glipper/feature-accelerators
Merge into: lp://staging/glipper
Diff against target: 151 lines (+81/-4)
4 files modified
glipper/AppIndicator.py (+12/-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://staging/~sdemchenko/glipper/feature-accelerators
Reviewer Review Type Date Requested Status
Laszlo Pandy Pending
Review via email: mp+118210@code.staging.launchpad.net

Description of the change

I have added accelerators to history items and to snippet menu and its items.
The change works pretty well, but I used the new functionality for 2 days only, so there could be a bug lurking.
Let me know please what you think of this functionality: is it very useful as it is, or shall I try and make it an optional plugin? shall visual decoration be changed? shall I reorganize my code a little?

To post a comment you must log in.
Revision history for this message
Stanislav Demchenko (sdemchenko) wrote :

In addition to accelerators on history and snippet menu items, a new plugin "Auto paste" can be enabled to accelerate Glipper usage even further (it sends Control+V key stroke to the active window right after an item selection). Auto-pasting works both for history and snippets.

121. By Stanislav Demchenko

Replaced hard-coded accelerator characters with their respective internationalized counterparts provided by GTK.

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.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches