Skip to content

Conversation

ybiquitous
Copy link
Contributor

@ybiquitous ybiquitous commented May 26, 2023

  • Use a smaller font size for the toggle symbol. (Currently, it seems a little too large)
  • Use the child combinator (>) to unify selectors.
  • Use margin-left instead of whitespace within the content property.
  • Use :: instead of outdated : for the pseudo-element symbol. (See https://developer.mozilla.org/en-US/docs/Web/CSS/::before)

Examples with Chrome 113:

Before After
image image
image image

Ref: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/summary

Note: Please feel free to close this PR if you disagree (since it reflects my preference).

- Use a smaller font size for the toggle symbol. (Currently, it seems a little too large)
- Use the child combinator (`>`) to unify selectors.
- Use `margin-left` instead of whitespace within the `content` property.
- Use `::` instead of outdated `:` for the pseudo-element symbol.
  (See https://developer.mozilla.org/en-US/docs/Web/CSS/::before)
@ybiquitous ybiquitous force-pushed the improve-sidebar-css branch from 8da992a to ff7b19c Compare May 26, 2023 14:09
@ybiquitous ybiquitous marked this pull request as ready for review May 26, 2023 14:12
@nobu nobu merged commit 61ce0a7 into ruby:master May 26, 2023
@ybiquitous ybiquitous deleted the improve-sidebar-css branch May 26, 2023 22:49
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