Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v3]

2023-08-16 Thread Ambarish Rapte
On Wed, 16 Aug 2023 09:28:57 GMT, Guillaume Tâche wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > Guillau

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v3]

2023-08-16 Thread Marius Hanl
On Wed, 16 Aug 2023 09:28:57 GMT, Guillaume Tâche wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > Guillau

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-16 Thread Guillaume Tâche
On Wed, 16 Aug 2023 07:08:10 GMT, Marius Hanl wrote: >> Guillaume Tâche has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8310885: Fixes review comment > > modules/javafx.graphics/src/test/java/test/javafx/stage/WindowTest.java line > 184

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v3]

2023-08-16 Thread Guillaume Tâche
> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and > before `show()`. > Now the `sizeToScene` flag is unset in these methods to ensure the right > values are set when the window is shown. Guillaume Tâche has updated the pull request incrementally with one addit

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-16 Thread Marius Hanl
On Tue, 8 Aug 2023 09:02:07 GMT, Guillaume Tâche wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > Guillaum

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-15 Thread Lukasz Kostyra
On Tue, 8 Aug 2023 09:02:07 GMT, Guillaume Tâche wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > Guillaum

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-15 Thread Michael Strauß
On Tue, 8 Aug 2023 09:02:07 GMT, Guillaume Tâche wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > Guillaum

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-08 Thread Guillaume Tâche
On Mon, 7 Aug 2023 15:18:48 GMT, Kevin Rushforth wrote: >> modules/javafx.graphics/src/test/java/test/javafx/stage/WindowTest.java line >> 85: >> >>> 83: } >>> 84: >>> 85: @Test >> >> Minor - I think it would be best to keep old code untouched to not mix up >> the file history in Git

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene [v2]

2023-08-08 Thread Guillaume Tâche
> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and > before `show()`. > Now the `sizeToScene` flag is unset in these methods to ensure the right > values are set when the window is shown. Guillaume Tâche has updated the pull request incrementally with one addit

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene

2023-08-07 Thread Kevin Rushforth
On Mon, 7 Aug 2023 14:09:51 GMT, Lukasz Kostyra wrote: >> `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` >> and before `show()`. >> Now the `sizeToScene` flag is unset in these methods to ensure the right >> values are set when the window is shown. > > modules/j

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene

2023-08-07 Thread Lukasz Kostyra
On Thu, 3 Aug 2023 14:57:50 GMT, Guillaume Tâche wrote: > `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and > before `show()`. > Now the `sizeToScene` flag is unset in these methods to ensure the right > values are set when the window is shown. Overall LGTM, t

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene

2023-08-07 Thread Lukasz Kostyra
On Thu, 3 Aug 2023 14:57:50 GMT, Guillaume Tâche wrote: > `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and > before `show()`. > Now the `sizeToScene` flag is unset in these methods to ensure the right > values are set when the window is shown. Don't see any i

Re: RFR: 8310885: Width/height of window is not set after calling sizeToScene

2023-08-03 Thread Kevin Rushforth
On Thu, 3 Aug 2023 14:57:50 GMT, Guillaume Tâche wrote: > `setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and > before `show()`. > Now the `sizeToScene` flag is unset in these methods to ensure the right > values are set when the window is shown. The fix looks r

RFR: 8310885: Width/height of window is not set after calling sizeToScene

2023-08-03 Thread Guillaume Tâche
`setHeight()` / `setWidth()` were ignored if called after `sizeToScene()` and before `show()`. Now the `sizeToScene` flag is unset in these methods to ensure the right values are set when the window is shown. - Commit messages: - 8310885: Width/height of window is not set aft