Skip to content
This repository has been archived by the owner on Feb 25, 2021. It is now read-only.

Eliminate wrapper elements #602

Merged
merged 5 commits into from Apr 18, 2018
Merged

Eliminate wrapper elements #602

merged 5 commits into from Apr 18, 2018

Commits on Apr 18, 2018

  1. Factor out all DOM-structure modifying code into an abstraction. Curr… 

    …ently, implementation is still backed by the same underlying DOM APIs
    @SteveSandersonMS
    SteveSandersonMS committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    532782f View commit details
    Browse the repository at this point in the history

  2. Implement LogicalElement properly (store hierarchy in own property; u… 

    …se comment nodes as containers)
    @SteveSandersonMS
    SteveSandersonMS committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    451b5a4 View commit details
    Browse the repository at this point in the history

  3. In LogicalElement, be explicit about unsupported scenarios

    @SteveSandersonMS
    SteveSandersonMS committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    953b8c3 View commit details
    Browse the repository at this point in the history

  4. Update E2E tests to stop assuming existence of wrapper elements

    @SteveSandersonMS
    SteveSandersonMS committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    a8e710b View commit details
    Browse the repository at this point in the history

  5. Where supported, store private properties using opaque symbols as keys

    @SteveSandersonMS
    SteveSandersonMS committed Apr 18, 2018
    Configuration menu
    Copy the full SHA
    733b0d0 View commit details
    Browse the repository at this point in the history