egui/crates/egui_glow/src
Andreas Reich f9f5db9885
PaintCallbackInfo::viewport_in_pixels now guarantees to stay in bounds (#3604)
* Fixes https://github.com/rerun-io/rerun/issues/4297
* tested against a very hasty and incomplete port of egui/trunk, found
[here](https://github.com/rerun-io/rerun/tree/andreas/experimental-egui-trunk)


In rare cases in can happen that the viewport returned by
`PaintCallbackInfo` is outside the bounds of the screen.
for at least [wgpu/webgpu in
particular](https://www.w3.org/TR/webgpu/#dom-gpurenderpassencoder-setviewport)
this is invalid usage, other backends might be affected as well.
Since this happened due to a float rounding error (in one repro case I
had I got (width==1126.5625) + (offset=715.4376) = 1842.0001 for a
resolution of 1842) I decided to do away with fractional values on the
viewport alltogether. They _technically_ make sense since a viewport is
only specifying the NDC to pixel coordinate conversion, but practically
this may lead to surprising sub-sampling issues.

---------

Co-authored-by: Emil Ernerfeldt <emil.ernerfeldt@gmail.com>
2023-11-22 21:01:38 +01:00
..
shader Improve text redering and do all color operation in gamma space (#2071) 2022-09-24 17:53:11 +02:00
lib.rs Multiple viewports/windows (#3172) 2023-11-16 11:25:05 +01:00
misc_util.rs Update MSRV to Rust 1.72 (#3595) 2023-11-21 17:26:35 +01:00
painter.rs PaintCallbackInfo::viewport_in_pixels now guarantees to stay in bounds (#3604) 2023-11-22 21:01:38 +01:00
shader_version.rs Replace `tracing` with `log` (#2928) 2023-04-18 21:11:26 +02:00
vao.rs Update MSRV to Rust 1.72 (#3595) 2023-11-21 17:26:35 +01:00
winit.rs Introduce global `zoom_factor` (#3608) 2023-11-22 20:34:51 +01:00