Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NEO-1075: Shapes have a minimum size enforced on creation / loading #654

Merged
merged 4 commits into from
Jan 6, 2025

Conversation

jaller94
Copy link
Contributor

@jaller94 jaller94 commented Nov 28, 2024

We already enforce this minimum size when resizing. Now, it's also enforced while.

I couldn't find the code that enforces the minimum size when reszing. Would be great to use the same code / values / constant here.

✔️ Checklist

  • A changeset describing the change and affected packages (more info).
  • Added or updated documentation.
  • Tests for new functionality and regression tests for bug fixes.
  • Screenshots or videos attached (for UI changes).
  • All your commits have a Signed-off-by line in the message (more info).

@jaller94 jaller94 requested a review from a team November 28, 2024 13:11
Copy link

changeset-bot bot commented Nov 28, 2024

🦋 Changeset detected

Latest commit: d6ae3eb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@nordeck/matrix-neoboard-widget Patch
@nordeck/matrix-neoboard-react-sdk Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mgcm mgcm force-pushed the nic/bugfix/NEO-1075 branch from fc373bb to ebf9a9f Compare January 2, 2025 11:42
@mgcm mgcm changed the title NEO-1075: Prevent a shape from being smaller than 2x2 on creation NEO-1075: Shapes have a minimum size enforced on creation / loading Jan 2, 2025
@mgcm mgcm requested a review from weeman1337 January 4, 2025 17:30
@mgcm mgcm force-pushed the nic/bugfix/NEO-1075 branch from d0bd5ba to d6ae3eb Compare January 6, 2025 15:49
@mgcm mgcm dismissed weeman1337’s stale review January 6, 2025 15:51

tests were added

@mgcm mgcm merged commit cd8480b into main Jan 6, 2025
5 checks passed
@mgcm mgcm deleted the nic/bugfix/NEO-1075 branch January 6, 2025 22:03
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.

4 participants