Skip to content

dragging into a fixed row grid#1519

Merged
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop
Dec 6, 2020
Merged

dragging into a fixed row grid#1519
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop

Conversation

@adumesny

@adumesny adumesny commented Dec 6, 2020

Copy link
Copy Markdown
Member

Description

  • dragging into a fixed row grid works better
    (check if it will fit, else try to append, else won't insert)

fix #1419

**** possible BREAK (unlikely you will use engine directly)

  • engine constructor takes Options struct rather than spelling arguments (easier to extend/use)
  • canBePlacedWithRespectToHeight() -> willItFit() like grid method.

Checklist

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

* dragging into a fixed row grid works better
(check if it will fit, else try to append, else won't insert)
*
**** possible BREAK (not likely you will use engine directly)
* engine constructor takes Options struct rather than spelling arguments (easier to extend/use)
* canBePlacedWithRespectToHeight() -> willItFit() like grid method.
@adumesny adumesny merged commit aa2e453 into gridstack:develop Dec 6, 2020
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.

maxRow not preventing dragIn from pushing past max limit

1 participant