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

Have alternating background stripes also in expanded rows in the dataframe view #7412

Closed
abey79 opened this issue Sep 12, 2024 · 2 comments
Closed
Labels
🧑‍🎨 Design Requires UX/UI designer input feat-dataframe-view Everything related to the dataframe view ui concerns graphical user interface

Comments

@abey79
Copy link
Member

abey79 commented Sep 12, 2024

We should have background stripes everywhere. This means we must compute the odd-ness for rows (and sub-rows) on the fly, based on the expansion cache.

@abey79 abey79 added ui concerns graphical user interface feat-dataframe-view Everything related to the dataframe view labels Sep 12, 2024
@abey79 abey79 added this to the 0.19 - Dataframe and web video milestone Sep 12, 2024
@abey79 abey79 changed the title Have background stripes also in expanded rows in the dataframe view Have alternating background stripes also in expanded rows in the dataframe view Sep 12, 2024
abey79 added a commit that referenced this issue Sep 17, 2024
…7400)

### What

- Closes #7066

This PR introduces the ability to expend dataframe view rows to inspect
individual instances.

Note: expanding instance can potentially lead to very loooong tables.
This doesn't affect performance as we only query/draw what is visible on
screen, it means we reaching the limits of egui's `f32` numerical
precision. This can lead to uneven scrolling or even visual glitches if
the number of instance is >>1M.

Follow-up todo:
- #7412


https://github.com/user-attachments/assets/6801a8a8-667f-43b5-9581-bebcea80882b

### 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/7400?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/7400?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)!
* [x] If have noted any breaking changes to the log API in
`CHANGELOG.md` and the migration guide

- [PR Build Summary](https://build.rerun.io/pr/7400)
- [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`.
@emilk
Copy link
Member

emilk commented Sep 22, 2024

I'm not sure that expanding one row into multiple lines should change the background stripes of existing rows. That might just be confusing. Maybe it is better to sometimes have double-stripes, or stripe expanded rows differently

@emilk emilk added the 🧑‍🎨 Design Requires UX/UI designer input label Sep 22, 2024
@abey79
Copy link
Member Author

abey79 commented Sep 23, 2024

Actually, that has already landed with #7066 (so I'm going to close this issue).

I had the exact same thought when implementing that, but I noticed because everything shifts up and down when rows are expanded/collapsed, it's extremely difficult to notice the "stripe inversion" on subsequent rows. So we're pretty good as is imo.

@abey79 abey79 closed this as completed Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍🎨 Design Requires UX/UI designer input feat-dataframe-view Everything related to the dataframe view ui concerns graphical user interface
Projects
None yet
Development

No branches or pull requests

2 participants