RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 11:07:35 +00:00
Code Issues Activity Actions

LibWeb: Return overflow rect height from Element::scroll_height()

Browse source 
Spec says that this function has to return "height of the element
scrolling area" which is height of "scrolling overflow rect" in our
model.
This commit is contained in:
Aliaksandr Kalenik 2024-02-16 01:22:52 +01:00 • committed by Andreas Kling
parent 9f581d0bc9
commit 591c8d2b68
3 changed files with 27 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/DOM/element-scroll-height.txt Normal file
View file

@ -0,0 +1 @@
Item 1Item 2Item 3Item 4Item 5Item 6Item 7Item 8Item 9Item 10Item 11Item 12 1200

25
Tests/LibWeb/Text/input/DOM/element-scroll-height.html Normal file
View file

@ -0,0 +1,25 @@
<!DOCTYPE html>
<script src="../include.js"></script>
<style>
* {
outline: 1px solid black;
}
.scrollable {
height: 300px;
overflow-y: auto;
}
span {
width: 100px;
height: 100px;
display: block
}
</style>
<div class="scrollable"><span>Item 1</span><span>Item 2</span><span>Item 3</span><span>Item 4</span><span>Item 5</span><span>Item 6</span><span>Item 7</span><span>Item 8</span><span>Item 9</span><span>Item 10</span><span>Item 11</span><span>Item 12</span></div>
<script>
test(() => {
const scrollable = document.querySelector(".scrollable");
println(scrollable.scrollHeight);
});
</script>

2
Userland/Libraries/LibWeb/DOM/Element.cpp
View file

@ -1430,7 +1430,7 @@ int Element::scroll_height() const
return 0; return 0;
// 7. Return the height of the element’s scrolling area. // 7. Return the height of the element’s scrolling area.
return paintable_box()->border_box_height().to_int(); return paintable_box()->scrollable_overflow_rect()->height().to_int();
} }
// https://html.spec.whatwg.org/multipage/semantics-other.html#concept-element-disabled // https://html.spec.whatwg.org/multipage/semantics-other.html#concept-element-disabled