New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve GUI responsivness #565

Closed
marmarek opened this Issue Mar 8, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 12 May 2012 10:20 UTC
After pressing the active-only/all switch button, refresh the list immediately, rather than wait until the next refresh period (3 sec or so). This makes the impression that the manager is very slow, or the system is heavily loaded.

Similarly, after the user brings the manager window in via the tray icon, draw the list immediately, and avoid showing the empty grey windows for a while...

And, while at it, change the semantics of how the tray icon works -- currently it always toggles the display of the manager window. So, if a user switches to another desktop space, and see no manager window (or covers the window by other apps), but the window is open, then clicking on the tray icon will actually... hide the window, and the user needs to click it again. This is counter intuitive. Instead, we should ensure that clicking on the tray icon always opens the manager window and brings it as active window. To close the window, the user will just click on the [X] icon in the window decoration, as for any other app.

Migrated-From: https://wiki.qubes-os.org/ticket/565

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by aga on 25 May 2012 14:44 UTC
Changed semantics of tray icon click - now the click activates manager window on the current desktop:
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=9d509917d6160ac4f64b65f36fb9ccd15a5b8a04

Member

marmarek commented Mar 8, 2015

Comment by aga on 25 May 2012 14:44 UTC
Changed semantics of tray icon click - now the click activates manager window on the current desktop:
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=9d509917d6160ac4f64b65f36fb9ccd15a5b8a04

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by aga on 29 May 2012 17:08 UTC
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=db03e5fdd241641c42ec564204defa7fe559baee

Switching between show_all and show_active_only works significantly faster now.

The empty grey window is still shown for a while ( although shorter), but I'm afraid that's a normal behavior while painting QTableWidget.

Member

marmarek commented Mar 8, 2015

Comment by aga on 29 May 2012 17:08 UTC
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=db03e5fdd241641c42ec564204defa7fe559baee

Switching between show_all and show_active_only works significantly faster now.

The empty grey window is still shown for a while ( although shorter), but I'm afraid that's a normal behavior while painting QTableWidget.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by joanna on 12 Jun 2012 13:39 UTC
As of 1.2.20, this all seems to be working fine now. Closing.

Member

marmarek commented Mar 8, 2015

Comment by joanna on 12 Jun 2012 13:39 UTC
As of 1.2.20, this all seems to be working fine now. Closing.

@marmarek marmarek closed this Mar 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment