Skip to content
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

Autohide prevent #275

Merged
merged 5 commits into from
Nov 9, 2015
Merged

Autohide prevent #275

merged 5 commits into from
Nov 9, 2015

Conversation

palinek
Copy link
Contributor

@palinek palinek commented Nov 6, 2015

closes lxqt/lxqt#871

I don't know if I catched all the shown standalone windows.... please, check.

Update: closes lxqt/lxqt#853

@palinek
Copy link
Contributor Author

palinek commented Nov 6, 2015

selfnote: TODO: check if the periodic hide timer can be removed

@palinek palinek force-pushed the autohide_prevent branch 2 times, most recently from f3cbfb7 to 0380ce1 Compare November 6, 2015 22:43
@pmattern
Copy link
Contributor

pmattern commented Nov 7, 2015

Couldn't see the problem discussed in lxqt/lxqt#871 any longer with 0380ce1.

What I could see was the kind of problem discussed in lxqt/lxqt#853 and friends.
E. g. after clicking on plugin-mount its GUI comes up (and unlike with earlier commits the panel stops auto-hiding as expected). If the UI gets closed by clicking outside the panel's UI the panel doesn't auto-hide at once, though, but only upon hovering.
But probably this is nothing new and was just sort of unmasked by 0380ce1 fixing lxqt/lxqt#871.

Side note:
While testing this stuff I saw that there's some inconsistency as to how GUIs of panel plugins disappear.
Some like the ruler etc. from plugin-volume disappear as soon as the pointer is moved away, others like plugin-menu need a click outside their UI. Not sure what's the better approach or something needs to be changed at all, though. E. g. I find the behaviour of plugin-volume approprieate but as for the calendar that comes up on clicking plugin-[world-]clock I'd definitely vote for having it disappear upon a click outside only as viewing that calendar while e. g. hovering over some office document to compare dates is something rather common, IMO.
But just saying. Has nothing to do with the issue discussed here.

@palinek palinek force-pushed the autohide_prevent branch 2 times, most recently from 7322c9e to b19afb1 Compare November 8, 2015 10:34
@palinek
Copy link
Contributor Author

palinek commented Nov 8, 2015

E. g. after clicking on plugin-mount its GUI comes up (and unlike with earlier commits the panel stops auto-hiding as expected). If the UI gets closed by clicking outside the panel's UI the panel doesn't auto-hide at once, though, but only upon hovering.

Forgot to notify the panel for observing the window before each show() (was only for the first time being shown).... fixed and the commit Prevent panel hiding... updated. @pmattern can you confirm if it's working now?

@pmattern
Copy link
Contributor

pmattern commented Nov 9, 2015

Latest b19afb1 is fixing both lxqt/lxqt#871 and lxqt/lxqt#853, IMO. Couldn't reproduce a single of all those findings any more.
Arch Linux x86_64 with all LXQt components from today's VCS checkouts except lxqt-panel from branch autohide_prevent.

You may want to let me know whether fiddling with some Qt 5.4 system as asked for here is still needed. Of course it could be done but would mean some effort I'd rather spare myself, tbh.

Notes on lxqt/lxqt#853 (lxqt/lxqt#629):
The panel does not auto-hide as long as a dialog like "Configure Panel" is open, even upon clicking somewhere else on the desktop. But the panel does auto-hide once the dialog is closed. IMO this is absolutely the right thing to do.
Never managed to reproduce that "first / following click" thing as described here.
Also, I didn't bother trying to reproduce those somewhat mixed-up findings about particular applications like Gajim, LibreOffice and whatnot after b19afb1 just seemed to do the trick.

@palinek
Copy link
Contributor Author

palinek commented Nov 9, 2015

@pmattern thanks.

You may want to let me know whether fiddling with some Qt 5.4 system as asked for here is still needed. Of course it could be done but would mean some effort I'd rather spare myself, tbh.

Actualy, with the new implementation we don't the "lost" Leave event that we were using for triggering the hide as described in lxqt/lxqt#853 (as such updating the 1st post).

Merging...

@palinek palinek merged commit 154c7d8 into master Nov 9, 2015
@palinek palinek deleted the autohide_prevent branch November 9, 2015 10:29
@jeremywh7
Copy link
Contributor

@palinek Looking good! Almost all cases are resolved now; one that isn't is the quick launch; right-clicking it still makes the panel stick unhidden. This relates to another issue; see #[2] below. Also note that running Gajim does not specifically affect this as previously noted in Issue 853.

The panel does not auto-hide as long as a dialog like "Configure Panel" is open, even upon clicking somewhere else on the desktop. But the panel does auto-hide once the dialog is closed. IMO this is absolutely the right thing to do.

I agree that it is helpful for the Panel to remain unhidden when right-clicking and opening "Configure Panel"; but IMO not for opening any other dialog (or application). Two use cases:

  1. Right-click to open, e.g. 'Configure Task Manager'; the dialog remains open when clicking outside the dialog and/or for Panel mouse-over.
  2. A worse scenario; right-click an application in the quick launch, then open it; the panel is also stuck unhidden until Panel mouse-over. This is probably just a result of the aforementioned remaining quick launch right-click issue.

palinek added a commit that referenced this pull request Nov 10, 2015
This is a fixup for @5ad2ff2 and @3c3664d for overlooked menu in plugin-quicklaunch
as the #275 already got merged.
@palinek
Copy link
Contributor Author

palinek commented Nov 10, 2015

@jeremywh7 Thanks for testing and pointing out the "forgotten" quicklauch menu. Fixed in @9df4de7

  1. Right-click to open, e.g. 'Configure Task Manager'; the dialog remains open when clicking outside the dialog and/or for Panel mouse-over.

I don't agree here. Configure dialogs of all plugins are triggering the "no auto-hide until configure window closed" behaviour.

@jeremywh7
Copy link
Contributor

Quick launch right-click confirmed fixed.

I don't agree here. Configure dialogs of all plugins are triggering the "no auto-hide until configure window closed" behaviour.

I agree now; I was more concerned about #[2] but that is now resolved w/ the general quick launch fix. Thanks @palinek !

jauhien added a commit to gentoo/gentoo that referenced this pull request Jan 2, 2016
@agaida agaida added this to Done in Pull Requests Apr 1, 2018
@agaida agaida added this to Old in PRs Nov 5, 2018
@agaida agaida added this to Old Archive in Pull Requests Mar 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
PRs
  
Changelog
Pull Requests
  
Old Archive (fucked up)
3 participants