Skip to content

Conversation

SSharma1103
Copy link
Contributor

… selection/fit/thumbnail; add undo/redo for width

… selection/fit/thumbnail; add undo/redo for width
Copy link

vercel bot commented Oct 12, 2025

@SSharma1103 is attempting to deploy a commit to the dottle's projects Team on Vercel.

A member of the Team first needs to authorize it.

@SSharma1103
Copy link
Contributor Author

saw an feature issue #605 (its my first contribution )

Copy link

vercel bot commented Oct 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
drawdb Ready Ready Preview Comment Oct 14, 2025 3:35pm

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey, this is great

can u add resizing from both sides?

also can u add an indicator like a circle in the middle of the side edges for some visibility?

thank you

… width with clamp; respects locked/read-only; undo/redo for x+width
@SSharma1103
Copy link
Contributor Author

SSharma1103 commented Oct 12, 2025

can u add resizing from both sides?-->done

also can u add an indicator like a circle in the middle of the side edges for some visibility?-->done (last one)

thank you --> thank you

@SSharma1103
Copy link
Contributor Author

is it fine ?

thank you :)

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please have the resize indicators be like the ones on subject areas (round with outline)

also have them visible only on hover not when selected

@SSharma1103
Copy link
Contributor Author

how does this look ?

thank you

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove the comments and it should be fine

@SSharma1103 SSharma1103 requested a review from 1ilit October 12, 2025 20:00
@SSharma1103
Copy link
Contributor Author

done

thank you

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@1ilit 1ilit linked an issue Oct 14, 2025 that may be closed by this pull request
@1ilit 1ilit merged commit ce97048 into drawdb-io:main Oct 14, 2025
4 checks passed
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.

[FEATURE] resizable note

2 participants