Skip to content

Add charset attribute to index.html#59

Merged
theimowski merged 2 commits intoSAFE-Stack:masterfrom
marcpiechura:patch-1
Apr 19, 2018
Merged

Add charset attribute to index.html#59
theimowski merged 2 commits intoSAFE-Stack:masterfrom
marcpiechura:patch-1

Conversation

@marcpiechura
Copy link
Copy Markdown
Contributor

I had to set the charset to utf8 in order to display str “äöü“ correctly, otherwise it fallbacks to ascii?! which looks quite ugly ;-)

Comment thread Content/src/Client/index.html Outdated
<title>SAFE Template</title>
<!--#if (Fulma != "none")
<meta name="viewport" content="width=device-width, initial-scale=1">
<meta charset="utf-8" name="viewport" content="width=device-width, initial-scale=1">
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.

I think I did that recently in my code as well and html validation said it should be in a separate meta tag.

@marcpiechura
Copy link
Copy Markdown
Contributor Author

marcpiechura commented Apr 19, 2018

@forki addressed your remarks, also closed the elmish div correctly, according to https://validator.w3.org <div /> isn't valid either.

@theimowski theimowski merged commit 4f1d7ad into SAFE-Stack:master Apr 19, 2018
@theimowski
Copy link
Copy Markdown
Member

Thanks! released in 0.13.1

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