tkcairo backend - #10436
tkcairo backend#10436
Conversation
|
Thanks, got a bit delayed by real life and was also waiting for the other backends to get in first. A few notes on the implementation: I moved the common parts to I also deprecated FigureManagerTkAgg and NavigationToolbar2TkAgg, both because FigureManagerTk and NavigationToolbar2Tk are available and because I would argue that the former should actually be accessed by third parties as AFAICT tkagg does all its rendering to the actual widget in draw() (rather than "buffering" it similarly to how Qt separates draw() from paintEvent()). While it may be interesting to rework this I did not change the design in this PR. |
c35126a to
de3da7c
Compare
QuLogic
left a comment
There was a problem hiding this comment.
Seemed to work for unchained.py for both TkAgg and TkCairo.
| self.window.attributes('-fullscreen', not is_fullscreen) | ||
|
|
||
|
|
||
| @cbook.deprecated("2.2") |
There was a problem hiding this comment.
Was this replaced by something in particular?
There was a problem hiding this comment.
This class was used by the old navigationtoolbar (pre-navigationtoolbar2) so basically unused for a very long time. But I moved it to _backend_tk like everyone else for a normal deprecation period.
|
You might want to rebase and re-target |
de3da7c to
8cefced
Compare
|
Rebased. Before I retarget this: do you want to later "forwardport" from 2.2 to master? |
|
I expect |
|
done |
8cefced to
c6309bf
Compare
|
needs to be merged into master as well. |
|
This looks like it will cause chaos when a new version is released out of the development version. If |
|
@ImportanceOfBeingErnest As far as I can see they are both still there. They are just empty aliases for the General non Agg specific version |
|
@jenshnielsen I wouldn't preclude that you have better eyes than I have, but I don't see them anywhere. |
|
They were deprecated in 2.2 and removed in #10426 (3.0) which is admittedly a shorter cycle than normal, but an API removal notice should be enough. |

PR Summary
same as wxcairo and qtcairo, but for tk :-)
PR Checklist