[OpenJDK 2D-Dev] [9] RFR JDK-6801613, , Cross-platform pageDialog and printDialog top margin entry broken
prasanta sadhukhan
prasanta.sadhukhan at oracle.com
Tue Apr 5 11:08:39 UTC 2016
Hi All,
Please review a fix for jdk9
Bug: https://bugs.openjdk.java.net/browse/JDK-6801613
webrev: http://cr.openjdk.java.net/~psadhukhan/6801613/webrev.00/
The issue was the PrinterJob cross-platform "Page setup dialog" top
margin entry is broken.
If you enter a value in the top margin field and hit "OK", the value
is not updated in the PageFormat instance return value,
nor is it updated in the attribute set used to call the pageDialog method.
Tabbing out of the top margin field before hitting OK does not fix the
problem.
Likewise, clicking on a different margin entry field before hitting OK
also does not fix the problem.
The reason was we created topMargin JFormattedTextField once and added
focusListener and ActionListener to this textfield but
we overwrote the topMargin textField again by creating another instance
and in that instance, we failed to add actionListener or focusListener,
so when we tabbed out of topmargin field, focusLost() method was not
called which would have called updateMargin() method to update the margins.
Fix was to remove this erroneous instance creation and overwriting of
topMargin textfield.
Regards
Prasanta
More information about the 2d-dev
mailing list