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: Add UUID Generator Utility #77

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

chrbu92
Copy link
Contributor

@chrbu92 chrbu92 commented Oct 3, 2024

Adds a UUID generator to the array of available utilities. Simply generates a random UUID and copies to clipboard.

No tests added.

@chrbu92 chrbu92 changed the title Add UUID Generator Utility Feat: Add UUID Generator Utility Oct 3, 2024
@chrbu92
Copy link
Contributor Author

chrbu92 commented Nov 15, 2024

@peckz would it be possible to get a review on this PR?

@peckz
Copy link
Collaborator

peckz commented Nov 22, 2024

@peckz would it be possible to get a review on this PR?

I might have a look over the weekend! Thank you!

@peckz
Copy link
Collaborator

peckz commented Nov 25, 2024

@chrbu92 Thanks for contributing.
Small advice: for layout development, prefer flex over static widths and floats. This approach is more future proof and industry standard nowadays

- make sure that each button has single responsibility
- adapt current look to be more like other tools
@peckz peckz force-pushed the feature/uuid-generator branch from 932e3c4 to 8dd2833 Compare November 25, 2024 08:32
@peckz peckz merged commit a5e1bf6 into jamdotdev:main Nov 25, 2024
1 check 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.

2 participants