Skip to content

settings_ui: Respect window decorations setting#57371

Merged
kubkon merged 1 commit into
zed-industries:mainfrom
hollmmax:config-over-env
May 21, 2026
Merged

settings_ui: Respect window decorations setting#57371
kubkon merged 1 commit into
zed-industries:mainfrom
hollmmax:config-over-env

Conversation

@hollmmax

Copy link
Copy Markdown
Contributor

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Closes #57370

Release Notes:

  • Server-side window decorations now also apply to the settings window.

@cla-bot

cla-bot Bot commented May 21, 2026

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Max Hollmann.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@zed-community-bot zed-community-bot Bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label May 21, 2026
@cla-bot

cla-bot Bot commented May 21, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @hollmmax on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@hollmmax

Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 21, 2026
@cla-bot

cla-bot Bot commented May 21, 2026

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@maxdeviant maxdeviant changed the title ui: Make settings ui look at settings of ui window decorations settings_ui: Respect window decorations setting May 21, 2026

@kubkon kubkon 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.

Nice one, thank you!

@kubkon kubkon enabled auto-merge May 21, 2026 10:38
@kubkon kubkon added this pull request to the merge queue May 21, 2026
Merged via the queue into zed-industries:main with commit 0ec218f May 21, 2026
41 checks passed
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 2, 2026
Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Closes zed-industries#57370

Release Notes:

- Server-side window decorations now also apply to the settings window.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings window ignores window decoration setting.

2 participants