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.
|
All commit authors signed the Contributor License Agreement. |
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.
terryjreedy
changed the title
terryjreedy
added
type-bug
|
@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? |
terryjreedy
changed the title
terryjreedy
added
type-feature
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 |
terryjreedy
changed the title
|
@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. |
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