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

style: Add rounded borders on original comment box #92

Merged
merged 3 commits into from
Mar 12, 2023

Conversation

RebootedDuck
Copy link
Contributor

@RebootedDuck RebootedDuck commented Mar 11, 2023

This adds "border-radius: 20px;" to the original comment window on new/old Reddit, this should improve view-ability and make the visual styling slightly more appealing

Sorry if this was an already considered suggestion that was rejected as I couldn't find anything

Copy link
Owner

@DenverCoder1 DenverCoder1 left a comment

Choose a reason for hiding this comment

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

Looks good, but I think maybe less rounded would be better. In my opinion, it should ideally match the roundness of the other elements on the page (4px on redesign, 7px on old reddit)

Feel free to use the "Commit suggestion" buttons if you agree and let me know what you think.

script.js Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
@RebootedDuck
Copy link
Contributor Author

RebootedDuck commented Mar 12, 2023

Oh great point, I didn't think to check what rounding other elements were using, consistent rounding definitely looks much better

Copy link
Owner

@DenverCoder1 DenverCoder1 left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for the contribution 🎉

@RebootedDuck
Copy link
Contributor Author

Cheers!

@DenverCoder1 DenverCoder1 changed the title feat: rounded borders on original comment box style: Add rounded borders on original comment box Mar 12, 2023
@DenverCoder1 DenverCoder1 merged commit 05105b6 into DenverCoder1:master Mar 12, 2023
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