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.

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

Hover previews can easily become too cluttered #3573

Closed
nikolausWest opened this issue Oct 1, 2023 · 3 comments · Fixed by #3579
Closed

Hover previews can easily become too cluttered #3573

nikolausWest opened this issue Oct 1, 2023 · 3 comments · Fixed by #3579
Assignees
Labels
😤 annoying Something in the UI / SDK is annoying to use ui concerns graphical user interface

Comments

@nikolausWest
Copy link
Member

nikolausWest commented Oct 1, 2023

Hovering over data in a space view can easily become quite cluttered (see screenshot and video).
Screenshot 2023-10-01 at 21 45 48

too_much_hover_data.mp4

We should therefore

  • Hide indicator components from hover ui #3548 Hide indicator components as they aren't really the data you are interested in inspecting in a view.
  • Not show PinholeProjection, Resolution, or AnnotationContext when hovering an image
    • Is there anything else?
  • Make sure the hover preview doesn't cover the part of the view where you're currently hovering.
    • If fixing this isn't achievable quickly, that should be broken out to a separate issue and handled in a later cycle
@nikolausWest nikolausWest added ui concerns graphical user interface 😤 annoying Something in the UI / SDK is annoying to use labels Oct 1, 2023
@nikolausWest nikolausWest changed the title Don't show indicators on hover Show less data on hover Oct 1, 2023
@nikolausWest
Copy link
Member Author

I also wonder about "hover position" for 3D points when hover in 2D. I think we should remove it but I may be wrong. I definitely don't understand it.

@nikolausWest nikolausWest changed the title Show less data on hover Hover previews can easily become too cluttered Oct 1, 2023
@Wumpf
Copy link
Member

Wumpf commented Oct 2, 2023

Afaik showing the position was a user request:

Also not happy with what it looks like though in the ui. There's some more notes on the topic here

@emilk emilk assigned emilk and unassigned abey79 Oct 2, 2023
@nikolausWest
Copy link
Member Author

Also not happy with what it looks like though in the ui. There's some more notes on the topic here

In my opinion the current implementation makes things worse more than better. It's not worth the added clutter when it's hard to understand

emilk added a commit that referenced this issue Oct 2, 2023
### What
Fit it on just two lines, and show full 3x3 matrix on hover:


![image](https://github.com/rerun-io/rerun/assets/1148717/ed1b009f-9149-4617-a2f3-ec069d3f12d5)

* Closes #3573

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/3579) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/3579)
- [Docs
preview](https://rerun.io/preview/217ca4a77e15709afaf049cb71e4dbb9eef5dca3/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/217ca4a77e15709afaf049cb71e4dbb9eef5dca3/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://ref.rerun.io/dev/bench/)
- [Wasm size tracking](https://ref.rerun.io/dev/sizes/)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😤 annoying Something in the UI / SDK is annoying to use ui concerns graphical user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants