Closed
Bug 1761159
Opened 2 years ago
Closed 2 years ago
Make SplitNodeResult constructors logical
Categories
(Core :: DOM: Editor, task, P3)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
100 Branch
Tracking | Status | |
---|---|---|
firefox100 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
Attachments
(1 file)
Currently, they take content nodes as physical position from a split point. However, for making the creators simpler, they should take the content nodes as logical, i.e., a new node and a original node.
Assignee | ||
Comment 1•2 years ago
|
||
Like JoinNodesResult
, SplitNodeResult
should take 2 content nodes as
"new one" and "original one" instead of "previous one" and "next one".
Then, the creators is easier to read.
Additionally, this patch creates some static methods and converter method
to create a result with explaining the result meaning.
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/30b727170d71 Make constructors of `SplitNodeResult` take content nodes with logical meaning rather than physical meaning r=m_kato
Comment 3•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox100:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•