FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix sliders by mdboom · Pull Request #581 · matplotlib/matplotlib · GitHub

Repository navigation

Fix sliders - #581

Closed
mdboom wants to merge 2 commits into
matplotlib:masterfrom
mdboom:sliders
Closed

mdboom wants to merge 2 commits into
matplotlib:masterfrom
mdboom:sliders

Conversation

mdboom commented Nov 10, 2011

Copy link
Copy Markdown
Member

Alternative to #576 that should have fewer side effects since it doesn't circumvent the Polygon.set_closed method. axhspan and axvspan should have been declared closed all along.

…ce it doesn't circumvent the Polygon.set_closed method. axhspan and axvspan should have been declared closed all along.
mdboom mentioned this pull request Nov 10, 2011

mdboom commented Nov 16, 2011

Copy link
Copy Markdown
Member Author

Ok. I think the next thing to do is decide whether this or #576 makes more sense.

cgohlke commented Nov 16, 2011

Copy link
Copy Markdown
Contributor

While this fixes the slider demo, #576 addresses two more issues:

  1. When a Slider is initialized with valmin == valinit, the indicator is a triangle (e.g. set a0 = 0.1 in slider_demo.py)
  2. It might not be safe to call set_closed(False) repeatedly, e.g. on a Polygon where all vertices have the same coordinates.

mdboom closed this Dec 14, 2011
mdboom deleted the sliders branch November 10, 2015 02:30
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL