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

feat: circle fit component #743

Closed
wants to merge 1 commit into from
Closed

feat: circle fit component #743

wants to merge 1 commit into from

Conversation

emccorson
Copy link
Collaborator

@emccorson emccorson commented Apr 15, 2024

A utility component for fitting text in a circle.

I'm not super happy with how complicated the code got, but it seems to work fine and responds to different border sizes.

Usage:

<div className="w-[500px] h-[500px] border-8 border-yellow rounded-[100%]">
  <CircleFit>
    Some text to fit in the circle
  </CircleFit>
</div>

2024-04-15-160828_703x719_scrot

Copy link

github-actions bot commented Apr 15, 2024

@github-actions github-actions bot temporarily deployed to pull request April 15, 2024 07:12 Inactive
@emccorson emccorson marked this pull request as draft April 15, 2024 12:27
@emccorson emccorson force-pushed the feat/circle-fit branch 2 times, most recently from 4e8da11 to 42f3336 Compare April 16, 2024 00:11
@github-actions github-actions bot temporarily deployed to pull request April 16, 2024 00:26 Inactive
@emccorson emccorson marked this pull request as ready for review April 16, 2024 23:27
@euharrison
Copy link
Contributor

@emccorson is this still relevant? Even if we close it, we can see the code in case it's useful for the future

@emccorson
Copy link
Collaborator Author

@euharrison this was mostly an experiment since the designs call for fitting things inside circles. I think it could be useful, but I can close it for now and leave it in the back of my mind if we need it.

@emccorson emccorson closed this Oct 24, 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