Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Humanize spell check language labels #12409

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Apr 9, 2024

Like they are in the dropdown used to add languages

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Apr 9, 2024
@t3chguy t3chguy requested a review from a team as a code owner April 9, 2024 12:23
@t3chguy t3chguy requested review from richvdh and florianduros April 9, 2024 12:23
@t3chguy t3chguy self-assigned this Apr 9, 2024
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

lgtm modulo a couple of nits

@@ -87,8 +90,9 @@ export default class SpellCheckLanguages extends React.Component<SpellCheckLangu
};

public render(): React.ReactNode {
const intl = new Intl.DisplayNames([getUserLanguage()], { type: "language", style: "short" });
Copy link
Member

Choose a reason for hiding this comment

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

does style: "short" actually do anything here, ooi?

If not, I'd favour omitting it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy enabled auto-merge April 10, 2024 22:13
@t3chguy t3chguy added this pull request to the merge queue Apr 10, 2024
Merged via the queue into develop with commit c40fea0 Apr 10, 2024
22 checks passed
@t3chguy t3chguy deleted the t3chguy/spell-check-langs branch April 10, 2024 22:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants