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

✨ ENHANCE: Soften edges in admonitions and remove some ink #352

Merged
merged 3 commits into from
Jul 7, 2021

Conversation

choldgraf
Copy link
Member

@choldgraf choldgraf commented Jul 3, 2021

This is an experiment at implementing #351

It rounds the corners of our admonitions and uses a grey border rather than a drop shadow. It also makes the "previous/next" buttons use bolded text without a border.

Tasks

  • Get some feedback on these changes
  • Convert into "non-draft PR" for final review

@choldgraf choldgraf changed the title [EXPERIMENT] Update CSS for softer edges [EXPERIMENT] Softer edges in admonitions and removing some ink Jul 3, 2021
@cooperrc
Copy link

cooperrc commented Jul 4, 2021

I like the rounded edges, but miss the left hand side bolder color.
Errors/warnings are great in the squared, bold style, but the muting is much better for notes and other nice-to-know sections.

@choldgraf
Copy link
Member Author

The latest push takes @pradyunsg and @cooperrc 's advice and keeps the left border, but rounds the edges and controls the box shadow a bit:

image

@choldgraf choldgraf changed the title [EXPERIMENT] Softer edges in admonitions and removing some ink ✨ ENHANCE: Softer edges in admonitions and removing some ink Jul 4, 2021
@choldgraf choldgraf marked this pull request as ready for review July 4, 2021 18:18
@choldgraf choldgraf changed the title ✨ ENHANCE: Softer edges in admonitions and removing some ink ✨ ENHANCE: Soften edges in admonitions and remove some ink Jul 7, 2021
@choldgraf choldgraf merged commit cdad8b2 into master Jul 7, 2021
@choldgraf choldgraf deleted the soft-edges branch July 7, 2021 17:30
@choldgraf
Copy link
Member Author

thanks for the feedback all - this is a much more subtle but I think nicer addition :-)

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