Skip to content

swap option for draw()#86

Merged
texodus merged 1 commit intomasterfrom
staging-light-dom
Sep 7, 2020
Merged

swap option for draw()#86
texodus merged 1 commit intomasterfrom
staging-light-dom

Conversation

@texodus
Copy link
Copy Markdown
Member

@texodus texodus commented Sep 7, 2020

Adds the swap option for the draw() method, which replaces the existing constant value. With this change, <table> swapping will only occur when this flag is set.

table.draw({swap: true});

In addition, the container element used for 'staging" changes during a swap has been moved to the light DOM, as a child of <regular-table>, sibling to the visible <table> container element. This allows CSS style which may impact layout, such as font size and cell dimensions, to be calculated correctly in swapped mode; as a side effect however, naive of async calls to querySelector() may resolve both hidden and cloned visible table elements.

@texodus texodus changed the title invalid option for draw() swap option for draw() Sep 7, 2020
@texodus texodus merged commit c863317 into master Sep 7, 2020
@texodus texodus deleted the staging-light-dom branch September 7, 2020 02:52
@texodus texodus added the enhancement Feature requests or improvements label Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Feature requests or improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant