160158 - Popup Menu won't select proper item

Status: VERIFIED FIXED

(Camino Graveyard :: HTML Form Controls, defect)

Description

[Brian Zappia]

Using Chimera 0.4 Milestone on MacOS X 10.1.5 The "Select Your State" Popup menu
on the above link does not allow you to select any state but alabma.

Reproduceable: always

Steps to Reproduce: 
  1. go to http://help.rr.com/rrwho.asp
  2. selet "Time Warner" in the "Select Your Provider" Popup menu
  3. selet any state except Alabama in the "Select Your State" Popup menu

Comment 1

[Brian Zappia]

Sorry, trackpad slip there's more
expected result:
  State is selected

Actual Result:
  Alabama is selected

Comment 2

[saari (gone)]

neat, are there any other sites that exhibit the problem? Is is a visual glitch
only (does the right state get submitted)?

Comment 3

[Brian Zappia]

This does not appear to be simply a visual glitch because if you notice the
third popup menu only has birmingham (a city in alabama) as a choice.

Comment 4

[Mike Pinkerton (not reading bugmail)]

form controls -> bryner

Comment 5

[Greg K.]

Confirmed using Chimera/20020724. Though it works using FizzillaCFM/2002072203,
JavaScript errors ("list.click is not a function") are generated when the
SELECTs are changed.

Adding qawanted for testcase minimization.

Comment 6

[Brian Zappia]

Another example of this at http://consumer.usa.canon.com/techsupport/index.html

Comment 7

[Brian Zappia]

Attachment: minimized testcase

Comment 8

[Greg K.]

[e-mail], thanks for minimizing the HTML. However, the JavaScript
still needs minimization, and it still generates an error I mentioned in comment
5. (It's still possible that if that error is fixed, the whole things works, you
understand.)

Comment 9

[Brian Zappia]

Attachment: Javascript Minimized

Comment 10

[Derek Schrock]

Would bug #160538 be a dup of this bug?

See http://bugzilla.mozilla.org/show_bug.cgi?id=160538#c10 for my reasoning

Comment 11

[Brian Ryner (not reading)]

Attachment: patch

Don't use the content id as an identifier for option elements, since it's
always 0 for dynamically inserted elements (and I understand that it will be
removed soon anyway).  Just stuff the content pointer into the menu item tag.

Comment 12

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 96912 [details] [diff] [review]
patch

r=pink

Comment 13

[Brian Ryner (not reading)]

checked in.

Comment 14

[sairuh (rarely reading bugmail)]

went thru the url and test cases here, and this looks fixed using 2002.08.28.05.

Comment 15

[sairuh (rarely reading bugmail)]

*** Bug 160538 has been marked as a duplicate of this bug. ***