-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Remove unused CSS rule #89736
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
Remove unused CSS rule #89736
Conversation
@@ -773,8 +773,6 @@ h2.small-section-header > .anchor { | |||
|
|||
.item-table { | |||
display: table-row; |
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.
display: table-row; | |
display: table; |
I just realised this should be table
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.
Let's do it in another PR (either you or me).
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.
Will do!
@bors: r=dns2utf8 rollup |
📌 Commit 5611b43 has been approved by |
…askrgr Rollup of 6 pull requests Successful merges: - rust-lang#89579 (Add regression test for issue 80108) - rust-lang#89632 (Fix docblock code display on mobile) - rust-lang#89691 (Move `DebuggerCommands` and `check_debugger_output` to a separate module) - rust-lang#89707 (Apply clippy suggestions for std) - rust-lang#89722 (Fix spelling: Cannonical -> Canonical) - rust-lang#89736 (Remove unused CSS rule) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
As you can see in the firefox devtools:
It needs the display to be
grid
orinline-grid
, which isn't the case.r? @dns2utf8