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

Polish grid layout logic #26

Merged
merged 4 commits into from
Sep 12, 2023
Merged

Polish grid layout logic #26

merged 4 commits into from
Sep 12, 2023

Conversation

emilk
Copy link
Member

@emilk emilk commented Sep 12, 2023

Was trying to chase down rerun-io/rerun#2576

I failed to reproduce, and fail to see how it could be possible. Perhaps we were running an older egui_tiles version then.

This PR:

  • Adds a test
  • Cleans up the code
  • Improves the logic slightly

@emilk emilk merged commit 199e5d9 into main Sep 12, 2023
@emilk emilk deleted the emilk/polish-grid-layout branch September 12, 2023 15:53
emilk added a commit to rerun-io/rerun that referenced this pull request Sep 13, 2023
### What
* Closes #2880
* Closes #2814
* Includes an update to rerun-io/egui_tiles#26

### 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/3295) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/3295)
- [Docs
preview](https://rerun.io/preview/bc86183c16f8c2c0e1097bcf0386995e6f425900/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/bc86183c16f8c2c0e1097bcf0386995e6f425900/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
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant