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

[html/en] Add more tags #4904

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[html/en] Add more tags #4904

wants to merge 1 commit into from

Conversation

Th3G33k
Copy link
Contributor

@Th3G33k Th3G33k commented Apr 21, 2024

meta, script, style, div, form, iframe

  • I solemnly swear that this is all original content of which I am the original author
  • Pull request title is prepended with [language/lang-code] (example [python/fr-fr] or [java/en])
  • Pull request touches only one file (or a set of logically related files with similar changes made)
  • Content changes are aimed at intermediate to experienced programmers (this is a poor format for explaining fundamental programming concepts)
  • If you've changed any part of the YAML Frontmatter, make sure it is formatted according to CONTRIBUTING.md
    • Yes, I have double-checked quotes and field names!

meta, script, style, div, form, iframe
@verhovsky
Copy link
Collaborator

I don't like how this is adding a third file-within-the-file. LXIYM should just be a single commented file, but I can see the value of having a simple version of the HTML document and then a commented version, but adding a third one doesn't seem right.

@Th3G33k
Copy link
Contributor Author

Th3G33k commented May 24, 2024

Thanks for the review.

It is already separated in three parts. The third part starts from here :

<!-- And that's it, creating an HTML file can be simple. -->

<!-- But it is possible to add many additional types of HTML tags. -->

<!-- The <img /> tag is used to insert an image. -->

I was trying to add more useful tags.

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.

2 participants