Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Thiago Milczarek Sayao
On Mon, 5 Dec 2022 16:03:25 GMT, Michael Strauß wrote: >> Thiago Milczarek Sayao has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add test > > tests/system/src/test/java/test/robot/javafx/scene/SceneChangeShouldNotFocusStageTest.java >

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Kevin Rushforth
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Michael Strauß
On Mon, 5 Dec 2022 08:19:59 GMT, Ambarish Rapte wrote: > After the fix, the behavior looks similar on all platform, but this way the > user does not get any notification of the Scene change in Application's > content. i.e. the behavior change on Windows(icon not flashing) and Mac(not >

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Michael Strauß
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Thiago Milczarek Sayao
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Thiago Milczarek Sayao
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-05 Thread Ambarish Rapte
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-12-01 Thread Ambarish Rapte
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-11-16 Thread Thiago Milczarek Sayao
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-11-15 Thread Kevin Rushforth
On Sun, 13 Nov 2022 01:02:36 GMT, Thiago Milczarek Sayao wrote: >> Simple fix to not requestFocus() on scene change. >> >> The attached bug sample shows that the TextField focused on the scene >> remains focused when the scene comes back. > > Thiago Milczarek Sayao has updated the pull

Re: RFR: 8296621: Stage steals focus on scene change [v2]

2022-11-12 Thread Thiago Milczarek Sayao
> Simple fix to not requestFocus() on scene change. > > The attached bug sample shows that the TextField focused on the scene remains > focused when the scene comes back. Thiago Milczarek Sayao has updated the pull request incrementally with one additional commit since the last revision: