gh-91398: Remove IDLE shell sidebar padding #32363
Conversation
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.
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
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.
|
@ambv This should be linked to the new GH issue when available. I overwrote the proposed change and will add blurb before merging. |
|
This missed the boat for inclusion in Python 3.9 which accepts security fixes only as of today. |
|
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
|
I don't believe that this is needed for 3.9. It is certainly more of an issue for 3.10+. There is a new CI button that requires CLA be signed, even for something trivial that should not require it. Adding a possibly false CLA signed label does not seem to fix that. If no signature, any workaround other than making a new PR myself? |
That's my bad. I messed up the CLA thing, and, long story short, I ended up making a new identical issue on the bugs.python.org thing that can be found here |
|
@BA-24 could you try signing the CLA again? This should work now. |
|
Should be signed now |
|
Hey, just updated to 3.11, is this getting merged or will I have to keep modifying the file manually? |
CI is failing because the news item is missing. |
|
I am still planning to look at this soon. |
As far as I'm concerned it's a typo and does not need a news entry. |
|
Hey, just updated to 3.12 and it seems this change still hasn't gone through. Updates? |
@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? |
I think this looks great! |
|
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 |
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