Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modalité des AlertBox #146

Closed
WinXaito opened this issue Jun 27, 2016 · 6 comments
Closed

Modalité des AlertBox #146

WinXaito opened this issue Jun 27, 2016 · 6 comments
Assignees
Labels
Milestone

Comments

@WinXaito
Copy link
Collaborator

WinXaito commented Jun 27, 2016

Avec les nouvelles AlertBox, elles ne sont plus modale par rapport à la fenêtre un principale, ce qui est assez gênant.

Edit: Pour les pages de statistiques également.

@WinXaito
Copy link
Collaborator Author

J'ai jeter un coup d'oeil, c'est vraiment étrange car initModality(Modality.APPLICATION_MODAL) devrait jouer...

A moins que ça devrait être WINDOW_MODAL ?

@firm1
Copy link
Owner

firm1 commented Jun 28, 2016

Justement c'est ce que je n'ai pas compris non plus. Tu as refais le test
sur la master ?

Le mar. 28 juin 2016 20:22, WinXaito notifications@github.com a écrit :

J'ai jeter un coup d'oeil, c'est vraiment étrange car
initModality(Modality.APPLICATION_MODAL) devrait jouer...

A moins que ça devrait être WINDOW_MODAL ?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#146 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFyPX347Q7QEHqY685xh00lv2ZRUwqNdks5qQWZNgaJpZM4I_Zw-
.

@WinXaito
Copy link
Collaborator Author

Késako ? Quoi master ?

@firm1
Copy link
Owner

firm1 commented Jun 28, 2016

Excuse moi, je parlais de la branche master.

Le mar. 28 juin 2016 20:41, WinXaito notifications@github.com a écrit :

Késako ? Quoi master ?


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#146 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFyPX9lTmCiwsroikxYJUlNKUq2Yp5UDks5qQWrWgaJpZM4I_Zw-
.

@WinXaito
Copy link
Collaborator Author

Bon j'ai trouvé un moyen de faire fonctionner cela, il suffit de faire:

alert.initOwner(primaryStage);

Mais il ne me semblais pas que c'était le cas avant.
J'ajoute cela partout ? (Il ne me semble pas que ça pose un problème, car on a quasiment tout le temps accès à mainApp).

Ou plus simple, je modifie le constructeur pour ajouter primaryStage.

@firm1
Copy link
Owner

firm1 commented Jun 29, 2016

C'est corrigé du coup.

@firm1 firm1 closed this as completed Jun 29, 2016
@firm1 firm1 added this to the 1.2.1 Hotfix milestone Jun 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants