system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-01 11:19:21 +00:00
Code Issues Projects Releases Wiki Activity

LibWeb: Remove unnecessary FIXME marker for CSSStyleDeclaration.cssFloat

Browse Source 
(cherry picked from commit 4c94202e9734099b6e2839f5495b8280eec2ab2f)
This commit is contained in:
Andreas Kling 2024-06-14 17:13:31 +02:00 committed by Nico Weber
parent 99851591b5
commit 7cd1114f05
3 changed files with 19 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Tests/LibWeb/Text/expected/css/cssFloat-property.txt Normal file
View File

@ -0,0 +1,2 @@
spanRule: [object CSSStyleRule] ~ span { float: right; }
spanRule.style.cssFloat: right

13
Tests/LibWeb/Text/input/css/cssFloat-property.html Normal file
View File

@ -0,0 +1,13 @@
<style>
span {
float: right;
}
</style>
<script src="../include.js"></script>
<script>
test(() => {
const spanRule = document.styleSheets[0].cssRules[0];
println("spanRule: " + spanRule + " ~ " + spanRule.cssText);
println("spanRule.style.cssFloat: " + spanRule.style.cssFloat);
});
</script>

5
Userland/Libraries/LibWeb/CSS/CSSStyleDeclaration.idl
View File

@ -14,6 +14,9 @@ interface CSSStyleDeclaration {
[CEReactions] CSSOMString removeProperty(CSSOMString property);
readonly attribute CSSRule? parentRule;
[FIXME, CEReactions, LegacyNullToEmptyString] attribute CSSOMString cssFloat;
// NOTE: cssFloat is implemented manually in the bindings code, along with the other property accessors.
// Hence, this does not need to be implemented.
// [CEReactions, LegacyNullToEmptyString] attribute CSSOMString cssFloat;
};