Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) #4211

rustbasic · 2024-03-22T13:09:41Z

Issues :

Pull requests :

ColinKinloch · 2024-03-22T13:38:26Z

This doesn't fix #4196

The issue is that on wayland when programmatically resizing a window via request_inner_size winit (intentionally) won't receive a Resized event.

Which means when running under wayland it's appropriate to resize the viewport directly after resizing the window.

ColinKinloch · 2024-03-22T13:39:56Z

Hense why in #4207 I check the size before and after each ViewportCommand is processed and resize the viewport if its changed.

rustbasic · 2024-03-22T13:49:20Z

Hense why in #4207 I check the size before and after each ViewportCommand is processed and resize the viewport if its changed.

Thank you for letting me know the results.

rustbasic · 2024-03-23T02:59:20Z

@ColinKinloch

I changed this a little more.
Could you try this one more time and let me know the results?

Fix ViewportCommand::InnerSize not resizing viewport on Wayland (#4211) #4211

ColinKinloch · 2024-03-23T10:53:14Z

It works!
Thanks for working on this.

I think however that logging "InnerSize ignered by winit" should be something else.
The return values of request_inner_size are:

None: New size will be in a Resized event
Requested size: The platform has synchronously resized the window
Unchanged size: The platform has denied the request
Different size: The platform has chosen a different size (maybe doesn't allow for windows bigger than the monitor)

Maybe only log when the value isn't None or the requested size.

rustbasic · 2024-03-23T17:24:05Z

Could you try this one more time and let me know the results?

@ColinKinloch

Could you try one more time to see if the size is saved properly and let me know the results?

rustbasic · 2024-03-30T07:19:51Z

I'm completed.

calling process_viewport_commands(), Unified with handle_viewport_output().

emilk · 2024-03-30T18:32:22Z

I'll finish this up

)

@dataphract

* Uses: rerun-io/egui_tiles#67 * Uses: lampsitter/egui_commonmark#51 * Split off from #6171 * Closes #5280 ### Test * [x] image loading * [x] gltf ### 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): * Using examples from latest `main` build: [rerun.io/viewer](https://rerun.io/viewer/pr/6448?manifest_url=https://app.rerun.io/version/main/examples_manifest.json) * Using full set of examples from `nightly` build: [rerun.io/viewer](https://rerun.io/viewer/pr/6448?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json) * [x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG * [x] If applicable, add a new check to the [release checklist](https://github.com/rerun-io/rerun/blob/main/tests/python/release_checklist)! - [PR Build Summary](https://build.rerun.io/pr/6448) - [Recent benchmark results](https://build.rerun.io/graphs/crates.html) - [Wasm size tracking](https://build.rerun.io/graphs/sizes.html) To run all checks from `main`, comment on the PR with `@rerun-bot full-check`. # egui changelog so far Full diff at https://github.com/emilk/egui/compare/0.27.0..HEAD #### ecolor * Fix `hex_color!` macro by re-exporting `color_hex` crate from `ecolor` [#4372](emilk/egui#4372) (thanks [@dataphract](https://github.com/dataphract)!) * Remove `extra_asserts` and `extra_debug_asserts` feature flags [#4478](emilk/egui#4478) #### eframe * Add `register_native_texture` in `eframe::Frame` [#4246](emilk/egui#4246) (thanks [@Chaojimengnan](https://github.com/Chaojimengnan)!) * Early-out from context switching the `glow` backend [#4284](emilk/egui#4284) * Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) [#4211](emilk/egui#4211) (thanks [@rustbasic](https://github.com/rustbasic)!) * Only avoid glow context switching on Windows [#4296](emilk/egui#4296) * Improve IME support with new `Event::Ime` [#4358](emilk/egui#4358) (thanks [@rustbasic](https://github.com/rustbasic)!) * Allow users to create viewports larger than monitor on Windows & macOS [#4337](emilk/egui#4337) (thanks [@lopo12123](https://github.com/lopo12123)!) * Use `objc2` and its framework crates [#4395](emilk/egui#4395) (thanks [@madsmtm](https://github.com/madsmtm)!) * Update to Rust 1.76 [#4411](emilk/egui#4411) * Egui-winit: emit physical key presses when a non-Latin layout is active [#4461](emilk/egui#4461) (thanks [@TicClick](https://github.com/TicClick)!) * IME for chinese [#4436](emilk/egui#4436) (thanks [@rustbasic](https://github.com/rustbasic)!) * Fix : In Windows, the 'egui_demo_app' screen does not appear. [#4410](emilk/egui#4410) (thanks [@rustbasic](https://github.com/rustbasic)!) * Fix: Window position creeps between executions on scaled monitors [#4443](emilk/egui#4443) (thanks [@avery-radmacher](https://github.com/avery-radmacher)!) * Update `image` crate to 0.25 [#4160](emilk/egui#4160) * Ignore synthetic key presses [#4514](emilk/egui#4514) (thanks [@hut](https://github.com/hut)!) * Fix: still track mouse when dragging outside web canvas [#4522](emilk/egui#4522) * Add `NativeOptions::persistence_path` [#4423](emilk/egui#4423) (thanks [@lucasmerlin](https://github.com/lucasmerlin)!) * Use ResizeObserver instead of `resize` event [#4536](emilk/egui#4536) (thanks [@jprochazk](https://github.com/jprochazk)!) * Fix: Don't `.forget()` RAF closure [#4551](emilk/egui#4551) (thanks [@jprochazk](https://github.com/jprochazk)!) #### egui_extras * Update `image` crate to 0.25 [#4160](emilk/egui#4160) #### egui_plot * `Plot::Items:allow_hover` give possibility to masked the interaction on hovered item [#2558](emilk/egui#2558) (thanks [@haricot](https://github.com/haricot)!) * Expose `ClosestElem` and `PlotConfig` [#4380](emilk/egui#4380) (thanks [@Narcha](https://github.com/Narcha)!) * Disable interaction for `ScrollArea` and `Plot` when UI is disabled [#4457](emilk/egui#4457) (thanks [@varphone](https://github.com/varphone)!) * Make sure plot size is positive [#4429](emilk/egui#4429) (thanks [@rustbasic](https://github.com/rustbasic)!) * Introduce lifetime to `egui_plot::Plot` to replace `'static` fields [#4435](emilk/egui#4435) (thanks [@Fabus1184](https://github.com/Fabus1184)!) * Hide all other series when alt-clicking in the legend [#4549](emilk/egui#4549) (thanks [@abey79](https://github.com/abey79)!) * Plot now respects the `interact_radius` set in the UI's style [#4520](emilk/egui#4520) (thanks [@YgorSouza](https://github.com/YgorSouza)!) #### egui_glow * Enable egui_glow's winit feature on wasm (#4420) [#4421](emilk/egui#4421) (thanks [@simon-frankau](https://github.com/simon-frankau)!) #### egui-wgpu * Update to wgpu 0.20 [#4433](emilk/egui#4433) (thanks [@KeKsBoTer](https://github.com/KeKsBoTer)!) * Revert update to wgpu 0.20 => downgrade to wgpu 0.19.1 [#4559](emilk/egui#4559) #### egui-winit * Update `webbrowser` to `v1.0.0` [#4394](emilk/egui#4394) (thanks [@torokati44](https://github.com/torokati44)!) * Emit physical key presses when a non-Latin layout is active [#4461](emilk/egui#4461) (thanks [@TicClick](https://github.com/TicClick)!) * IME for chinese [#4436](emilk/egui#4436) (thanks [@rustbasic](https://github.com/rustbasic)!) * Fix: Window position creeps between executions on scaled monitors [#4443](emilk/egui#4443) (thanks [@avery-radmacher](https://github.com/avery-radmacher)!) * Ignore synthetic key presses [#4514](emilk/egui#4514) (thanks [@hut](https://github.com/hut)!) #### egui * Improve the UI for changing the egui theme [#4257](emilk/egui#4257) * Change the resize cursor when you reach the resize limit [#4275](emilk/egui#4275) * Make `TextEdit` an atomic widget [#4276](emilk/egui#4276) * Overload operators for `Rect + Margin`, `Rect - Margin` etc [#4277](emilk/egui#4277) * Implement blinking text cursor in `TextEdit` [#4279](emilk/egui#4279) * Rename `fn scroll2` to `fn scroll` [#4282](emilk/egui#4282) * Change `Frame::multiply_with_opacity` to multiply in gamma space [#4283](emilk/egui#4283) * Support order on windows [#4301](emilk/egui#4301) (thanks [@alexparlett](https://github.com/alexparlett)!) * Fix wrong replacement function in deprecation notice of `drag_released*` [#4314](emilk/egui#4314) (thanks [@sornas](https://github.com/sornas)!) * Consider layer transform when positioning text agent [#4319](emilk/egui#4319) (thanks [@juancampa](https://github.com/juancampa)!) * Fix incorrect line breaks [#4377](emilk/egui#4377) (thanks [@juancampa](https://github.com/juancampa)!) * Fix `hex_color!` macro by re-exporting `color_hex` crate from `ecolor` [#4372](emilk/egui#4372) (thanks [@dataphract](https://github.com/dataphract)!) * Change `Ui::allocate_painter` to inherit properties from `Ui` [#4343](emilk/egui#4343) (thanks [@varphone](https://github.com/varphone)!) * Use parent `Ui`s style for popups [#4325](emilk/egui#4325) (thanks [@alexparlett](https://github.com/alexparlett)!) * Fix : take `rounding` into account when using `Slider::trailing_fill` [#4308](emilk/egui#4308) (thanks [@rustbasic](https://github.com/rustbasic)!) * Add a way to specify Undoer settings and construct Undoers more easily [#4357](emilk/egui#4357) (thanks [@valadaptive](https://github.com/valadaptive)!) * Add xtask crate [#4293](emilk/egui#4293) (thanks [@YgorSouza](https://github.com/YgorSouza)!) * Add `ViewportCommand::RequestCut`, `RequestCopy` and `RequestPaste` to trigger Clipboard actions [#4035](emilk/egui#4035) (thanks [@bu5hm4nn](https://github.com/bu5hm4nn)!) * Fix `Panel` incorrect size [#4351](emilk/egui#4351) (thanks [@zhatuokun](https://github.com/zhatuokun)!) * Improve IME support with new `Event::Ime` [#4358](emilk/egui#4358) (thanks [@rustbasic](https://github.com/rustbasic)!) * Allow users to create viewports larger than monitor on Windows & macOS [#4337](emilk/egui#4337) (thanks [@lopo12123](https://github.com/lopo12123)!) * Added ability to define colors at UV coordinates along a path [#4353](emilk/egui#4353) (thanks [@murl-digital](https://github.com/murl-digital)!) * Eframe: update ViewportBuilder.with_icon() documentation [#4408](emilk/egui#4408) (thanks [@roccoblues](https://github.com/roccoblues)!) * Update to Rust 1.76 [#4411](emilk/egui#4411) * Add a `Display` impl for `Vec2`, `Pos2`, and `Rect` [#4428](emilk/egui#4428) (thanks [@tgross35](https://github.com/tgross35)!) * Remove `extra_asserts` and `extra_debug_asserts` feature flags [#4478](emilk/egui#4478) * Egui-winit: emit physical key presses when a non-Latin layout is active [#4461](emilk/egui#4461) (thanks [@TicClick](https://github.com/TicClick)!) * Disable interaction for `ScrollArea` and `Plot` when UI is disabled [#4457](emilk/egui#4457) (thanks [@varphone](https://github.com/varphone)!) * Update ahash 0.8.6 -> 0.8.11 [#4507](emilk/egui#4507) (thanks [@hellodword](https://github.com/hellodword)!) * `include_image!` now accepts expressions [#4521](emilk/egui#4521) (thanks [@YgorSouza](https://github.com/YgorSouza)!) * Remove `Event::Scroll` and handle it in egui [#4524](emilk/egui#4524) * Remove scroll latency for smooth trackpads [#4526](emilk/egui#4526) * Smooth out zooming with discreet scroll wheel [#4530](emilk/egui#4530) * Add `Options::line_scroll_speed` and `scroll_zoom_speed` [#4532](emilk/egui#4532) * Don't panic when replacement glyph is not found [#4542](emilk/egui#4542) (thanks [@RyanBluth](https://github.com/RyanBluth)!) * Make `TextEdit::return_key` optional [#4543](emilk/egui#4543) (thanks [@doonv](https://github.com/doonv)!) * Add `TextEdit::hint_text_font` [#4517](emilk/egui#4517) (thanks [@zaaarf](https://github.com/zaaarf)!) * Add `Options::reduce_texture_memory` to free up RAM [#4431](emilk/egui#4431) (thanks [@varphone](https://github.com/varphone)!) * Fix `Ui::scroll_with_delta` only scrolling if the `ScrollArea` is focused [#4303](emilk/egui#4303) (thanks [@lucasmerlin](https://github.com/lucasmerlin)!) * Add support for text truncation to `egui::Style` [#4556](emilk/egui#4556) (thanks [@abey79](https://github.com/abey79)!) * Hide toolip when opening `ComboBox` drop-down [#4546](emilk/egui#4546) (thanks [@abey79](https://github.com/abey79)!) * Better spacing and sizes for (menu) buttons [#4558](emilk/egui#4558) #### epaint * Add `RectShape::blur_width` to implement shadows [#4267](emilk/egui#4267) * Overload operators for `Rect + Margin`, `Rect - Margin` etc [#4277](emilk/egui#4277) * Fix incorrect line breaks [#4377](emilk/egui#4377) (thanks [@juancampa](https://github.com/juancampa)!) * Fix `hex_color!` macro by re-exporting `color_hex` crate from `ecolor` [#4372](emilk/egui#4372) (thanks [@dataphract](https://github.com/dataphract)!) * Add `emath::OrderedFloat` (moved from `epaint::util::OrderedFloat`) [#4389](emilk/egui#4389) * Added ability to define colors at UV coordinates along a path [#4353](emilk/egui#4353) (thanks [@murl-digital](https://github.com/murl-digital)!) * Add a `Display` impl for `Vec2`, `Pos2`, and `Rect` [#4428](emilk/egui#4428) (thanks [@tgross35](https://github.com/tgross35)!) * Remove `extra_asserts` and `extra_debug_asserts` feature flags [#4478](emilk/egui#4478) * Make `epaint::mutex::RwLock` allow `?Sized` types [#4485](emilk/egui#4485) (thanks [@crumblingstatue](https://github.com/crumblingstatue)!) * Update ahash 0.8.6 -> 0.8.11 [#4507](emilk/egui#4507) (thanks [@hellodword](https://github.com/hellodword)!) * Don't panic when replacement glyph is not found [#4542](emilk/egui#4542) (thanks [@RyanBluth](https://github.com/RyanBluth)!) --------- Co-authored-by: Antoine Beyeler <antoine@rerun.io> Co-authored-by: Andreas Reich <r_andreas2@web.de>

Update lib.rs

1c62cbe

rustbasic mentioned this pull request Mar 22, 2024

Fix InnerSize command not resizing viewport #4207

Closed

rustbasic added 2 commits March 22, 2024 22:22

Update lib.rs

53bfe46

Merge branch 'emilk:master' into patch28

bf8c031

rustbasic changed the title ~~Fix InnerSize command not resizing viewport~~ Fix : i think this is the part that needs to be corrected. Mar 22, 2024

rustbasic mentioned this pull request Mar 23, 2024

(rustbasic) In Progress : Testing... #4182

Closed

rustbasic added 3 commits March 23, 2024 11:48

Update lib.rs

418e287

Update glow_integration.rs

1b0ad1f

Update wgpu_integration.rs

09eae9e

rustbasic changed the title ~~Fix : i think this is the part that needs to be corrected.~~ Fix InnerSize command not resizing viewport Mar 23, 2024

rustbasic added 4 commits March 23, 2024 12:01

Update glow_integration.rs

47337b5

Update wgpu_integration.rs

431b244

Update glow_integration.rs

ac1951c

Update wgpu_integration.rs

302cb1b

rustbasic added 8 commits March 24, 2024 00:51

Update lib.rs

6faa4ac

Update wgpu_integration.rs

f326abf

Update glow_integration.rs

1eabdff

Update wgpu_integration.rs

2f25d18

Update glow_integration.rs

19bd398

Update lib.rs

9126521

Update wgpu_integration.rs

c868f48

Update glow_integration.rs

1daa19f

rustbasic added 2 commits March 24, 2024 02:28

Update glow_integration.rs

2256cc2

Update wgpu_integration.rs

583199f

rustbasic added 8 commits March 30, 2024 15:36

Update wgpu_integration.rs

41cb51f

Update glow_integration.rs

b79d4a2

Update glow_integration.rs

2ff20c6

Update wgpu_integration.rs

003c73a

Update glow_integration.rs

1d8baa6

Update wgpu_integration.rs

e63616d

Update glow_integration.rs

13deb07

Update wgpu_integration.rs

2056f02

rustbasic added 4 commits March 30, 2024 16:28

Merge branch 'emilk:master' into patch28

72ccd11

Merge branch 'emilk:master' into patch28

52ed45f

Update wgpu_integration.rs

0668f39

Merge branch 'emilk:master' into patch28

ddfefa6

rustbasic mentioned this pull request Mar 30, 2024

Fix panic when dragging window between monitors of different pixels_per_point #4088

Closed

emilk added 6 commits March 30, 2024 19:33

Rename deferred_commands and use std::mem::take

d19c521

Merge branch 'master' into patch28

1be505b

Initialize minimized and maximized correctly on Mac

bf46a1d

Remove unused parameters

ed443b7

Clean up inner_rect_in_points and outer_rect_in_points

4beea7b

Improve comments

882ce4a

emilk approved these changes Mar 30, 2024

View reviewed changes

Fix Android build

3c9c23a

emilk changed the title ~~Fix InnerSize command not resizing viewport of Wayland~~ Fix ViewportCommand::InnerSize not resizing viewport on Wayland (#4211) Mar 30, 2024

emilk added the native-linux Problem specific to Linux label Mar 30, 2024

emilk merged commit 21835c3 into emilk:master Mar 30, 2024
20 checks passed

emilk pushed a commit that referenced this pull request Apr 2, 2024

Fix ViewportCommand::InnerSize not resizing viewport on Wayland (#4211

1eb333f

)

emilk mentioned this pull request May 29, 2024

Update to egui master rerun-io/rerun#6448

Merged

7 tasks

emilk added the bug Something is broken label Jun 26, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) #4211

Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) #4211

