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
Remove hspace and vspace attributes from input field #8083
Remove hspace and vspace attributes from input field #8083
Conversation
EWS run on previous version of this PR (hash 62d8b9a) |
@karlcow - Sweet!! Win Win!!! |
62d8b9a
to
fd22a4c
Compare
EWS run on current version of this PR (hash fd22a4c) |
@anttijk Next step. merge-queue. π |
https://bugs.webkit.org/show_bug.cgi?id=244279 rdar://99356718 Reviewed by Antti Koivisto. Removes hspace and vspace attribute mapping to CSS, but keeps it in the case of input of type=image as mandated by the WPT test in http://wpt.live/html/rendering/unmapped-attributes.html but still passing http://wpt.live/html/rendering/dimension-attributes.html * LayoutTests/imported/w3c/web-platform-tests/html/rendering/unmapped-attributes-expected.txt: * Source/WebCore/html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::collectPresentationalHintsForAttribute): Canonical link: https://commits.webkit.org/258360@main
fd22a4c
to
9065879
Compare
Committed 258360@main (9065879): https://commits.webkit.org/258360@main Reviewed commits have been landed. Closing PR #8083 and removing active labels. |
} else if (name == hspaceAttr) { | ||
if (isImageButton()) { | ||
addHTMLLengthToStyle(style, CSSPropertyMarginLeft, value); | ||
addHTMLLengthToStyle(style, CSSPropertyMarginRight, value); | ||
} |
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.
can the indentation be fixed up?
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.
ooopsie. @nt1m what's the best way to do that usually. Open a new webkit bug with a fix?
9065879
fd22a4c
π§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-AS-debug-wk2