Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't show hover cards for primitive types #4871

Open
elliette opened this issue Dec 2, 2022 · 1 comment · May be fixed by #8528
Open

Don't show hover cards for primitive types #4871

elliette opened this issue Dec 2, 2022 · 1 comment · May be fixed by #8528
Assignees
@elliette
Labels
bug bash found during a bug bash cost: low Low engineering cost to implement. fix it friday P2 important to work on, but not at the top of the work list. screen: inspector

Comments

@elliette
Copy link
Member

elliette commented Dec 2, 2022

Hovering over field with a primitive type doesn't provide any extra information. Eg:

Screenshot 2022-12-02 at 10 24 01 AM

It might make sense to never show a hovercard for a primitive type value. We could also change the cursor style to indicate whether hovering will provide any extra info or not.
@elliette elliette added the bug bash found during a bug bash label Dec 2, 2022
@elliette
Copy link
Member Author

elliette commented Dec 2, 2022

This would also improve some of the performance issues we've seen inspecting Flutter Web apps, since less expression evaluations would need to be done. FYI @CoderDake @annagrin

@kenzieschmoll kenzieschmoll added screen: inspector P2 important to work on, but not at the top of the work list. fix it friday cost: low Low engineering cost to implement. labels Jul 31, 2024
This was referenced Nov 14, 2024
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elliette elliette

Labels
bug bash found during a bug bash cost: low Low engineering cost to implement. fix it friday P2 important to work on, but not at the top of the work list. screen: inspector
Projects
None yet
Milestone
No milestone
2 participants
@elliette @kenzieschmoll