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

Hard to navigate Visible History feature #5315

Closed
karimo87 opened this issue Feb 27, 2024 · 1 comment · Fixed by #6785
Closed

Hard to navigate Visible History feature #5315

karimo87 opened this issue Feb 27, 2024 · 1 comment · Fixed by #6785
Labels
😤 annoying Something in the UI / SDK is annoying to use 🏎️ Quick Issue Can be fixed in a few hours or less ui concerns graphical user interface

Comments

@karimo87
Copy link
Member

Describe the annoyance

I want to use the visible history feature. When I try to set up my override I am first struggling with the overlaid information blocking out the fields I am interested in. Once I can see the "current time with offset" it takes time for me to figure out that the set value is behind the cutoff on the right-hand site.

This might be my setup, but I have replicated this on the web with both Chrome and Safari.

To Reproduce
Steps to reproduce the behavior:

  1. Hover Visible History drop-down to see how overlay is blocking background info
  2. Select override and see where the offset value field ends up

Expected behavior
I'd expect to see the field for setting the offset value when choosing that option.

I'm not sure what would be better behaviour for the overlay.

Screenshots

Visibility.of.visible.history.mp4
@karimo87 karimo87 added 😤 annoying Something in the UI / SDK is annoying to use 👀 needs triage This issue needs to be triaged by the Rerun team labels Feb 27, 2024
@Wumpf Wumpf added ui concerns graphical user interface 🏎️ Quick Issue Can be fixed in a few hours or less and removed 👀 needs triage This issue needs to be triaged by the Rerun team labels Apr 23, 2024
@emilk
Copy link
Member

emilk commented Apr 25, 2024

increasing the default tooltip delay seems like a good start

@emilk emilk mentioned this issue Jul 5, 2024
Merged
5 tasks
@emilk emilk closed this as completed in 3c8461f Jul 5, 2024
emilk added a commit that referenced this issue Jul 5, 2024
@emilk
Update to egui 0.28.1 (#6785)
a916f9d 
## What
* Closes #5315

## egui changelog

### ⭐ Added
* Add `Image::uri()` [#4720](emilk/egui#4720) by
[@rustbasic](https://github.com/rustbasic)

### 🔧 Changed
* Better documentation for `Event::Zoom`
[#4778](emilk/egui#4778) by
[@emilk](https://github.com/emilk)
* Hide tooltips when scrolling
[#4784](emilk/egui#4784) by
[@emilk](https://github.com/emilk)
* Smoother animations [#4787](emilk/egui#4787)
by [@emilk](https://github.com/emilk)
* Hide tooltip on click [#4789](emilk/egui#4789)
by [@emilk](https://github.com/emilk)

### 🐛 Fixed
* Fix default height of top/bottom panels
[#4779](emilk/egui#4779) by
[@emilk](https://github.com/emilk)
* Show the innermost debug rectangle when pressing all modifier keys
[#4782](emilk/egui#4782) by
[@emilk](https://github.com/emilk)
* Fix occasional flickering of pointer-tooltips
[#4788](emilk/egui#4788) by
[@emilk](https://github.com/emilk)

## eframe changelog
* Web: only capture clicks/touches when actually over canvas
[#4775](emilk/egui#4775) by
[@lucasmerlin](https://github.com/lucasmerlin)

## 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 the web demo (if applicable):
* Using examples from latest `main` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/6785?manifest_url=https://app.rerun.io/version/main/examples_manifest.json)
* Using full set of examples from `nightly` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/6785?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG
* [x] If applicable, add a new check to the [release
checklist](https://github.com/rerun-io/rerun/blob/main/tests/python/release_checklist)!

- [PR Build Summary](https://build.rerun.io/pr/6785)
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)

To run all checks from `main`, comment on the PR with `@rerun-bot
full-check`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
😤 annoying Something in the UI / SDK is annoying to use 🏎️ Quick Issue Can be fixed in a few hours or less ui concerns graphical user interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update to egui 0.28.1 rerun-io/rerun
3 participants
@emilk @Wumpf @karimo87