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

Clean up desktop's index.html #236

Merged
merged 1 commit into from
Feb 12, 2022

Conversation

asaaki
Copy link
Contributor

@asaaki asaaki commented Feb 12, 2022

Initially I wanted to delete only the duplicated <html> opening tag, but then decided to make it a fully valid HTML document.

Passes https://validator.w3.org/nu/#textarea check.

Initially I wanted to delete only the duplicated `<html>` opening tag,
but then decided to make it a fully valid HTML document.

Passes https://validator.w3.org/nu/#textarea check.
</script>

<head>
<title>Dioxus app</title>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not strictly necessary for dioxus, but turns this file into a fully valid document.
It get's overwritten by dioxus or the user's application configuration anyway.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know should we add this title tag. but maybe it doesn't matter.

@mrxiaozhuox
Copy link
Contributor

looks good, you have format the HTML code.

@codecov
Copy link

codecov bot commented Feb 12, 2022

Codecov Report

Merging #236 (b645748) into master (65ca542) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   36.96%   36.96%           
=======================================
  Files          63       63           
  Lines        4880     4880           
=======================================
  Hits         1804     1804           
  Misses       3076     3076           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 65ca542...b645748. Read the comment docs.

@jkelleyrtp jkelleyrtp merged commit 2591ef2 into DioxusLabs:master Feb 12, 2022
@jkelleyrtp
Copy link
Member

Thank you!

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