Skip to content

main: update constants #414

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

titoBouzout
Copy link
Contributor

@titoBouzout titoBouzout commented May 2, 2025

Updates DOM constants in branch main

  • add some new boolean properties/attributes (including missing allowFullscreen , defer)
  • update prop aliases to add new properties (including missing allowFullscreen )
  • add webview to html tags
  • add some missing deprecated tags (as the list includes deprecated tags, I added the ones I know are missing, following suit)

Question: Should we include the deprecated/non-standard? The later list is case-sensitive so must be included in PropAliases increasing the size considerably. In my opinion shouldn't be included, but in theory in next we can support this without barcoding it.

compact
declare
incremental
webkitdirectory

allowPaymentRequest
mozOpaque
noHref
noResize
noShade
noWrap
trueSpeed

Note: if the PR looks good to you, can you please let me know before merging, so I sort the lists, I avoided sorting the lists because the diffs become harder to read. By sorting it will make it easier in the future to check for differences

Thanks!

…`webview`, add some missing deprecated html tags for completness
@titoBouzout titoBouzout marked this pull request as draft May 2, 2025 09:40
@titoBouzout titoBouzout marked this pull request as ready for review May 2, 2025 09:42
@titoBouzout
Copy link
Contributor Author

Something weird is going on with the tests in GitHub. In local, I see no errors.

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.

1 participant