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

Strange blueprint reordering after drag and drop #5081

Closed
nikolausWest opened this issue Feb 7, 2024 · 2 comments · Fixed by #5107
Closed

Strange blueprint reordering after drag and drop #5081

nikolausWest opened this issue Feb 7, 2024 · 2 comments · Fixed by #5107
Assignees
@abey79
Labels
🪳 bug Something isn't working ui concerns graphical user interface
Milestone
0.13

Comments

@nikolausWest
Copy link
Member

What: After dragging and dropping a space view in the blueprint, it seems to reorder the other items.

How to reproduce:
just rs-plot-dashboard --num-plots 12 --num-series-per-plot 5 --num-points-per-series 5000 --freq 1000 --connect

strange.reordering.mp4
@nikolausWest nikolausWest added 🪳 bug Something isn't working ui concerns graphical user interface labels Feb 7, 2024
@nikolausWest nikolausWest added this to the 0.13 milestone Feb 7, 2024
@abey79
Copy link
Member

abey79 commented Feb 7, 2024

Yet another manifestation of us using egui_tiles as an application layer data structure. I'll check if there is a quick fix.

@abey79 abey79 mentioned this issue Feb 7, 2024
Closed
@abey79
Copy link
Member

abey79 commented Feb 7, 2024

Root cause: rerun-io/egui_tiles#52

Note that this affects only grid containers.

@abey79 abey79 mentioned this issue Feb 7, 2024
Merged
emilk pushed a commit to rerun-io/egui_tiles that referenced this issue Feb 7, 2024
@abey79
Fix move_tile_to_container behavior for grid-to-same-grid moves wit…
e528aa0 
…h reflow enabled (#53)

- Fixes #52
- Required for rerun-io/rerun#5081
abey79 added a commit that referenced this issue Feb 7, 2024
@abey79
Update egui_tiles to 0.7.2
9760fd9 
- Fixes #5081
@abey79 abey79 mentioned this issue Feb 7, 2024
Merged
5 tasks
teh-cmc pushed a commit that referenced this issue Feb 7, 2024
@abey79
Update egui_tiles to 0.7.2 (#5107)
ff1bf9d 
### What

- Fixes #5081
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abey79 abey79

Labels
🪳 bug Something isn't working ui concerns graphical user interface
Projects
None yet
Milestone
0.13
Development

Successfully merging a pull request may close this issue.

Update egui_tiles to 0.7.2 rerun-io/rerun
2 participants
@nikolausWest @abey79