Bug 67690 - [Forms] setting text disabled looses the border
Summary: [Forms] setting text disabled looses the border
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Dejan Glozic CLA
QA Contact:
URL:
Whiteboard:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2004-06-17 14:33 EDT by Anthony Hunter CLA
Modified: 2005-06-02 10:24 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anthony Hunter CLA 2004-06-17 14:33:37 EDT
Take the org.eclipse.ui.forms.examples FormView and make the Text field
disabled by adding the lines:

		text.setText("This is the text for the text field");
		text.setEnabled(false);

The text is greyed and you cannot edit the text in the text field as expected.

However there is a problem that the flat border has been lost.
Comment 1 Debbie Wilson CLA 2004-06-17 14:52:11 EDT
Which version of 3.0 are you using?
Comment 2 Dejan Glozic CLA 2004-06-17 14:53:55 EDT
Use 'setEditable(false)' instead.
Comment 3 Anthony Hunter CLA 2004-06-17 15:02:53 EDT
We are using 3.0 RC2 (200406111814).

I will pass on the information, setEditable(false) would seem to do as a 
workaround.

Comment 4 Måns af Klercker CLA 2005-04-11 15:38:03 EDT
I still get same bug on 3.1M6. When test.setEnabled(true) is called again, the
border is not added either. It takes a reflow() on the form (I found out when
expanding a folded section on the same page) to make the borders show again. 

It would sure be nice to have this fixed, since the editable=false property
still makes the control focusable and sets a caret inside it, even though you
can't type anything into it.
Comment 5 Dejan Glozic CLA 2005-06-02 10:24:27 EDT
setEditable is preferred because by moving focus into it, you can get screen 
readers to read it for you, while disabled controls cannot get focus.