qt: raise each new window #5553
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Is there an analogous solution for Qt4 (which is still fairly relevant given that's what Anaconda gives?) |
|
It should work for both. I only have qt4 on my machine, so that is all I have tested it with. Most of qt4 support is inherited from the backend_qt5 modules. |
|
The naming on the qt4/qt5 modules is a bit odd (sorry, that is my fault), but qt5 is the 'base' implementation and the qt4 implementations sub-classes from it and patches over the API changes (the rearrangement of Qt it self and the change in PyQt5 to use cooperative sub-classes). |
|
Backported to v2.0.x as 1c7fb5b |
tacaswell
pushed a commit
to tacaswell/matplotlib
that referenced
this pull request
Dec 19, 2015
qt: raise each new window
|
Backport to v2.x is actually via dee9757. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an attempt to make progress on #596 with a one-line change to the qt backend that adds a call to
raise_(). It raises and grabs focus. I would prefer that it not grab focus, but at least on OSX I don't know how to avoid it. The present behavior on OSX is so annoying that I am willing to accept the focus grab. Contrary to what I thought, my test of the qt4 backend on Linux without this change shows that at least on my XFCE desktop, the window is already being raised, including the focus grab; so the changeset here is just making OSX behave like this Linux desktop.I also tried the approach of using the
WindowStaysOnTopHint, but the problem is that it really stays on top, and that's no good either.