rustbasic commented Mar 22, 2024 •

edited

Loading

ColinKinloch commented Mar 22, 2024

ColinKinloch commented Mar 22, 2024 •

edited

Loading

rustbasic commented Mar 22, 2024

rustbasic commented Mar 23, 2024 •

edited

Loading

ColinKinloch commented Mar 23, 2024

rustbasic commented Mar 23, 2024

rustbasic commented Mar 30, 2024

emilk commented Mar 30, 2024

Fix ViewportCommand::InnerSize not resizing viewport on Wayland (#4211) #4211

Fix ViewportCommand::InnerSize not resizing viewport on Wayland (#4211) #4211

Conversation

rustbasic commented Mar 22, 2024 • edited Loading

ColinKinloch commented Mar 22, 2024

ColinKinloch commented Mar 22, 2024 • edited Loading

rustbasic commented Mar 22, 2024

rustbasic commented Mar 23, 2024 • edited Loading

ColinKinloch commented Mar 23, 2024

rustbasic commented Mar 23, 2024

rustbasic commented Mar 30, 2024

emilk commented Mar 30, 2024

Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) #4211

Fix `ViewportCommand::InnerSize` not resizing viewport on Wayland (#4211) #4211

rustbasic commented Mar 22, 2024 •

edited

Loading

ColinKinloch commented Mar 22, 2024 •

edited

Loading

rustbasic commented Mar 23, 2024 •

edited

Loading