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
lazy cursor
  • Loading branch information
Matt Terry committed Jun 4, 2013
commit 557adaa70bef25cc46387dd01083fec334243d9e
14 changes: 12 additions & 2 deletions lib/matplotlib/widgets.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1711,10 +1711,9 @@ def __init__(self, ax, s='', horizontalalignment='left',

self.region = self.canvas.copy_from_bbox(ax.bbox)
Copy link
Contributor

Choose a reason for hiding this comment

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

self.region is not used by TextBox or Widget. Do other things assume it is there or can it be dropped? Dropping it would help appease macos.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point; I have no idea if other things will assume it. But perhaps a try/except around this would be the safest way to go, not knowing any better?


self._cursor = None
self._cursorpos = len(self.text.get_text())
r = self._get_text_right()

self.cursor, = ax.plot([r,r], [0.2, 0.8], transform=ax.transAxes)
self.active = False

self.redraw()
Expand All @@ -1724,6 +1723,17 @@ def __init__(self, ax, s='', horizontalalignment='left',

self.canvas.mpl_connect('button_press_event',self._mouse_activate)

@property
def cursor(self):
# Macos has issues with render objects. Lazily generating the cursor
# solve some of the problems associated
if self._cursor is None:
r = self.get_text_right() # needs a renderer
self._cursor, = self.ax.plot([r,r], [0.2, 0.8],
transform=self.ax.transAxes)
self._cursor.set_visible(False)
return self._cursor

def redraw(self):
# blitting doesn't clear old text
#self.ax.redraw_in_frame()
Expand Down
0