Skip to content

Change "Ok" to "OK" in UI - #58744

Merged
danilo-leal merged 2 commits into
zed-industries:mainfrom
albertbogusz:capitalse-ok-in-ui
Jun 8, 2026
Merged

Change "Ok" to "OK" in UI#58744
danilo-leal merged 2 commits into
zed-industries:mainfrom
albertbogusz:capitalse-ok-in-ui

Conversation

@albertbogusz

@albertbogusz albertbogusz commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

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 #58543 and extends to all instances of "Ok" button labels.

Release Notes:

  • N/A

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

@danilo-leal danilo-leal 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.

Thank you, appreciate the eye for detail here! I pushed a commit improving the one case where I feel like we could be clearer than just "OK". Also removed the release notes as I think this is mostly just a UI tweak with no functional added value for the user.

@danilo-leal
danilo-leal enabled auto-merge June 8, 2026 14:39
@danilo-leal
danilo-leal added this pull request to the merge queue Jun 8, 2026
Merged via the queue into zed-industries:main with commit 1d1a4be Jun 8, 2026
34 checks passed
@albertbogusz
albertbogusz deleted the capitalse-ok-in-ui branch June 8, 2026 14:52
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] 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#58543 and extends to all instances of "Ok" button labels.

Release Notes:

- N/A

---------

Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] 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#58543 and extends to all instances of "Ok" button labels.

Release Notes:

- N/A

---------

Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants