-
Notifications
You must be signed in to change notification settings - Fork 758
Improve UI implementation of EventListeners list #7879
Comments
hey, can i take this bug? |
Works for me. :) Some things to note:
|
/claim |
Thanks for claiming the issue! 👋 Here are some links for getting setup, contributing, and developing. We're always happy to answer questions in slack! If you become busy, feel free to 🦊 Debugger team! |
Hey, @fvsch , about this:
I think the row height not being 20px is because of the inconsistent checkbox size on different platforms. Couldn't we just set a fixed size for the checkbox on all platforms? |
Nope. Changing the checkbox size doesn't work on all platforms. It seems to work alright on macOS, not sure on Windows, and doesn't work on Linux. Making rows 20px should be possible nonetheless. For example, using flexbox, you could have a
|
Hmm ok i made that change |
The UI spec: https://mozilla.invisionapp.com/share/H3PYO9EMC9D#/screens/340771972_Event_Breakpoints_1
A few issues:
I'll probably take this bug. :)
The text was updated successfully, but these errors were encountered: