-
-
Notifications
You must be signed in to change notification settings - Fork 34.5k
bpo-32585: Add ttk::spinbox #5221
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.
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
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
a2a46f3
Add ttk::spinbox
alandmoore 505c77d
Added NEWS.d/next entry
alandmoore c5cbac7
Removed double colon that was causing a warning/error during build.
alandmoore 7b8219e
Add more arguments to ttk.Spinbox docstring
alandmoore 826f4e3
Make spinbox inherit Combobox rather than Entry
alandmoore 3228370
Expand on usage information in Spinbox docstring
alandmoore af4e1a1
Revert Spinbox to Entry baseclass, add set/current methods
alandmoore 16a5b98
Keep __all__ alphabetzied
alandmoore a56a970
Test class for ttk.Spinbox
alandmoore 1a3b7c1
Documentation for ttk.Spinbox
alandmoore a933457
Remove nonexistent "current" method
alandmoore 6d4f9b5
Update spinbox-related documentation
alandmoore aa37596
Remove some blank lines and reformat some long ones
alandmoore 02cb17b
Add patch author's name to ACKS
alandmoore 032b44f
Added tkinter.ttk.Spinbox addition.
alandmoore eecd3d4
Remove tabs inserted by accident
alandmoore 61d2aa0
Remove tab introduced by bad editor setting
alandmoore 95bae22
Resolved merge conflict on whatsnew file
alandmoore 3a5eadb
Fix table cell indent for Spinbox docs
alandmoore b533b2e
Merge branch 'master' into fix-issue-32585
alandmoore 2aa62c3
Fix role-less inline code
alandmoore 9f91776
Merge remote-tracking branch 'origin/fix-issue-32585' into fix-issue-…
alandmoore 2df569d
add SpinboxTest to test_gui tuple
alandmoore 8387517
Fix tests on HiDPI displays.
serhiy-storchaka 1ef7ca0
Add missing options, update_idletasks to Spinbox tests
alandmoore bf8bd5c
Merge branch 'fix-issue-32585' of https://github.com/alandmoore/cpyth…
alandmoore 92ecb6c
add update() to spinbox tests to ensure callbacks are called
alandmoore 53046dc
Use update() after setting a command in the test.
serhiy-storchaka File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
1 change: 1 addition & 0 deletions
1
Misc/NEWS.d/next/Library/2018-01-18-13-09-00.bpo-32585.qpeijr.rst
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Add Ttk spinbox widget to to tkinter.ttk. Patch by Alan D Moore. |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be
Combobox.__init__now?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quite right. Is there a particular reason why none of the classes in this library use
super()?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm actually going to revert the base class to Entry per @terryjreedy 's comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am pretty sure super came years later than the original Tkinter module. In 2.x it was a bit harder to use. For single inheritance, I don't think that there is much advantage. (I believe that there are cases of multiple inheritance in tkinter.init, but have not rechecked.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this should have a base class of Entry, but with that, I also think
currentandsetfrom Combobox should be included here. It would probably be overkill, but maybe Spinbox and Combobox should both inherit from an Entry subclass that hascurrentandset?The tests will be able to use
currentandset.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
currentandsetmethods correspond thecurrentandsetsubcommands of Ttk widgets. The ttk::entry widget doesn't have these commands.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I meant that both Spinbox and Combobox have
currentandset. I didn't know if it was better to implement them both directly from Entry or to have another class in between, that is a subclass of Entry that implements current and set and is the superclass to Combobox and Spinbox.