Skip to main content

Please review my fix for 6456337

3 replies [Last post]
naryl
Offline
Joined: 2007-11-16
Points: 0

"Apply" and "Reset" were replaced with "Cancel".

Note that there was commented out code referring to cancelBtn. Looks like it was present sometime ago.

https://jtharness.dev.java.net/source/browse/jtharness?rev=1042&view=rev
https://jtharness.dev.java.net/source/browse/jtharness?rev=1043&view=rev

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Integrated.

Note that with uif.createCancelButton, you do not need to add the Cancel button properties to the local properties file. You do need to add the tooltip (.tip) property though, which you did do.

bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Did basic testing, looks good! Integrating.

Docs will need updating for this.

bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Looks good, thanks!