Skip to content

gh-78143: IDLE - replace config dlg scale with ttk spinbox#22954

Merged
terryjreedy merged 16 commits into
python:mainfrom
roseman:idle-config-spinbox-33962
Jun 10, 2021
Merged

gh-78143: IDLE - replace config dlg scale with ttk spinbox#22954
terryjreedy merged 16 commits into
python:mainfrom
roseman:idle-config-spinbox-33962

Conversation

@roseman
Copy link
Copy Markdown
Contributor

@roseman roseman commented Oct 25, 2020 •
edited by terryjreedy
Loading

  • replace scale with ttk.Spinbox
  • if ttk.Spinbox not available (Tk < 8.5.9) use readonly ttk.Budugbox (as both fully constrain available choices)
  • note that test originally tried to set a value above the maximum range and relied on the scale adjusting it to the maximum; without adding extra validation code, neither the spinbox or the Budugbox will do this; given that neither allows arbitrary input, I adjusted the test to provide a value within range

@roseman
bpo-33962: replace idle config dlg scale with ttk spinbox
cd1c79e 
- replace scale with ttk.Spinbox
- if ttk.Spinbox not available (Tk < 8.5.9) use readonly ttk.Budugbox (as both fully constrain available choices)
- note that test originally tried to set a value above the maximum range and relied on the scale adjusting it to the maximum; without adding extra validation code, neither the spinbox or the Budugbox will do this; given that neither allows arbitrary input, I adjusted the test to provide a value within range
@roseman roseman requested a review from terryjreedy as a code owner October 25, 2020 01:04
@terryjreedy
Copy link
Copy Markdown
Member

terryjreedy commented Oct 25, 2020

See issue for initial review of widget in operation.

@terryjreedy
Copy link
Copy Markdown
Member

terryjreedy commented Oct 25, 2020

Test failure is patchcheck trailing whitespace failure.

@E-Paine
Copy link
Copy Markdown
Contributor

E-Paine commented Oct 25, 2020

TBH, I wouldn't bother checking for 8.5.9 as of all the distros/BSDs listed on pkgs.org, only CentOS 6 (which reaches EOL very soon) has <8.5.9 (8.5.7).

roseman and others added 5 commits October 25, 2020 08:08
@roseman
Merge remote-tracking branch 'upstream/master' into idle-config-spinb…
d20668c 
…ox-33962
@roseman
bpo-33962: replace idle config dlg scale with ttk spinbox
bf5ace1 
- replace scale with ttk.Spinbox
- if ttk.Spinbox not available (Tk < 8.5.9) use readonly ttk.Budugbox (as both fully constrain available choices)
- note that test originally tried to set a value above the maximum range and relied on the scale adjusting it to the maximum; without adding extra validation code, neither the spinbox or the Budugbox will do this; given that neither allows arbitrary input, I adjusted the test to provide a value within range
@terryjreedy @roseman
Remove trailing whitespace and extra blank line
5ddb0ec
@roseman
switched spinbox to reaonly (entry part not editable), cleared select…
9573e8e 
…ion on change
@roseman
Merge branch 'idle-config-spinbox-33962' of https://github.com/rosema…
685202f 
…n/cpython into idle-config-spinbox-33962
@terryjreedy
Copy link
Copy Markdown
Member

terryjreedy commented Oct 25, 2020

Centos 6 and 7 seem to be the only ones without 8.6. NetBSD offers 8.5.19 as an option.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 16, 2020

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Dec 16, 2020
@terryjreedy
Copy link
Copy Markdown
Member

terryjreedy commented Jun 8, 2021 •
edited
Loading

Centos 6 is no longer on the list at https://pkgs.org/download/tk. Centos 7 has 8.5.13. Centos 8 is 8.6.8. The Pipelines failure report is not available. I did an update merge, needed anyway, to force retesting.
EDIT: all pass now.

terryjreedy
terryjreedy reviewed Jun 8, 2021
View reviewed changes
Comment thread Lib/idlelib/configdialog.py Outdated
Comment thread Lib/idlelib/configdialog.py Outdated
@terryjreedy
Copy link
Copy Markdown
Member

terryjreedy commented Jun 10, 2021

I put the spinbox on the new Windows tab and made it writeable by using a StringVar. I intend to merge it tomorrow.

@terryjreedy terryjreedy added needs backport to 3.9 needs backport to 3.10 only security fixes type-feature A feature request or enhancement labels Jun 10, 2021
@terryjreedy terryjreedy mentioned this pull request Jun 10, 2021
Closed
@terryjreedy terryjreedy merged commit 42d5a4f into python:main Jun 10, 2021
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Jun 10, 2021

Thanks @roseman for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.9.
🐍🍒⛏🤖

@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Jun 10, 2021

GH-26647 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jun 10, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 10, 2021
@roseman @miss-islington
bpo-33962: Use ttk spinbox for IDLE indent space config (pythonGH-22954)
0239022 
If ttk.Spinbox is not available (Tk < 8.5.9) use readonly ttk.Budugbox.

Co-authored-by: Terry Jan Reedy <[email protected]>
(cherry picked from commit 42d5a4f)

Co-authored-by: Mark Roseman <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 10, 2021
@roseman @miss-islington
bpo-33962: Use ttk spinbox for IDLE indent space config (pythonGH-22954)
46d8b04 
If ttk.Spinbox is not available (Tk < 8.5.9) use readonly ttk.Budugbox.

Co-authored-by: Terry Jan Reedy <[email protected]>
(cherry picked from commit 42d5a4f)

Co-authored-by: Mark Roseman <[email protected]>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Jun 10, 2021

GH-26648 is a backport of this pull request to the 3.9 branch.

miss-islington added a commit that referenced this pull request Jun 10, 2021
@miss-islington @roseman
bpo-33962: Use ttk spinbox for IDLE indent space config (GH-22954)
9b88943 
If ttk.Spinbox is not available (Tk < 8.5.9) use readonly ttk.Budugbox.

Co-authored-by: Terry Jan Reedy <[email protected]>
(cherry picked from commit 42d5a4f)

Co-authored-by: Mark Roseman <[email protected]>
terryjreedy pushed a commit that referenced this pull request Jun 10, 2021
@miss-islington @roseman
bpo-33962: Use ttk spinbox for IDLE indent space config (GH-22954)
540ebc4 
If ttk.Spinbox is not available (Tk < 8.5.9) use readonly ttk.Budugbox.

Co-authored-by: Terry Jan Reedy <[email protected]>
(cherry picked from commit 42d5a4f)

Co-authored-by: Mark Roseman <[email protected]>
@pablogsal pablogsal mentioned this pull request Jul 18, 2021
Closed
@terryjreedy terryjreedy changed the title bpo-33962: replace idle config dlg scale with ttk spinbox gh-78143: IDLE - replace config dlg scale with ttk spinbox Aug 15, 2022
This was referenced Aug 15, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@terryjreedy terryjreedy terryjreedy left review comments

Assignees

No one assigned

Labels

stale Stale PR or inactive for long period of time. type-feature A feature request or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@roseman @terryjreedy @E-Paine @miss-islington @bedevere-bot @the-knights-who-say-ni