Skip to content
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

gh-88516: show file proxy icon in IDLE editor windows on macOS #112894

Merged
merged 4 commits into from
Feb 23, 2024

Conversation

ronaldoussoren
Copy link
Contributor

@ronaldoussoren ronaldoussoren commented Dec 9, 2023

The platform standard on macOS is to show a proxy icon for open files in the titlebar of Windows. Make sure IDLE matches this behaviour.

Also:

  • Maintain the right window modification status for editor windows ("dot" in the close indicator for unsaved files)
  • Don't show both the short and long (file) name to match platform behaviour with other editors.

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.
@ronaldoussoren
Copy link
Contributor Author

@terryjreedy, could you please review this PR?

I'd prefer to merge this because this improves IDLE on macOS without big changes from cross-platform behaviour.

@ronaldoussoren
Copy link
Contributor Author

@terryjreedy, I intent to merge this tomorrow afternoon (CET timezone). It is a minor change, but aligns the GUI better with platform conventions without IMHO making it too different from other platforms.

Lib/idlelib/editor.py Outdated Show resolved Hide resolved
Lib/idlelib/editor.py Outdated Show resolved Hide resolved
Lib/idlelib/editor.py Show resolved Hide resolved
@bedevere-app
Copy link

bedevere-app bot commented Feb 3, 2024

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@ronaldoussoren
Copy link
Contributor Author

I have made the requested changes; please review again.

@terryjreedy terryjreedy added needs backport to 3.11 only security fixes needs backport to 3.12 bug and security fixes labels Feb 23, 2024
@terryjreedy terryjreedy self-assigned this Feb 23, 2024
@terryjreedy terryjreedy enabled auto-merge (squash) February 23, 2024 01:55
@terryjreedy
Copy link
Member

Test failures were flakey test_queue, since fixed. Set to automerge and backport.

@terryjreedy terryjreedy merged commit b481018 into python:main Feb 23, 2024
33 of 36 checks passed
@miss-islington-app
Copy link

Thanks @ronaldoussoren for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 23, 2024
…ythonGH-112894)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

(cherry picked from commit b481018)

Co-authored-by: Ronald Oussoren <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 23, 2024

GH-115840 is a backport of this pull request to the 3.12 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 23, 2024
…ythonGH-112894)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

(cherry picked from commit b481018)

Co-authored-by: Ronald Oussoren <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Feb 23, 2024
@bedevere-app
Copy link

bedevere-app bot commented Feb 23, 2024

GH-115841 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Feb 23, 2024
terryjreedy added a commit that referenced this pull request Feb 23, 2024
…GH-112894) (#115840)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

(cherry picked from commit b481018)

Co-authored-by: Ronald Oussoren <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
terryjreedy added a commit that referenced this pull request Feb 23, 2024
…GH-112894) (#115841)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

(cherry picked from commit b481018)

Co-authored-by: Ronald Oussoren <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
smontanaro pushed a commit to smontanaro/cpython that referenced this pull request Feb 23, 2024
… macOS (pythonGH-112894)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

(cherry picked from commit b481018)

Co-authored-by: Ronald Oussoren <[email protected]>
Co-authored-by: Terry Jan Reedy <[email protected]>
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
…ython#112894)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

Co-authored-by: Terry Jan Reedy <[email protected]>
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…ython#112894)

The platform standard on macOS is to show a proxy icon for open
files in the titlebar of Windows. Make sure IDLE matches this
behaviour.

Don't use both the long and short names in the window title.
The behaviour of other editors (such as Text Editor) is to show
only the short name with the proxy icon.

Co-authored-by: Terry Jan Reedy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants