-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add CSS for tables #13822
Add CSS for tables #13822
Conversation
.superclass-hierarchy .superclass + .superclass::before { | ||
.superclass-hierarchy .superclass+.superclass::before { |
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.
Personally, I find (all occurrences of) this change less readable.
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.
I can revert it - it was just the default formatting performed by VS Code.
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.
I'll leave it to the core team to decide. The rest of the changes are also quite subjective btw, as they depend heavily on your prettier configuration.
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.
I'm not sure I follow, other than the formatting this just adds table css and allowing splits between namespaces
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.
I meant the formatting-related ones.
I'd prefer three separate PRs for these changes. They are pretty much unrelated to each other than that some of the same files are touched. |
Add CSS for tables that matches code blocks (#11899)