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

Fix egui_extras::Table scrolling bug #3690

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

abey79
Copy link
Collaborator

@abey79 abey79 commented Dec 7, 2023

egui_extras::Table now uses the clip rectangle to decide which rows to draw when using TableBody::rows().

@abey79 abey79 added egui_extras bug Something is broken labels Dec 7, 2023
@emilk emilk changed the title Use clip rectangle to compute rows to be drawn in egui_extras::Table Fix egui_extras::Table scrolling bug Dec 7, 2023
@emilk emilk merged commit d8a7955 into master Dec 8, 2023
36 of 37 checks passed
@emilk emilk deleted the antoine/fix-table-rows-with-clip-rect branch December 8, 2023 08:08
emilk pushed a commit that referenced this pull request Dec 8, 2023
`egui_extras::Table` now uses the clip rectangle to decide which rows to
draw when using `TableBody::rows()`.

- Closes #3682
- Closes #3670
emilk added a commit that referenced this pull request Dec 8, 2023
Includes:
* #3690

---------

Co-authored-by: Antoine Beyeler <49431240+abey79@users.noreply.github.com>
emilk added a commit to rerun-io/rerun that referenced this pull request Dec 10, 2023
### What
* Contains emilk/egui#3690

### 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):
  * Full build: [app.rerun.io](https://app.rerun.io/pr/4461/index.html)
* Partial build:
[app.rerun.io](https://app.rerun.io/pr/4461/index.html?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json)
- Useful for quick testing when changes do not affect examples in any
way
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4461)
- [Docs
preview](https://rerun.io/preview/bfe0b472532fc19fb3dc9da1ba544099ee7f24b6/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/bfe0b472532fc19fb3dc9da1ba544099ee7f24b6/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken egui_extras
Projects
None yet
2 participants