8000 Added a TextBox widget by keflavich · Pull Request #1983 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

Added a TextBox widget #1983

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 26 commits into from
Closed

Added a TextBox widget #1983

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
487c15a
added TextBox Widget
keflavich May 7, 2013
557adaa
lazy cursor
Jun 4, 2013
4110887
rename (de)activate, remove inconsistent active properties
Jun 4, 2013
82e3f9b
remove unused self.region
Jun 4, 2013
8ed729b
inherit/use AxesWidget
Jun 4, 2013
19fd160
cleanup __init__
Jun 4, 2013
dc4b04c
actually inherit from AxesWidget
Jun 4, 2013
32d9bb0
rely on default Text properties
Jun 4, 2013
dc8ef76
docs
Jun 4, 2013
003c931
ignore callbacks when active=False
Jun 4, 2013
e77e5e5
fix spelling of _get_text_right
Jun 4, 2013
952e122
fix spelling mouse_activate, ignore when apropriate
Jun 4, 2013
e307788
remove unused vars
Jun 4, 2013
a83b096
tab todo
Jun 4, 2013
c11973c
spell _mouse_activate correctly
Jun 4, 2013
1a2a1aa
Merge pull request #1 from mrterry/textboxwidget
keflavich Jun 4, 2013
5239d7e
spelling and formatting
Jun 5, 2013
f523505
drop float-specific formatting rules
Jun 5, 2013
42b8489
remove redraw option from set_text
Jun 5, 2013
d80083b
drop more float-specific logic
Jun 5, 2013
4b963a5
TextBox produces str values
Jun 5, 2013
aba387a
convenience function for producing TextBoxes with float values
Jun 5, 2013
a1e0207
use on_changed() convention. use drawon, too
Jun 10, 2013
a81e0b8
TextBoxFloat is a class
Jun 10, 2013
8f753d0
spelling
Jun 10, 2013
b382ceb
Merge pull request #2 from mrterry/textboxwidget
keflavich Jun 10, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove redraw option from set_text
  • Loading branch information
Matt Terry committed Jun 5, 2013
commit 42b8489beac961cafa2a10a4165d7a7d4a90b00f
6 changes: 1 addition & 5 deletions lib/matplotlib/widgets.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1804,7 +1804,7 @@ def keypress(self, event):
self.cursor.set_xdata([r, r])
self.redraw()

def set_text(self, text, redraw=False):
def set_text(self, text):
try:
# only try to update if there's a real value
if (not(text.strip() in ('-.','.','-',''))
Expand All @@ -1815,10 +1815,6 @@ def set_text(self, text, redraw=False):
except ValueError:
pass

if redraw:
self.text.draw(self.text._renderer)
self.redraw()

def _get_text_right(self):
bbox = self.text.get_window_extent()
Copy link
Contributor

Choose a reason for hiding this comment

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

This get_window_extent() causes problems on MacOSX. (related to the blitting problems with macosx). You can get around it by calling self.canvas.draw() before get_window_extent(), but that can be laggy. If you check the backend, you can only inflict the lag on macosx users. But then you're putting backend checks into supposedly backend independent widgets.

Any less hacky ideas of how to get the rendered width of a block of text?

l, b, w, h = bbox.bounds
Expand Down
0