Skip to main content

Please review 6498090 template/config load "launch editor" checkbox

2 replies [Last post]
bkurotsu
Offline
Joined: 2004-12-13

Simply rename the checkbox to be appropriate -
if loading template - offer to load TE
if loading config - offer to load CE

The behavior was already correct, it was just the text which was wrong.

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

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
fda
Offline
Joined: 2005-05-27

reviewed. fix is ok.
but I would suggest use more readable names for properties:

launchteditor --> launchtempleditor
launncheditor --> launchcfgeditor

just for easier recognition.

please integrate the fix with or without the name changes

bkurotsu
Offline
Joined: 2004-12-13

Thanks Dima.

Yes, I agree about the property names. :) Maybe I will come back later and revise them - the one spelled incorrectly was already there, so I left it alone.

Integrated rev 1470.