constellation-app / constellation

A graph-focused data visualisation and interactive analysis application.
https://constellation-app.com
Apache License 2.0
386 stars 58 forks source link

Test Notifications OK button throws exception #2139

Closed andromeda-224 closed 2 months ago

andromeda-224 commented 3 months ago

Prerequisites

Description

An Unexpected exception is thrown on Test Notifications when OK clicked: image

Steps to Reproduce

  1. Create a new Graph.

  2. Select menu item Experimental -> Developer -> Test Notifications

  3. Click OK. An Unexpected Exception is thrown.

Expected behaviour: Expected a series of notification dialogs. This can be seen if you click Cancel instead of OK.

Actual behaviour:

Reproduces how often: 100%

Additional Information

Happening on v.3.0.0 - rc1. No exception thrown in v02.11.0.

HolandCrick commented 3 months ago

problem is not only with ok button but also in cancel and X all three giving error.

andromeda-224 commented 3 months ago

The notification dialogs (plain, warning, info, error) displayed are expected. The exception on the Ok button click is not expected and should be fixed.

andromeda-224 commented 3 months ago

The issue with "Show Details" and "Hide Details" being difficult to see in FlatLaf Dark Mode is not related to https://github.com/constellation-app/constellation/issues/2137. This is fixed with stylesheet. New ticket has been created for Display alert dialogs appearing on different screen #2159.

HolandCrick commented 2 months ago

@andromeda-224 Thanks for addressing the bug. It's generating 13 Notifications without exception error.

image