RFR: Draft: 8316423: [linux] Secondary Stage does not respect Scene's dimensions when shown [v3]
Thiago Milczarek Sayao
tsayao at openjdk.org
Sat Oct 14 18:39:36 UTC 2023
> It looks like a condition between calls from `set_visible` to `request_focus`. When `request_focus` is called, the window is not yet mapped (map is requested on `set_visible`).
>
> 200x200 is the default gtk window size if none is specified. Since the window is presented at the requested size and then changes to 200x200 I would guess the size request on `gtk_window_present` (which is called on `request_focus`) is being queued and released later. But I'm not really sure.
>
> With this change immediate `request_focus` after the window is created will be ignored. But the default is to request focus on map, so it will work. `request_focus` will only be useful when called after the window is mapped (presented on the screen).
>
> This also fixes [JDK-8316425](https://bugs.openjdk.org/browse/JDK-8316425)
Thiago Milczarek Sayao has updated the pull request incrementally with one additional commit since the last revision:
Experiment on not realizing the window
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/1249/files
- new: https://git.openjdk.org/jfx/pull/1249/files/c5b631d5..d5ef6de1
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=1249&range=02
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=1249&range=01-02
Stats: 75 lines in 5 files changed: 31 ins; 35 del; 9 mod
Patch: https://git.openjdk.org/jfx/pull/1249.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/1249/head:pull/1249
PR: https://git.openjdk.org/jfx/pull/1249
More information about the openjfx-dev
mailing list