-
Notifications
You must be signed in to change notification settings - Fork 9.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
DevTools report redesign #9019
Labels
Comments
Oh, you'd have to toggle the pill meant for metric descriptions to see the error.. |
This was referenced May 23, 2019
Closed
fixed! (modulo other open report rendering issues) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I collected some screenshots of what master looks like now in DevTools. Not pretty (yet) :)
This is what 4.2 (in stable) looks like now (yes, DevTools LH had dark mode already):

Dark mode, master (I zoomed out a lot here):

Light mode, master (same zoom settings as 4.2 img):


--
Dark mode is messed up b/c it was implemented (in DT) with a theme - a bunch of custom CSS overwriting the pre 5.0 stuff. Figure we just need to remove most (or all) of what that theme is doing.
Chromium changes: https://chromium-review.googlesource.com/c/chromium/src/+/1623894
The text was updated successfully, but these errors were encountered: