Opened 8 years ago

Closed 8 years ago

#13707 closed defect (fixed)

dojox.mobile.parser is failing with a dojox.mobile.TextBox on IE8.

Reported by: Ed Chatelain Owned by: bill
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: Cc: Douglas Hays, bill
Blocked By: Blocking:

Description

dojox.mobile.parser is failing with a dojox.mobile.TextBox? on IE8.

The dojox.mobile.parser's instaniate function is throwing an error: "This command is not suppported" when trying to process a dojox.mobile.TextBox? on IE8 (with Browser Mode: IE8 and Document Mode: IE8 Standards).

You can see the problem by running the dojox/mobile/tests/test_FormControls.html test on IE8.

I ran into this when working on http://bugs.dojotoolkit.org/ticket/13611

Attachments (1)

13707-patch.txt (2.0 KB) - added by Ed Chatelain 8 years ago.
Fix for mobile TextBox?, Button and RadioButton? on IE.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 8 years ago by Ed Chatelain

Turns out the problem was caused when trying to call

node.setAttribute(attrNames[lc]
name, value)

when name is "type".

The fix is to add: _setTypeAttr: null

The problem was also happening with dojox/mobile/Button.js and RadioButton?.js

I will provide a patch.

Doug, thanks for the help.

Changed 8 years ago by Ed Chatelain

Attachment: 13707-patch.txt added

Fix for mobile TextBox?, Button and RadioButton? on IE.

comment:2 Changed 8 years ago by bill

Milestone: tbd1.7
Owner: changed from ykami to bill
Status: newassigned

Ah right, dojo.parser used to have the same problem too, before the refactor, that IE would report a type=... setting on nodes even though the markup didn't say anything about type=...

I'll check in your patch.

comment:3 Changed 8 years ago by bill

Resolution: fixed
Status: assignedclosed

In [26287]:

Make dojox.mobile.parser work with dojox.mobile.TextBox? etc. on IE8, patch from edchat (IBM, CCLA), thanks! Fixes #13707 !strict.

Note: See TracTickets for help on using tickets.