benjamin84 / fest

Automatically exported from code.google.com/p/fest
0 stars 0 forks source link

JFileChooserFixture textbox access #176

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Very minor suggestion:

The textbox on the JFileChooserFixture is accessed through
"fileNameTextBox".  Textboxes in other fixtures are accessed through just
"textBox".  Should this be changed for consistency?

Version 1.0 alpha 3 - Windows XP SP2

Original issue reported on code.google.com by dbguse...@gmail.com on 15 Jul 2008 at 1:42

GoogleCodeExporter commented 9 years ago
I agree with you about naming consistency. My only concern is that having 
only "textBox" may confuse users. In this case (JFileChooserFixture), the name 
is 
pretty intuitive. On the other hand, there is only one text box in a 
JFileChooser.

Any thoughs?

Thanks!
-Yvonne

Original comment by wanghy1...@gmail.com on 15 Jul 2008 at 3:05

GoogleCodeExporter commented 9 years ago
Hi,

I have to agree with Yvonne, textBox would clash with ContainerFixture#textBox.
fileNameTextBox reinforces the fact that JFileChooser isn't really a container.
Also, JFileChooser might contain a custom component which might be a text box 
or it
might have a text box descendant.

Regards,
Csabi

Original comment by csaba.ju...@gmail.com on 17 Jul 2008 at 7:50

GoogleCodeExporter commented 9 years ago
Thanks Csabi. I think it is safe to mark this issue as 'invalid'.

Original comment by wanghy1...@gmail.com on 19 Jul 2008 at 3:49

GoogleCodeExporter commented 9 years ago
Set the module as a label, instead of being part of the title.

Original comment by Alex.Rui...@gmail.com on 1 Dec 2008 at 2:08