RFR: WIP: 8260528: Clean glass-gtk sizing and positioning code [v4]
Thiago Milczarek Sayao
tsayao at openjdk.java.net
Mon Feb 15 00:10:09 UTC 2021
> This is a new approach to rewrite parts of gtk glass backend to be more clean.
>
> I will provide small "manageable" PR to incrementally make the backend better.
>
> This PR adresses cleanup of the Size and Positioning code. It makes code more "straightforward" and easier to maintain.
>
> Current status (Ubuntu 20.04):
> 
>
> (*) Some of the iconify tests are also failing on the main branch.
>
> `gradlew -PFULL_TEST=true -PUSE_ROBOT=true :systemTests:test --tests test.robot.javafx.stage.IconifyTest` on a second run produces 4 tests, 2 failures.
Thiago Milczarek Sayao has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 25 additional commits since the last revision:
- Reduce size notify to java
- Merge branch 'master' into glass_gtk_new_position_and_size
- Merge pull request #16 from openjdk/master
Update
- Reduce size notify to java
- Revert change to reduce size notify events due to frame extents adjustment - it makes some tests fail.
- Merge branch 'master' into glass_gtk_new_position_and_size
- Merge pull request #15 from openjdk/master
Update from jfx
- Avoid redundant resize notify
- Fix parent window being resizable (it should not)
- Minor fix to positioning
- ... and 15 more: https://git.openjdk.java.net/jfx/compare/2e19b548...5f87da28
-------------
Changes:
- all: https://git.openjdk.java.net/jfx/pull/367/files
- new: https://git.openjdk.java.net/jfx/pull/367/files/e8528ef2..5f87da28
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jfx&pr=367&range=03
- incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=367&range=02-03
Stats: 257585 lines in 5458 files changed: 125365 ins; 93757 del; 38463 mod
Patch: https://git.openjdk.java.net/jfx/pull/367.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/367/head:pull/367
PR: https://git.openjdk.java.net/jfx/pull/367
More information about the openjfx-dev
mailing list