Modify

Opened 11 years ago

Closed 11 years ago

#10693 closed defect (fixed)

`select.val()` does not work on Internet Explorer with Trac 0.11.7

Reported by: Jun Omae Owned by: Ryan J Ollos
Priority: normal Component: AdminEnumListPlugin
Severity: normal Keywords:
Cc: Stepan Riha, Jun Omae Trac Release: 0.11

Description

When changing a select box in "Order" columns, I got the following error on IE 8 with Trac 0.11.7. No error with IE 8 and Trac 0.12.4.

Message: No object
Line: 136
Char: 3
Code: 0
URI: http://localhost:3000/chrome/adminenumlistplugin/adminenumlist.js

It looks that select.val() does not work with jquery 1.2.6 on IE.

I found the workaournd that we can use $(':selected', select).val() instead of select.val(), fix-select-val-on-ie-r12424.diff.

Attachments (1)

fix-select-val-on-ie-r12424.diff (877 bytes) - added by Jun Omae 11 years ago.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by Jun Omae

comment:1 Changed 11 years ago by Ryan J Ollos

Status: newassigned

comment:2 Changed 11 years ago by Ryan J Ollos

It looks like the issue might be due to the option elements not having a value attribute. See here and here.

comment:3 Changed 11 years ago by Ryan J Ollos

(In [12457]) Refs #10693: Workaround for select.val() not working on IE8 with Trac 0.11.7. This appears to be due to the option elements not having a value, an issue that was fixed in later versions of jQuery. Patch by Jun Omae (jun66j5).

comment:4 Changed 11 years ago by Ryan J Ollos

Summary: `select.val()` does not work on Internet Exploer with Trac 0.11.7`select.val()` does not work on Internet Explorer with Trac 0.11.7

comment:5 Changed 11 years ago by Ryan J Ollos

Resolution: fixed
Status: assignedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.