Merge lp://staging/~shippo/plank/fix-1251625 into lp://staging/plank
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~shippo/plank/fix-1251625 |
Merge into: | lp://staging/plank |
Diff against target: |
523 lines (+177/-62) 15 files modified
data/net.launchpad.plank.gschema.xml.in.in (+5/-0) data/ui/preferences.ui (+24/-0) docklets/Clippy/ClippyDockItem.vala (+1/-1) docklets/Clock/ClockDockItem.vala (+1/-1) docklets/Trash/TrashDockItem.vala (+1/-1) lib/DockPreferences.vala (+3/-0) lib/Items/ApplicationDockItem.vala (+77/-44) lib/Items/DockElement.vala (+3/-6) lib/Items/DockItem.vala (+12/-0) lib/Items/FileDockItem.vala (+10/-3) lib/Items/PlankDockItem.vala (+1/-1) lib/Services/WindowControl.vala (+20/-0) lib/Widgets/DockWindow.vala (+3/-3) lib/Widgets/PreferencesWindow.vala (+13/-0) lib/libplank.symbols (+3/-2) |
To merge this branch: | bzr merge lp://staging/~shippo/plank/fix-1251625 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Peter Feichtinger (community) | Needs Resubmitting | ||
Rico Tzschichholz | Needs Information | ||
Review via email: mp+317909@code.staging.launchpad.net |
Description of the change
This fixes Bug #1251625 (partially).
There are two related changes in this branch which could be merged individually, but I think they belong together:
* First, there is now a new preference setting that, when enabled, causes application items to show their menu when left clicked, instead of bringing all windows up. This way a window to be brought up can be selected from the menu without having to right click.
* Second, the application item menu is changed to indicate the currently active window (in bold font) and allow it to be minimized by selecting it, and to indicate minimized windows by dimming their items slightly.
There is no window preview as requested in the original bug report, but I think this change makes the dock way more usable (and doesn't change the default behavior when disabled, aside from the new menu item styles).
Unmerged revisions
- 1591. By Peter Feichtinger
-
Fix window menu showing with only one item.
- 1590. By Peter Feichtinger
-
Add indication for active and minimized windows and allow mimizing.
This adds an indication in the window menu for the currently active
window, as well as for minimized windows. The active window is minimized
when selected. - 1589. By Peter Feichtinger
-
Add support for switching windows when an application item is clicked.
* It should be possible for a DockItem to show a menu based on the
current number of open windows, so a new method in DockItem is used in
DockWindow to check whether an item wants to show a menu when any
button is pressed.
* ApplicationDockItem is changed to show the window menu on left click,
when more than one window is open for the application.
* A new preference switch for using the window menu is added: when
'Use Window Menu' is enabled, left clicking on an item with multiple
windows will open the item menu, allowing selection of only one window
to show. When disabled, clicking has the default behavior.
Overall this looks good, besides some minor code-style issues.
I would prefer if the special left-click menu would *only* contain the window-listing.
Also making it *2* commits is sufficient:
* the menuitem-style change
* the new menu including its requirements
- don't be lazy with variable names like "lbl" -> "label"
- prefix connectors in if-conditions in case of line-breaks