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

Update base.html #3061

Closed
wants to merge 1 commit into from
Closed

Conversation

watarutmnh
Copy link
Contributor

Fixed a bug where the <html lang> attribute was hard-set to en-US rather than the current application language. (Refer: #3053)

Fixed a bug where the `<html lang>` attribute was hard-set to `en-US` rather than the current application language. (Refer: craftcms#3053)
@lukeyouell
Copy link

@watarutmnh this has already been changed as per your request, no?

#3053

@watarutmnh
Copy link
Contributor Author

@lukeyouell You are right, but I wanted to change to Craft2 too.

@brandonkelly
Copy link
Member

Fix is going to be a little more complicated in Craft 2 since craft.locale will return locale IDs in the format en_gb rather than en-GB.

@watarutmnh
Copy link
Contributor Author

watarutmnh commented Jul 5, 2018

@brandonkelly I see the point. it's much more complicate than I thought. For example, en_bw the locale ID, should not convert to en-BW. It should be "BW" to along with ISO 3166 format.
I'm grad that Craft 3 had solved this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants