Opened 12 years ago

Closed 12 years ago

Last modified 8 years ago

#6262 closed defect (wontfix)

[patch][ccla]IE only: ValidationTextBox fonts are not being inherited

Reported by: haysmark Owned by: Douglas Hays
Priority: high Milestone: 1.2
Component: Dijit - Form Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

Load in IE: http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/test_Spinner.html

The second spinner's buttons are a little too short since the font is not Courier as it should be. Same problem with any ValidationTextBox? subclass. This happens across all themes.

Attachments (1)

6262.patch (5.1 KB) - added by Douglas Hays 12 years ago.
Move onresize code from _Spinner to ValidationTextBox?? and allow for inherited font to change. Add some font family/size tests to help verify correct function.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by Douglas Hays

Description: modified (diff)
Summary: test_Spinner: white gap beneath buttons in IE7IE only: ValidationTextBox fonts are not being inherited

comment:2 Changed 12 years ago by Douglas Hays

Milestone: 1.1

[12806] regression.

Changed 12 years ago by Douglas Hays

Attachment: 6262.patch added

Move onresize code from _Spinner to ValidationTextBox?? and allow for inherited font to change. Add some font family/size tests to help verify correct function.

comment:3 Changed 12 years ago by Douglas Hays

This problem can be worked around by adding a CSS rule like:
#widget_integerspinner2 INPUT { font-family: Courier; }

comment:4 Changed 12 years ago by Douglas Hays

Summary: IE only: ValidationTextBox fonts are not being inherited[patch][ccla]IE only: ValidationTextBox fonts are not being inherited

comment:5 Changed 12 years ago by Douglas Hays

(In [13158]) References #6262. Unit test CSS workaround for IE font-family inheritance problems with ValidationTextBox? subclass widgets.

comment:6 Changed 12 years ago by Douglas Hays

Milestone: 1.11.2

unit test workaround committed in 1.1

comment:7 Changed 12 years ago by bill

Description: modified (diff)

We decided not to support this in dijit, see #6307. Users must set fonts manually (if they want to change them).

comment:8 Changed 12 years ago by bill

Resolution: wontfix
Status: newclosed

comment:9 Changed 8 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.