Skip to content

Conversation

@JPerez00
Copy link

A new feature to the RoundedTool that allows users to make images perfectly circular, regardless of their original aspect ratio. This is implemented via a new "Make image a circle" checkbox toggle.

wide-img-circle-disabled
wide-img-circle-enabled

It’d be awesome to add X/Y controls to choose the crop spot, but that’s a bit out of my league. Also had to also update the border-radius-selector.tsx component to make it work.

pnpm run check

@vercel
Copy link

vercel bot commented Nov 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
quickpic ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2024 2:51am

@t3dotgg
Copy link
Owner

t3dotgg commented Nov 13, 2024

A bit confusing ux-wise with the "circle" option being a checkbox (while also leaving the border-radius selected). I don't have an obvious answer but this isn't it. Might even be better to have it as a separate tool 🤔

Copy link
Owner

@t3dotgg t3dotgg left a comment

Choose a reason for hiding this comment

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

Blocking until we find a better way to represent the states

@JPerez00
Copy link
Author

Thanks for the feedback, and sorry for the time wasted. Ok, so I replaced the circle checkbox with a "Shape" selector titled "Pick Your Shape" for dumb people like me lol.

pick.your.shape.mp4

Ran the checks, no issues.

pnpm run check - updates

I thought about making it a separate tool, but it felt like extra bulk (files and folders, etc). Let me know if this works for you.

@JPerez00 JPerez00 requested a review from t3dotgg November 16, 2024 01:59
@JPerez00 JPerez00 changed the title Added 'Make image a circle' toggle to RoundedTool & updated the border-radius-selector component. Added a "Pick Your Shape" selector to switch between rounded edges and a perfect circle. Nov 27, 2024
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