Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#6651 closed task (fixed)

TextBox: disabled TextBox-based widgets don't look disabled on safari

Reported by: bill Owned by: Douglas Hays
Priority: high Milestone: 1.2
Component: Dijit - LnF Version: 1.1.0
Keywords: Cc:
Blocked By: Blocking:

Description

See second tab in themeTester.html. On FF etc you can see the disabled fields are gray:

However on safari they look enabled:

Not sure why. Web inspector says that the CSS is color:gray.

Attachments (2)

ff.png​ (7.2 KB) - added by bill 12 years ago.
safari.png​ (6.9 KB) - added by bill 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by bill

Attachment: ff.png​ added

Changed 12 years ago by bill

Attachment: safari.png​ added

comment:1 Changed 12 years ago by bill

Keywords: ibm removed

comment:2 Changed 12 years ago by Douglas Hays

Resolution: → fixed
Status: new → closed

(In [14181]) Fixes #6651. Safari needed a special gray color.

comment:3 Changed 12 years ago by dante

Resolution: fixed
Status: closed → reopened

briefly: why not remove the 4 special rules for safari, and just set color:#eee instead of color:grey for everyone. #eee == grey

comment:4 Changed 12 years ago by Douglas Hays

Resolution: → fixed
Status: reopened → closed

safari is rendering everything much darker than FF or IE, so #eee looked bad on those browsers. You might be able to save bytes by using dijitDisabled instead of dijitTextBoxDisabled, et al but it would take some investigation to make sure there's no side-effect.

comment:5 Changed 12 years ago by bill

(In [15035]) Fix text to be gray when disabled (again), regression from [14181], fixes #6087, refs #6651.

Also fixed some spacing in Textarea.js.

Note: See TracTickets for help on using tickets.