Skip to content

Conversation

@harshsomankar123-tech
Copy link
Contributor

@harshsomankar123-tech harshsomankar123-tech commented Jan 6, 2026

Purpose / Description

The Browser Options dialog contains text using inconsistent capitalization (e.g. title case instead of sentence case), which differs from surrounding UI text and affects visual consistency.

This PR aligns Browser Options strings with the established sentence case style used across the app.

Fixes #15760

Approach

  • Identified strings used directly by the Browser Options dialog
  • Updated only those strings to sentence case
  • Left adjacent and unrelated strings unchanged to avoid expanding scope

How Has This Been Tested?

  • Manually verified the Browser Options dialog UI
  • Confirmed no other screens or features are affected

Checklist

  • Descriptive commit message with a short title
  • Self-reviewed the changes
  • Scope limited strictly to Browser Options
  • No functional or behavioral changes
  • UI change verified manually

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Important

Maintainers: This PR contains Strings changes

  1. Sync Translations before merging this PR and wait for the action to complete
  2. Review and merge the auto-generated PR in order to sync all user-submitted translations
  3. Sync Translations again and merge the PR so the huge automated string changes caused by merging this PR are by themselves and easy to review

@david-allison
Copy link
Member

This seems fine, but I'm not sure I understand the scope, this is well outside just the scope of Browser Options, but there's some adjacent strings which haven't been handled in the files which are edited.

@harshsomankar123-tech
Copy link
Contributor Author

Thanks for the review!
The intent here is limited to Browser Options. let me know if you’d like any scope adjustments.

@david-allison david-allison added Needs Second Approval Has one approval, one more approval to merge and removed Needs Second Approval Has one approval, one more approval to merge labels Jan 6, 2026
Copy link
Member

@david-allison david-allison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@david-allison david-allison added Needs Author Reply Waiting for a reply from the original author Needs Second Approval Has one approval, one more approval to merge labels Jan 6, 2026
@harshsomankar123-tech
Copy link
Contributor Author

Thanks!
I’ve updated the PR description using the template and kept the scope limited to Browser Options only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Author Reply Waiting for a reply from the original author Needs Second Approval Has one approval, one more approval to merge Strings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Design] Inconsistent String Casing

2 participants