New layout for qt4 subplottool + QMainWindow -> QDialog#2594
New layout for qt4 subplottool + QMainWindow -> QDialog#2594efiring merged 5 commits intomatplotlib:masterfrom rhoef:newsubplottool
Conversation
|
@tacaswell rebased + use new style qt signals |
There was a problem hiding this comment.
We only have this one other place in the code base, is there a policy on this? @mdboom
There was a problem hiding this comment.
There are 4 occurrences of __author__ in the mpl codebase
examples/api/sankey_demo_old.py:__author__ = "Yannick Copin <[email protected]>"
examples/user_interfaces/interactive.py:__author__ = "Fernando Perez <[email protected]>"
examples/user_interfaces/svg_histogram.py:__author__="[email protected]"
lib/matplotlib/sankey.py:__author__ = "Kevin L. Davies"There was a problem hiding this comment.
I would argue that the examples are not part of the library proper (as they are stand alone examples of using the library)
|
@Thisch Awesome, thanks! I am a little concerned about two non-code things (both of which may be me worrying too much):
|
|
@tacaswell: Put @rhoef back into the commit logs (by squashing the #2112 branch) |
|
I just tested this quickly on my machine, and it looks very nice. Just one glitch: the label on the "tight layout" button is misspelled (starts with "th" instead of "t"). |
Signed-off-by: Thomas Hisch <[email protected]>
|
@efiring fixed typo |
New layout for qt4 subplottool + QMainWindow -> QDialog
This supersedes the subplottool part from #2112.