Partially fix rubberbanding in GTK3. by anntzer · Pull Request #17772 · matplotlib/matplotlib · GitHub
Skip to content

Partially fix rubberbanding in GTK3. - #17772

Merged
tacaswell merged 1 commit into
matplotlib:masterfrom
anntzer:gtk3rubberband
Jul 1, 2020
Merged

tacaswell merged 1 commit into
matplotlib:masterfrom
anntzer:gtk3rubberband

Conversation

@anntzer

@anntzer anntzer commented Jun 26, 2020

Copy link
Copy Markdown
Contributor

The code became wrong because of crossed PRs (#13144+#17111 resulting in assignment in
the readonly ctx property). Let's make ctx an instance attribute
again for now, which at least avoids an exception; however the
rubberband is invisible for reasons unclear to me.

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@anntzer anntzer added Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. GUI: gtk labels Jun 26, 2020
@anntzer anntzer added this to the v3.3.0 milestone Jun 26, 2020

@QuLogic QuLogic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes the crash for me.

The code became wrong because of crossed PRs (resulting in assignment in
the readonly `ctx` property).  Let's make `ctx` an instance attribute
again for now, which at least avoids an exception; however the
rubberband is invisible for reasons unclear to me.
@anntzer

anntzer commented Jun 27, 2020

Copy link
Copy Markdown
Contributor Author

@tacaswell
tacaswell merged commit 207fd42 into matplotlib:master Jul 1, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jul 1, 2020
@anntzer
anntzer deleted the gtk3rubberband branch July 1, 2020 08:35
timhoffm added a commit that referenced this pull request Jul 1, 2020
…772-on-v3.3.x

Backport PR #17772 on branch v3.3.x (Partially fix rubberbanding in GTK3.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GUI: gtk Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants