Bug 19007 - Consider adding a swap() method
Consider adding a swap() method
Status: NEW
Product: WebAppsWG
Classification: Unclassified
Component: DOM
unspecified
PC Windows 3.1
: P2 normal
: ---
Assigned To: Anne
public-webapps-bugzilla
blocked on insufficient interest
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-25 12:52 UTC by Mike Kamermans
Modified: 2014-05-22 14:03 UTC (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Kamermans 2012-09-25 12:52:46 UTC
The new DOM mutation methods are excellently great, but I would like to request the addition of one more function: swap.

Particularly for DOM diffing (turning one element into another) where the operations are removals, insertions and relocations, as well as "collection" relocations (moving a sequence of elements up/down in their container) a swap() operation will greatly improve legibility as well as writability.

Compare these two DOM fragments:

A:

<figure>
  <!--image-->
  <img...>
  <!-- guide text -->
  <figcaption>text</figcaption>
<figure>

B:

<figure>
  <!-- guide text -->
  <figcaption>text</figcaption>
  <!--image-->
  <img...>
<figure>

Turning A into B (as a diff, or because a user did something that was equivalent to a drag operation on the first two or last two non-Text nodes) using a .swap() operation would be a two line constructions:

  figure.swap(1,5); // or figure.swap(commentElement1, commentElement2)
  figure.swap(2,6); // or figure.swap(imgelement, figCaptionElement)

This code would be preferable to having to relocate elements using inserts, where each insert must be followed by checks to see whether a relocation now requires an insert, or an append because the next swap can be performed.

It would also reduce the number of DOM mutation events being generated for a swap from two events that are either the same (two inserts) or different (an insert and an append) to a single event that always represents a swap.
Comment 1 Mike Kamermans 2012-09-25 12:57:53 UTC
There is no way to edit a bug text, so:

s/two line constructions/two line construction/
s/an append because/an append before/