Merge lp:~fboucault/unity-2d/panel_show_window_buttons_better into lp:unity-2d

Proposed by Florian Boucault
Status: Merged
Approved by: Gerry Boland
Approved revision: 696
Merged at revision: 699
Proposed branch: lp:~fboucault/unity-2d/panel_show_window_buttons_better
Merge into: lp:unity-2d
Diff against target: 23 lines (+5/-3)
1 file modified
panel/applets/appname/appnameapplet.cpp (+5/-3)
To merge this branch: bzr merge lp:~fboucault/unity-2d/panel_show_window_buttons_better
Reviewer Review Type Date Requested Status
Gerry Boland Pending
Review via email: mp+74103@code.launchpad.net

Description of the change

[panel] Show window control buttons whenever the application menu is shown and the window is maximised.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'panel/applets/appname/appnameapplet.cpp'
--- panel/applets/appname/appnameapplet.cpp 2011-08-22 09:17:03 +0000
+++ panel/applets/appname/appnameapplet.cpp 2011-09-05 14:00:25 +0000
@@ -218,14 +218,16 @@
218 bool isUserVisibleApp = app ? app->user_visible() : false;218 bool isUserVisibleApp = app ? app->user_visible() : false;
219 bool isOnSameScreen = d->m_windowHelper->isMostlyOnScreen(QApplication::desktop()->screenNumber(this));219 bool isOnSameScreen = d->m_windowHelper->isMostlyOnScreen(QApplication::desktop()->screenNumber(this));
220 bool isUnderMouse = rect().contains(mapFromGlobal(QCursor::pos()));220 bool isUnderMouse = rect().contains(mapFromGlobal(QCursor::pos()));
221 bool showMenu = (!d->m_menuBarWidget->isEmpty() && isUserVisibleApp && isOnSameScreen)221 bool isOpened = isOnSameScreen &&
222 && (isUnderMouse222 (isUnderMouse
223 || KeyboardModifiersMonitor::instance()->keyboardModifiers() == Qt::AltModifier223 || KeyboardModifiersMonitor::instance()->keyboardModifiers() == Qt::AltModifier
224 || d->m_menuBarWidget->isOpened()224 || d->m_menuBarWidget->isOpened()
225 );225 );
226 bool showMenu = isOpened && !d->m_menuBarWidget->isEmpty() && isUserVisibleApp;
227 bool showWindowButtons = isOpened && isMaximized;
226 bool showLabel = !(isMaximized && showMenu) && isUserVisibleApp && isOnSameScreen;228 bool showLabel = !(isMaximized && showMenu) && isUserVisibleApp && isOnSameScreen;
227229
228 d->m_windowButtonWidget->setVisible(isOnSameScreen && isMaximized && isUnderMouse);230 d->m_windowButtonWidget->setVisible(showWindowButtons);
229231
230 d->m_label->setVisible(showLabel);232 d->m_label->setVisible(showLabel);
231 if (showLabel) {233 if (showLabel) {

Subscribers

People subscribed via source and target branches