dj-raven / glasspane-popup Goto Github PK

Hi,

your use of DecimalFormat can depend on PC settings of a user. So it can put "," instead of "." for decimal separator.
private final DecimalFormat df = new DecimalFormat("#.###");

You can use this:
private final DecimalFormat df = new DecimalFormat( "#.###" , DecimalFormatSymbols.getInstance( Locale.US ) );
to avoid that bug.

Hello Raven, I love you very much uvu, thank you for your contributions, I saw potential in this repository to have something similar to those messages as android and experimenting with it I have run into a problem.

When opened and closed very quickly it throws a small exception.

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: Component must already be added to parent!

and something like this happens:

Inspecting the code, I see that it originates from the Popup class in the initAnimator() method and the problem is this line of code:

layout.setComponentConstraints(component,lc);

At first I thought that if I checked that it was already being managed by the layout with the isManagingComponent method it would be solved, but it was not the case, reviewing the code further I saw that the method uses a map to save each component with its respective constraint and this would work well, if it were not for the fact that it is inside an animator and this changes constantly...
So I relied better on the setConstraintMap method to maintain the same map within the animation, so I changed the problem line of code to this one.

layout.setConstraintMap(Map.of(component,lc));

and the problem was solved:

And yes, I click very fast 🥲