Skip to content

more setColumn() tweaks - #1103

Merged
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop
Dec 20, 2019
Merged

more setColumn() tweaks#1103
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop

Conversation

@adumesny

@adumesny adumesny commented Dec 20, 2019

Copy link
Copy Markdown
Member

Description

#1098 - part 3

  • restored _updateNodeWidths() which setColumn() calls (split code into GridStackEngine)
  • added _layoutsNodesChange(nodes) - called whenever a node is added or moved to update the cached layouts
    (clears lower one, update higher ones)

Checklist

  • Created tests which fail without the change (if possible)
  • All tests passing (yarn test)
  • Extended the README / documentation, if necessary

* restored `_updateNodeWidths()` which setColumn() calls (split code into GridStackEngine)
* added `_layoutsNodesChange(nodes)` - called whenever a node is added or moved to update the cached layouts
(clears lower one, update higher ones)
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 66.988% when pulling cc9043f on adumesny:develop into 79ec5fe on gridstack:develop.

@adumesny
adumesny merged commit 385ee1c into gridstack:develop Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants