Skip to content

Add definition for composed selection range #1342

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.

Sign up for GitHub

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

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Add definition for composed selection range #1342

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
7c48fa5
Add definition for composed live range
dizhang168 Dec 17, 2024
487cac0
Review, format changes
dizhang168 Jan 21, 2025
461aed7
Fix collapse logic for set the start or end
dizhang168 Jan 23, 2025
413348d
Fix for= typo
dizhang168 Jan 24, 2025
c7f38b1
Linking nit
domfarolino Jan 28, 2025
1f74a51
Remove em dash to fix spacing in rendered spec
domfarolino Jan 28, 2025
f5b6935
Review changes: rename composed live range, improve format of set the…
dizhang168 Jan 28, 2025
367288b
define composed selection range separately from live range
dizhang168 Feb 25, 2025
fe020ca
Update composed selection range definition
dizhang168 Feb 27, 2025
4760e78
Change to compare with shadow-including root instead of document node
dizhang168 Mar 5, 2025
d8ca5a5
legacy uncomposed range + format
dizhang168 Mar 13, 2025
99b212b
Merge remote-tracking branch 'upstream/main' into composed-live-range
dizhang168 Mar 21, 2025
700ca5e
Add composed position definition, rewrite set the start/end
dizhang168 Mar 24, 2025
1cf2b12
fix typo
dizhang168 Mar 24, 2025
59d9595
Name is composed range + improve spec language
dizhang168 Mar 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
130 changes: 75 additions & 55 deletions dom.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8149,6 +8149,41 @@ be between 0 and the <a>boundary point</a>'s <a for="boundary point">node</a>'s
</ol>


<p>The
<dfn export id=concept-range-bp-composed-position for="boundary point">composed position</dfn> of a
<a>boundary point</a> (<var>nodeA</var>, <var>offsetA</var>) relative to a <a>boundary point</a>
(<var>nodeB</var>, <var>offsetB</var>) is
<dfn export id=concept-range-bp-composed-before for="boundary point">composed before</dfn>,
<dfn export id=concept-range-bp-composed-equal for="boundary point">composed equal</dfn>, or
<dfn export id=concept-range-bp-composed-after for="boundary point">composed after</dfn>,
as returned by these steps:

<ol>
<li><p>Assert: <var>nodeA</var> and <var>nodeB</var> have the same <a>shadow-including root</a>.

<li>If <var>nodeA</var> comes before <var>nodeB</var> in <a>shadow-including tree order</a>, then
return <a for="boundary point">composed before</a>.

<li>If <var>nodeA</var> comes after <var>nodeB</var> in <a>shadow-including tree order</a>, then
return <a for="boundary point">composed after</a>.

<li>
<p>If the <a for="boundary point">position</a> of (<var>nodeA</var>, <var>offsetA</var>)
relative to (<var>nodeB</var>, <var>offsetB</var>) is:

<dl class=switch>
<dt><a for="boundary point">before</a>
<dd>Return <a for="boundary point">composed before</a>.

<dt><a for="boundary point">equal</a>
<dd>Return <a for="boundary point">composed equal</a>.

<dt><a for="boundary point">after</a>
<dd>Return <a for="boundary point">composed after</a>.
</dl>

</ol>

<h3 id=interface-abstractrange>Interface {{AbstractRange}}</h3>

