beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Compare anchor/focus offsets in selection.isCollapsed

Browse source 
The "isCollapsed" attribute on a selection must "return true if and only
if the anchor and focus are the same".

In addition to checking that the anchor and focus belonged to the same
DOM node, we now also check that they refer to the same position within
the node.

With this change Ladybird passes all the subtests in the "isCollapsed"
WPT suite.

https://wpt.live/selection/isCollapsed.html
This commit is contained in:
John Diamond 2024-10-12 02:42:55 +02:00 committed by Andreas Kling
parent 27b1d94e04
commit fadb14d31d
Notes: github-actions[bot] 2024-10-12 13:01:26 +00:00 
Author: https://github.com/jdmnd 🔰
Commit: https://github.com/LadybirdBrowser/ladybird/commit/fadb14d31d0
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/1737
3 changed files with 32 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/is-collapsed.txt Normal file
View file

@ -0,0 +1 @@
PASS

27
Tests/LibWeb/Text/input/is-collapsed.html Normal file
View file

@ -0,0 +1,27 @@
<script src="include.js"></script>
<p id="a">Simply selectable</p>
<script>
test(() => {
var selection = window.getSelection();
var range = document.createRange();
range.setStart(a.firstChild, 0);
range.setEnd(a.firstChild, 1);
selection.addRange(range);
if (selection.isCollapsed) {
println(`FAIL: "${selection}" is not collapsed`);
return;
}
selection.collapseToStart();
if (!selection.isCollapsed) {
println(`FAIL: "${selection}" is collapsed`);
return;
}
selection.removeAllRanges();
if (!selection.isCollapsed) {
println(`FAIL: "${selection}" is collapsed`);
return;
}
println("PASS");
})
</script>

5
Userland/Libraries/LibWeb/Selection/Selection.cpp
View file

@ -96,7 +96,10 @@ bool Selection::is_collapsed() const
{
// The attribute must return true if and only if the anchor and focus are the same
// (including if both are null). Otherwise it must return false.
return const_cast<Selection*>(this)->anchor_node() == const_cast<Selection*>(this)->focus_node();
if (!m_range)
return true;
return const_cast<Selection*>(this)->anchor_node() == const_cast<Selection*>(this)->focus_node()
&& m_range->start_offset() == m_range->end_offset();
}
// https://w3c.github.io/selection-api/#dom-selection-rangecount