-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
initial-letter: clarify usage and see also #36581
Conversation
Preview URLs External URLs (1)URL:
(comment last updated: 2024-10-30 22:17:16) |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@estelle all looks good to me! My only suggestion would be that maybe you could add some vertical margin between each line of your new example, so it is a little easier to see what is going on. They are currently cramped close together.
i intentionally didn't, to show the issue, but maybe i should add an example with multiple lines of texts to make that clearer. will do it in a different PR. |
make it clear in the intro which elements apply.
add selectors for those elements to the see also