<pre class=idl>
Expand Down Expand Up @@ -8315,15 +8350,14 @@ interface Range : AbstractRange {
<p>Objects implementing the {{Range}} interface are known as
<dfn export id=concept-live-range>live ranges</dfn>.

<p>A <dfn export id=concept-composed-selection-range>composed selection range</dfn> is a
<a>range</a> with an associated <a>live range</a> called
<dfn export id=concept-legacy-uncomposed-range for="composed selection range">legacy uncomposed range</dfn>.

<p class=note>Algorithms that modify a <a>tree</a> (in particular the <a for=/>insert</a>,

<a for=/>remove</a>, <a for=/>move</a>, <a>replace data</a>, and <a lt="split a Text node">split</a>
algorithms) modify <a>live ranges</a> associated with that <a>tree</a>.

<p>A <dfn export id=concept-composed-selection-range>composed selection range</dfn> is a
<a>range</a> with an associated <a>live range</a> called
<dfn export id=concept-legacy-uncomposed-range for="composed selection range">legacy uncomposed range</dfn>.

<p class=note>The <a for="composed selection range">legacy uncomposed range</a> is used to maintain
backward compatibility with the {{Selection/getRangeAt()}} API.

Expand Down Expand Up @@ -8493,74 +8527,60 @@ set the start or end</dfn> of a <a>live range</a> <var>range</var> to a <a>bound
<ol>
<li>If <var>range</var>'s
<a for="live range">root</a> is not equal to
<var>node</var>'s <a for=tree>root</a>, then:
<ol>
<li>Set <var>range</var>'s <a for=range>end</a>
to <var>bp</var>.
<var>node</var>'s <a for=tree>root</a>,
or if <var>bp</var> is
<a for="boundary point">after</a> the
<var>range</var>'s <a for=range>end</a>, set
<var>range</var>'s <a for=range>end</a>
to <var>bp</var>.

<li>If <var>composed range</var> is not null and
<var>range</var>'s <a for=range>start node</a>'s
<li>Set <var>range</var>'s
<a for=range>start</a> to <var>bp</var>.

<li>f <var>composed range</var> is not null, then:
<ol>
<li>If <var>range</var>'s <a for=range>start node</a>'s
<a>shadow-including root</a> is not
<var>node</var>'s <a>shadow-including root</a>,
set <var>composed range</var>’s
or if <var>bp</var> is
<a for="boundary point">composed after</a> the
<var>range</var>'s <a for=range>end</a>, set
<var>composed range</var>'s
<a for="range">end</a> to <var>bp</var>.
</ol>

<li>Otherwise, if <var>bp</var> is
<a for="boundary point">after</a> the
<var>range</var>'s <a for=range>end</a>, then:
<ol>
<li>Set <var>range</var>'s <a for=range>end</a>
to <var>bp</var>.

<li>If <var>composed range</var> is not null,
set <var>composed range</var>’s
<a for="range">end</a> to <var>bp</var>.
<li>Set <var>composed range</var>’s
<a for="range">start</a> to <var>bp</var>.
</ol>

<li>Set <var>range</var>'s
<a for=range>start</a> to <var>bp</var>.

<li>If <var>composed range</var> is not null,
set <var>composed range</var>’s
<a for="range">start</a> to <var>bp</var>.
</ol>
<dt>If these steps were invoked as "set the end"
<dd>
<ol>
<li>If <var>range</var>'s
<a for="live range">root</a> is not equal to
<var>node</var>'s <a for=tree>root</a>, then:
<ol>
<li>Set <var>range</var>'s <a for=range>start</a>
to <var>bp</var>.
<var>node</var>'s <a for=tree>root</a>,
or if <var>bp</var> is
<a for="boundary point">before</a> the
<var>range</var>'s <a for=range>start</a>, set
<var>range</var>'s <a for=range>start</a>
to <var>bp</var>.

<li>Set <var>range</var>'s
<a for=range>end</a> to <var>bp</var>.

<li>If <var>composed range</var> is not null and
<var>range</var>'s <a for=range>start node</a>'s
<li>f <var>composed range</var> is not null, then:
<ol>
<li>If <var>range</var>'s <a for=range>end node</a>'s
<a>shadow-including root</a> is not
<var>node</var>'s <a>shadow-including root</a>,
set <var>composed range</var>’s
or if <var>bp</var> is
<a for="boundary point">composed before</a> the
<var>range</var>'s <a for=range>start</a>, set
<var>composed range</var>'s
<a for="range">start</a> to <var>bp</var>.
</ol>

<li>Otherwise, if <var>bp</var> is
<a for="boundary point">before</a> the
<var>range</var>'s <a for=range>start</a>, then:
<ol>
<li>Set <var>range</var>'s <a for=range>start</a>
to <var>bp</var>.

<li>If <var>composed range</var> is not null,
set <var>composed range</var>’s
<a for="range">start</a> to <var>bp</var>.
<li>Set <var>composed range</var>’s
<a for="range">end</a> to <var>bp</var>.
</ol>

<li>Set <var>range</var>'s
<a for=range>end</a> to <var>bp</var>.

<li>If <var>composed range</var> is not null,
set <var>composed range</var>’s
<a for="range">end</a> to <var>bp</var>.
</ol>
</dl>
</ol>
Expand Down