-
Notifications
You must be signed in to change notification settings - Fork 403
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
Radio buttons for callstacks(implementation filters) look like they are grayed out and disabled #2088
Comments
Actually I have this PR #2063 that changes them a bit. Let's see if this is better :) Basically we try to stick with photon styling here. The aforementioned PR is fixing a small error when we tried to apply photon initially. |
I find this very surprising and would like to find out more. |
PR #2106 landed with the aforementioned changes. I think it's now much more difficult to think they're grayed out. |
Thanks for working on it Julien. I believe it was @loganfsmyth. hey @loganfsmyth, I remember us talking about this checkbox problem during the workshop. Would you mind taking a look at the new radio buttons and telling us if you think it's fixed or what we should do to make it more explicit? Currently the PR is merged but we haven't deployed yet. You can use https://deploy-preview-2106--perf-html.netlify.com/ to test it. for example, here's an example profile to see the new radio buttons: |
Thanks for the useful feedback! |
This came out during the profiling workshop with devtools team. It looks like our radio buttons are misleading and seem like they are grayed out and disabled. We should find a better styling for them to display.
┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: