-
Notifications
You must be signed in to change notification settings - Fork 216
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
Add missing <meta charset="UTF-8"> and make HTML standard #58
Conversation
- Remove unnecessary codes meet the standard coding for html. - Adding `user-scalable=no` to meta `viewport` to prevent users to scale webview, because it is work as an standard app.
- Add missing `<meta charset="UTF-8">` to index.html
- Add missing `<meta charset="UTF-8">` to index.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, except for the charset issue I commented on.
Looks good now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good now. Thank you very much for your contribution! 🥇
Thank you my friend :) |
Anyway lowercase or uppercase not affect the code. And if you wrote it in small letter, it also work. Also in xml charset default written in UTF-8 Also for doctype html It is the same, but the default is DOCTYPE html You can refer to those standards from w3.org, or w3schools Regards |
<meta charset="UTF-8">
to<head>
section.type="text/css"
,type="text/javascript"
. Because those types is default inHTML5
user-scalable=no
to metaviewport
to prevent users to scale webview, because by default, mobile apps should not allow users to scale the main webview.Platforms affected
Motivation and Context
Description
Testing
Checklist
(platform)
if this change only applies to one platform (e.g.(android)
)