-
Notifications
You must be signed in to change notification settings - Fork 13
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
New definition of "H", "L", "M", "T", new feature FORMS
(if needed)
#56
Conversation
what should I do? |
Thanks! I just need some time to look into this. |
It looks like there are already better definitions for all of the characters. Am I missing something? |
you mean there is something better than |
I think so. At least I could not find a case where the new definitions produce shorter code for some combination of browsers. |
the expected combination is browsers except old Android web view. |
|
can we just treat that's part of |
I think it's easier to treat it as a separate feature. If we make |
and, the "V" definition based on |
or should I add new simulation now? what should I do? |
Yes, that would be better. I will split a new feature. |
Co-authored-by: Francesco Trotta <github@fasttime.org>
Co-authored-by: Francesco Trotta <github@fasttime.org>
And, how to simulate? |
To emulate
for example: document =
{
createElement: createElement,
forms: '[object HTMLCollection]',
nodeName: '#document',
}; |
Co-authored-by: Francesco Trotta <github@fasttime.org>
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.
Nice work, thanks!
afaik from MDN,
document.forms+[]
returns to "[object HTMLCollection]" in most browsers, but idk how this actually work in old Android webview, so I created a new feature just in case (idk how to simulate, tho). btw, afaik,forms
have a relatively shorter code in those similar properties ofdocument
, but idk is that the shortest.