Closed Bug 339395 Opened 18 years ago Closed 17 years ago

Readonly textboxes and inputs shouldn't look active

Categories

(Core :: Widget: Gtk, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: helpwanted, testcase)

Gtk version of bug 339192 - <input readonly> in HTML really shouldn't look just like an active input, and Firefox (in bug 337344) and Thunderbird (in bug 297815) both need to have <textbox readonly="true"> not look like an active textbox that's broken, because you can't type in it for some buggy reason. Disabled background but active text seems like a reasonable appearance.

HTML testcase in attachment 223267 [details] and XUL testcase in 223268 (but unfortunately nsNativeThemeGTK.cpp isn't close enough to just port the patch, and in fact seems to ignore what I tell it about the disabled state of readonly textboxes).
This is a problem across all platforms and products (for example, Camino on Mac in my case, and also Firefox 1.5).

Moreover, on trunk (Camino/1.2+ and GranParadiso 3.0a1) readonly inputs also show a nonblinking I-beam cursor when focused, which is even more misleading.

See testcase attachment 147140 [details] on bug 143520.

Reassigning to Layout: Form Controls and setting All/All.
Component: Widget: Gtk → Layout: Form Controls
Keywords: testcase
OS: Linux → All
QA Contact: gtk → layout.form-controls
Hardware: PC → All
Bug 143520 may also be relevant.
Putting back where it belongs: please do actually read comment 0 before moving bugs around. You see it in Firefox 1.5 on Mac because I fixed it for Mac after 1.5 shipped. If you see a bug which is not "readonly textboxes on Linux have the same appearance as non-readonly textboxes" then it is not this bug, and you'll want to file another one.
Component: Layout: Form Controls → Widget: Gtk
OS: All → Linux
QA Contact: layout.form-controls → gtk
Hardware: All → PC
(In reply to comment #3)
> Putting back where it belongs: please do actually read comment 0 before moving
> bugs around. 

Sorry about that, Phil. I don't know where my mind was.

Fixed as part of the patch in bug 329846.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
No longer blocks: 337344
You need to log in before you can comment on or make changes to this bug.