Yep, its reasonable!
Will fix that and update diff.
The following is my input for this webrev:
In case the LAF is Aqua, the check for combobox
alignment is skipped.
Instead, if default LAF is Aqua, the LAF should be
changed to cross platform LAF to check if it works on other
LAF or not.
Either run this in a loop with checks for all LAF
versions available on OS, or check for default LAF at the
beginning and change it to Cross-Platform LAF to check for
The current changes makes the test pass but does
seem to give false positives by process of elimination for
Also, found the following minor issues which could
be looked into if possible:
variables done and error
and mnDayData could be declared as final.
@override annotations are missing
for methods used in ComboPopupCheckListener, PopupHandler.
panel is redeclared within scope
instead of reusing available variable in method createContentPane().
wildcard * is
used in import statements which according to
guidelines should be replace to use only the classes required.