-
Notifications
You must be signed in to change notification settings - Fork 89
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 subtle striping for table in <wpt-results/>
#4089
Conversation
@YoshiRulz Hi James! First of all, thank you for the fix! One minor follow-up: the test names are not vertically centered around the cells. Could you center them? |
I had a look and it seems non-trivial ( |
You are right! I couldn't get it to move after fiddling with the margin and padding in Another thing is that on the homepage, it uses a different shade of grey, #EEEEEE I would like to make it consistent! Thanks again! |
@YoshiRulz Thank you for your patience! Merged your CL and thanks again for the help :) |
Description
Resolves #1783/#2217 by adding a grey background to every other row in the table.
Before/after:
Untested because I'm on NixOS and Docker isn't trivial, though based on #3840 I'm guessing it wouldn't have worked anyway. The above screenshot was taken by adding the rule via the dev tools.
Review Information
idk man
Changes
Added a single CSS rule. edit: two now
Requirements
N/A