-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Context Menus demo crashes on negative width or height #4425
Comments
Also crashes in debug mode with Stack trace
|
I am experiencing a possibly related issue that I can reproduce 100% of the time running the demo app:
If the assertion is disabled the next one to be triggered is This does not happen on tag https://github.com/emilk/egui/releases/tag/0.27.2 If unrelated I will open a dedicated issue about this. Hope this provides additional context though, if related. Finished dev [unoptimized + debuginfo] target(s) in 0.14s
Running `target/debug/egui_demo_app`
thread 'main' panicked at crates/egui/src/layout.rs:600:9:
assertion failed: child_size.x >= 0.0 && child_size.y >= 0.0
stack backtrace:
0: rust_begin_unwind
at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/std/src/panicking.rs:645:5
1: core::panicking::panic_fmt
at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:72:14
2: core::panicking::panic
at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:144:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
Process finished with exit code 134 (interrupted by signal 6:SIGABRT) |
* Closes #4425 Fix: in Plot, Minimum values for screen protection.
Describe the bug
In the web demo, when the plot in the Context Menus demo has negative width or height, the app crashes due to invalid arguments for
f64::clamp
.To Reproduce
Steps to reproduce the behavior:
panicked at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/num/f64.rs:1496:9: min > max, or either was NaN. min = 440.00000059604645, max = -440.00000059604645
Expected behavior
The width or height should be set to 0.
Desktop:
The text was updated successfully, but these errors were encountered: