Skip to content

gh-91398: Remove IDLE shell sidebar padding - #32363

Merged
serhiy-storchaka merged 7 commits into
python:mainfrom
BA-24:patch-1
Sep 15, 2026
Merged

serhiy-storchaka merged 7 commits into
python:mainfrom
BA-24:patch-1

Conversation

@BA-24

@BA-24 BA-24 commented Apr 6, 2022

Copy link
Copy Markdown
Contributor

I noticed there was a white bar to the left in my IDLE, it stuck out like a soar thumb since I have a custom dark theme. This was not an issue with earlier versions.

Changing this line of code solves this issue and leaves the color choices up to the user.

#89614, #91398

I noticed there was a white bar to the left in my IDLE, it stuck out like a soar thumb since I have a custom dark theme. This was not an issue with earlier versions.

Changing this line of code solves this issue and leaves the color choices up to the user.
@the-knights-who-say-ni

This comment was marked as outdated.

@terryjreedy terryjreedy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This only affects the shell sidebar, which is the subject of bpo-45451. I made the change I requested.

On my Windows with dark theme, I prefer the change. But change needs evaluation on light and dark themes on all 3 major OSes. Will ask OP of issue for opinion. So not approved yet.

Comment thread Lib/idlelib/sidebar.py Outdated
@terryjreedy terryjreedy changed the title Remove white bar bpo-45451: Remove IDLE shell sidebar padding Apr 11, 2022
@python python deleted a comment from bedevere-bot Apr 11, 2022
@terryjreedy terryjreedy added type-bug An unexpected behavior, bug, or error needs backport to 3.9 needs backport to 3.10 only security fixes topic-IDLE 3.11 only security fixes 3.10 only security fixes 3.9 (EOL) end of life labels Apr 11, 2022
@terryjreedy

Copy link
Copy Markdown
Member

@ambv This should be linked to the new GH issue when available. I overwrote the proposed change and will add blurb before merging.

@ambv

ambv commented May 17, 2022

Copy link
Copy Markdown
Contributor

This missed the boat for inclusion in Python 3.9 which accepts security fixes only as of today.

@hugovk hugovk removed the needs backport to 3.10 only security fixes label Apr 8, 2023
@BA-24

BA-24 commented Mar 13, 2024

Copy link
Copy Markdown
Contributor Author

Hey, just updated to 3.12 and it seems this change still hasn't gone through. Updates?

@hugovk hugovk removed the needs backport to 3.11 only security fixes label Apr 22, 2024
@StanFromIreland

Copy link
Copy Markdown
Member

I am still planning to look at this soon. (2022)

@terryjreedy Just a little reminder :) Is it possible for you to run all the actions (I do not have the permissions to do this) so we can see the current state of the PR?

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@terryjreedy you asked for previews on major oss, here is Linux.

Classic

image

Dark

image

@BA-24

BA-24 commented Feb 5, 2025

Copy link
Copy Markdown
Contributor Author

@terryjreedy you asked for previews on major oss, here is Linux.

I think this looks great!

@python python deleted a comment Apr 7, 2025
@bedevere-app

bedevere-app Bot commented Apr 23, 2025

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 10, 2026
@bedevere-app

bedevere-app Bot commented Sep 15, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes and removed stale Stale PR or inactive for long period of time. type-feature A feature request or enhancement labels Sep 15, 2026
@serhiy-storchaka
serhiy-storchaka enabled auto-merge (squash) September 15, 2026 16:27
@serhiy-storchaka
serhiy-storchaka merged commit 82952e3 into python:main Sep 15, 2026
61 checks passed
@miss-islington-app

Copy link
Copy Markdown

Thanks @BA-24 for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Sep 15, 2026

Copy link
Copy Markdown

GH-157574 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Sep 15, 2026
@bedevere-app

bedevere-app Bot commented Sep 15, 2026

Copy link
Copy Markdown

GH-157575 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Sep 15, 2026
@bedevere-app

bedevere-app Bot commented Sep 15, 2026

Copy link
Copy Markdown

GH-157576 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Sep 15, 2026
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.

10 participants