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

Old notifications shown are not aligned with new notifications when the window is moved #1114

Closed
Ciruman opened this issue Mar 8, 2019 · 2 comments

Comments

Projects
None yet
2 participants
@Ciruman
Copy link
Contributor

commented Mar 8, 2019

How to reproduce:

  1. Start HelloNotifications.java standalone application
  2. Choose "Set Owner"
  3. Change fade delay to maximum
  4. Show a notification
  5. Move the application window
  6. Show another notification

Current status:
The new notification will be shown wrongly aligned.

Expected:
The new notifications are aligned with the new ones.

@Ciruman

This comment has been minimized.

Copy link
Contributor Author

commented Mar 12, 2019

#1116 will also fix the first time shown, currently wrong.

Ciruman added a commit to Ciruman/controlsfx that referenced this issue Mar 22, 2019

controlsfx#1114 Notifications: Fix Old notifications shown are not al…
…igned with new notifications when the window is moved

Ciruman added a commit to Ciruman/controlsfx that referenced this issue Apr 5, 2019

controlsfx#1114 Notifications: Fix notifications position
The position of the elements does not use the notification position
anymore. It uses the position where the notifications will be shown.

The following errors are fixed with this change:
- Fix old notifications shown not aligned with new notifications
when the window is moved
- Fix animation, it does not use the position of the notification
because if the animation is not finished, the position is not
correct. Previously, the position between the first and second
notification was not right if there were several notifications.
@Ciruman

This comment has been minimized.

Copy link
Contributor Author

commented May 3, 2019

abhinayagarwal added a commit that referenced this issue May 11, 2019

Merge pull request #1132 from Ciruman/JDK8/notifications_position
#1114 Notifications: Fix notifications position
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.