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

Internationalization: Set lang of HTML page to user language preference #74513

Merged
merged 1 commit into from Sep 7, 2023

Conversation

ypnos
Copy link
Contributor

@ypnos ypnos commented Sep 7, 2023

What is this feature?

Set "lang" attribute of <html> element to the user language setting.

Why do we need this feature?

HTML "lang" attribute should reflect the actual language used in texts.
It is important for accessibility functions (screen readers), but also usable in CSS.
Panel plugins can also read the lang attribute for i18n.

Who is this feature for?

International users.

@ypnos ypnos requested a review from a team as a code owner September 7, 2023 08:43
@ypnos ypnos requested review from tskarhed and ashharrison90 and removed request for a team September 7, 2023 08:43
@CLAassistant
Copy link

CLAassistant commented Sep 7, 2023

CLA assistant check
All committers have signed the CLA.

@grafana-pr-automation grafana-pr-automation bot added area/frontend pr/external This PR is from external contributor labels Sep 7, 2023
Copy link
Contributor

@joshhunt joshhunt left a comment

Choose a reason for hiding this comment

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

Seems like a good idea!

public/views/index-template.html Outdated Show resolved Hide resolved
@joshhunt joshhunt added no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes labels Sep 7, 2023
HTML "lang" attribute should reflect the actual language used in texts. Important for accessibility functions (screen readers), but also usable in CSS.
@joshhunt joshhunt added this to the 10.2.x milestone Sep 7, 2023
@joshhunt joshhunt enabled auto-merge (squash) September 7, 2023 14:31
@joshhunt joshhunt removed the no-changelog Skip including change in changelog/release notes label Sep 7, 2023
@joshhunt joshhunt changed the title Internationalization: Adapt document language Internationalization: Set lang of HTML page to user language preference Sep 7, 2023
Copy link
Contributor

@joshhunt joshhunt left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@joshhunt joshhunt merged commit 1f54342 into grafana:main Sep 7, 2023
16 of 17 checks passed
chauchausoup pushed a commit to chauchausoup/grafana that referenced this pull request Sep 15, 2023
Adapt document language

HTML "lang" attribute should reflect the actual language used in texts. Important for accessibility functions (screen readers), but also usable in CSS.
rwwiv pushed a commit that referenced this pull request Oct 2, 2023
Adapt document language

HTML "lang" attribute should reflect the actual language used in texts. Important for accessibility functions (screen readers), but also usable in CSS.
@zerok zerok modified the milestones: 10.2.x, 10.2.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/frontend no-backport Skip backport of PR pr/external This PR is from external contributor
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants