MantisBT - NetSurf
View Issue Details
0002521NetSurfLayoutpublic2017-02-07 12:162017-02-11 18:18
ReporterRik Griffin 
Assigned To 
PlatformIyonixOSRISC OSOS Version5.22
Product Version3.7 
Target VersionFixed in Version 
Fixed in CI build #
Reported in CI build #3978
URL of problem page
Summary0002521: Inputs with attributes "readonly" or "disabled" are rendered inconsitently
DescriptionInputs with type "text" are rendered greyed if they have the attribute "readonly" but not if they have the "disabled" attribute.

Inputs with type "checkbox" are never greyed even if they have one or both of these attributes. The same applies to type "radio".

For what it's worth, Chrome renders "disabled" inputs greyed out, whereas Firefox renders both "disabled" and "readonly" as greyed out. I don't think either of those browsers get it fully correct, as they both allows the user to change a "readonly" checkbox/radio.

I think the correct behaviour would be for both "readonly" and "disabled" input items to be greyed and not editable by the user?

Steps To ReproduceLoad attached file into Netsurf
Additional InformationVersion 3.7 build 3978
TagsNo tags attached.
Attached Fileshtml test.html (1,084) 2017-02-07 12:16

Chris Young   
2017-02-10 19:58

According to that, our textbox behaviour has the readonly/disabled visual greyed out effect the wrong way round. I can also gain focus on, and edit, the disabled textbox, which is incorrect.
Chris Young   
2017-02-11 17:31   
CI build 3983 makes disabled textboxes non-editable.
Chris Young   
2017-02-11 18:18   
I've also switched the visual indication over so that disabled textboxes get greyed out (and read-only ones don't). I believe behaviour of textboxes is correct now.

Checkboxes and other form elements still need fixing to be uneditable and visually disabled.

Issue History
2017-02-07 12:16Rik GriffinNew Issue
2017-02-07 12:16Rik GriffinFile Added: test.html
2017-02-10 19:58Chris YoungNote Added: 0001516
2017-02-10 19:58Chris YoungStatusnew => confirmed
2017-02-10 19:58Chris YoungCategoryRISC OS-specific => Layout
2017-02-10 19:58Chris YoungProduct Version => 3.7
2017-02-11 17:31Chris YoungNote Added: 0001517
2017-02-11 18:18Chris YoungNote Added: 0001518