#468 bug
Paul Schreiber

EventObserver fires only once when observing radio buttons

Reported by Paul Schreiber | November 26th, 2008 @ 09:50 PM | in After 1.7

This is in Rails Trac: http://dev.rubyonrails.org/ticke...

When observing radio button elements, change events are fired only once, the first time that particular radio button is clicked. Subsequent clicks on that button (to another button, then back to the first button) do not cause the observer code given to Prototype to execute. This is semantically incorrect.

Tim Morrow has described the issue in depth in two posts to the spinoffs mailing list, proposed several fixes, and provided code for one of them.

John-David Dalton attached a patch to the trac ticket, which I have updated for the latest version of prototype, adjusted (quoted groupName) and attached.

Comments and changes to this ticket

  • Andrew Dupont

    Andrew Dupont November 29th, 2008 @ 05:11 AM

    • Tag changed from event, form, needs_tests, patched, radio to discuss, event, form, needs_tests, patched, radio
    • Milestone set to 1.7
    • State changed from “new” to “bug”
    • Assigned user set to “Andrew Dupont”

    Ick. How do we want to fix this? I'm not wild about the API, so can someone propose something that integrates better with the existing Form.EventObserver API?

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:01 AM

    • Tag changed from discuss, event, form, needs_tests, patched, radio to discuss, event, needs_tests, patched, section:dom

    [not-tagged:"form" tagged:"section:dom" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:15 AM

    • Tag changed from discuss, event, needs_tests, patched, section:dom to needs_discussion, needs_tests, section:dom
  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 02:26 AM

    • Tag changed from needs_discussion, needs_tests, section:dom to missing:tests, needs_discussion, section:dom

    [not-tagged:"needs_tests" tagged:"missing:tests" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 03:36 AM

    • Tag changed from missing:tests, needs_discussion, section:dom to needs:tests, needs_discussion, section:dom

    [not-tagged:"missing:tests" tagged:"needs:tests" bulk edit command]

  • Tobie Langel

    Tobie Langel July 24th, 2009 @ 03:38 AM

    • Tag changed from needs:tests, needs_discussion, section:dom to needs:discussion, needs:tests, section:dom

    [not-tagged:"needs_discussion" tagged:"needs:discussion" bulk edit command]

  • T.J. Crowder

    T.J. Crowder November 16th, 2009 @ 04:50 PM

    • Assigned user cleared.

    [responsible:none bulk edit command]

  • Andrew Dupont

    Andrew Dupont October 17th, 2010 @ 11:04 PM

    • Milestone changed from 1.7 to After 1.7
    • Importance changed from “” to “”

    Unless someone can produce a version that involves no new/altered APIs, this is too big of an issue for 1.7. Pushing it back.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

The Prototype JavaScript library.

Shared Ticket Bins

Attachments

Pages