Merge lp:~mcintire-evan/unity/add_screenshot_shortcut_hints into lp:unity
Proposed by
Evan McIntire
Status: | Merged |
---|---|
Approved by: | Marco Trevisan (Treviño) |
Approved revision: | no longer in the source branch. |
Merged at revision: | 4057 |
Proposed branch: | lp:~mcintire-evan/unity/add_screenshot_shortcut_hints |
Merge into: | lp:unity |
Diff against target: |
98 lines (+37/-4) 4 files modified
shortcuts/AbstractShortcutHint.h (+2/-1) shortcuts/CompizShortcutModeller.cpp (+10/-0) shortcuts/MockShortcutHint.h (+3/-0) shortcuts/ShortcutHint.cpp (+22/-3) |
To merge this branch: | bzr merge lp:~mcintire-evan/unity/add_screenshot_shortcut_hints |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Marco Trevisan (Treviño) | Approve | ||
Review via email: mp+280671@code.launchpad.net |
Commit message
ShortcutHint: Add key shortcut hints for Screenshot and Window Screenshot
Key bindings are dynamically taken from gnome settings.
Description of the change
ShortcutHint: Add key shortcut hints for Screenshot and Window Screenshot
Key bindings are dynamically taken from gnome settings.
To post a comment you must log in.
Hi, thanks for you contribution.
Please, revert the changes in the po folder, these are generated automatically.
As for making this option dynamic I think there's some little more work involved: tHint.h gnome.settings- daemon. plugins. media-keys <key_name> :Impl:: GrabDBusAcceler ator you can see how odeller. cpp you should add something like: :make_shared< shortcut: :Hint>( menubar, "", "", _("Take a screenshot."),
shortcut: :OptionType: :GNOME,
"screenshot" ) :make_shared< shortcut: :Hint>( menubar, "", "", _("Take a screenshot of the current window."),
shortcut: :OptionType: :GNOME,
"window- screenshot" )
- enable the GNOME option type in AbstractShortcu
- inside ShortcutHint.cpp read the options from:
org.
You can see how to read gsettings from UnitySettings.cpp,
while in GnomeGrabber:
to parse them.
- In CompizShortcutM
std:
std:
It should be fun :)