Skip to content

new compact() method, tweak setColumn()#1101

Merged
adumesny merged 3 commits intogridstack:developfrom
adumesny:develop
Dec 20, 2019
Merged

new compact() method, tweak setColumn()#1101
adumesny merged 3 commits intogridstack:developfrom
adumesny:develop

Conversation

@adumesny
Copy link
Copy Markdown
Member

@adumesny adumesny commented Dec 20, 2019

Description

new compact() method

  • relayout grid items to reclaim any empty space
  • update two.html demo to have a compact button, and test case

also (separate PR)
tweaks to setColumn() - #1098 part 2

  • improved code, now using gridEngine.addNode() instead of removeAll() + addWidget() which is much lighter now
  • use a higher res layout when going up and cache is missing
    (better to down-sample say from 12 then up-sample from 1!)

Checklist

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

* relayout grid items to reclaim any empty space
* update two.html demo to have a compact buton, and test case
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 20, 2019

Coverage Status

Coverage increased (+0.7%) to 66.029% when pulling 30dc3d2 on adumesny:develop into e91937e on gridstack:develop.

Alain Dumesny added 2 commits December 19, 2019 18:30
* improved code, now using gridEngine.addNode() instead of
removeAll() + addWidget()
which is much lighter now
* use a higher res layout when going up and cache is missing
(better to downsample say from 12 then up-sample from 1!)
@adumesny adumesny changed the title new compact() method new compact() method, improved setColumn() Dec 20, 2019
@adumesny adumesny merged commit 79ec5fe into gridstack:develop Dec 20, 2019
@adumesny adumesny changed the title new compact() method, improved setColumn() new compact() method, tweak setColumn() 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