8154847: Rendering is incorrect or not visible with StageStyle.UNIFIED on some graphics cards#6
Conversation
|
👋 Welcome back crschnick! A progress list of the required criteria for merging this PR into |
|
@crschnick This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be: You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 3 new commits pushed to the
Please see this link for an up-to-date comparison between the source branch of this pull request and the As you do not have Committer status in this project an existing Committer must agree to sponsor your change. ➡️ To flag this PR as ready for integration with the above commit message, type |
|
This backport pull request has now been updated with issue from the original commit. |
|
|
|
/approval |
|
@crschnick usage: |
|
@crschnick As Skara notes, you forgot the "request" modifier for the "approval" command. Something like this is what you will need: |
|
/approval request This JBS issue has seen a lot of activity over the years, so there is probably demand for it |
|
@crschnick |
|
In general, how for can backporting go? I would argue this fix has even more relevance for older applications and JavaFX runtimes. If you are running older applications on older hardware, there is a higher change the application uses UNIFIED than now and also a higher chance that the bug occurs on older systems |
|
Approved. Since this is a clean backport, it only needs Maintainer approval. Review is optional (and not needed in this case). Once the approval is recorded, you may integrate it. /approve yes |
|
@kevinrushforth |
That would be up to the maintainer of each code line. Gluon maintains jfx25u and jfx17u, so @johanvos is the approver. Feel free to make the request and he will decide. There is currently no maintainer for the the OpenJFX 11u or 8u code lines. The jfx11u repo has been archived and we never created a jfx8u GitHub repo (that code line had no maintainer when we moved from Mercurial to git). |
|
/integrate |
|
@crschnick |
|
@crschnick I completely missed that this was waiting for a sponsor. I'll do that now. /sponsor |
|
Going to push as commit 89e4d11.
Your commit was automatically rebased without conflicts. |
|
@kevinrushforth @crschnick Pushed as commit 89e4d11. 💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored. |
Hi all,
This pull request contains a backport of commit 28bde153 from the openjdk/jfx repository.
The commit being backported was authored by Christopher Schnick on 5 Feb 2026 and was reviewed by Lukasz Kostyra and Kevin Rushforth.
Thanks!
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jfx26u.git pull/6/head:pull/6$ git checkout pull/6Update a local copy of the PR:
$ git checkout pull/6$ git pull https://git.openjdk.org/jfx26u.git pull/6/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 6View PR using the GUI difftool:
$ git pr show -t 6Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jfx26u/pull/6.diff
Using Webrev
Link to Webrev Comment