FLUID-3462: Autobinding not working for radio buttons when making a selection with the keyboard: using Safari 4

Metadata

Source
FLUID-3462
Type
Bug
Priority
Critical
Status
Closed
Resolution
Fixed
Assignee
Justin Obara
Reporter
Justin Obara
Created
2010-01-07T10:04:39.000-0500
Updated
2013-09-06T14:02:26.118-0400
Versions
  1. 1.1.2
Fixed Versions
  1. 1.2
Component
  1. Infusion Builder
  2. Renderer

Description

Autobinding not working for radio buttons when making a selection with the keyboard

Steps to reproduce:

1) Open the renderer demo

2) Click the render button to render out the data

3) Using the keyboard change the radio button selection

Notice that the data to the right isn't updated, but if you use the mouse to change the radio button the data is updated.

This means that for components that render out radio buttons (such as the Infusion Builder) changes made to the radio button selection with the keyboard will have no effect.

Environments

Safari 4 (Mac OS 10.5, 10.6, Windows 7), Chrome nightly

Attachments

Comments

  • Antranig Basman commented 2010-01-07T10:08:02.000-0500

    Patch to manually trigger on "keyup" on radio buttons. Really disagreeable necessity.

  • Colin Clark commented 2010-01-07T12:39:44.000-0500

    Let's try a "grown up patch" without the crass comments. Otherwise the code looks reasonable.

  • Justin Obara commented 2010-01-07T13:00:48.000-0500

    I tried out the patch using Safari 4 and FF 3.5.

    Safari 4 is now working, and there are no changes to FF 3.5.

  • Antranig Basman commented 2010-01-07T13:12:17.000-0500

    This issue has been reported upstream at Webkit as https://bugs.webkit.org/show_bug.cgi?id=33328 - it appears to affect every browser based on recent webkit.

  • Antranig Basman commented 2010-01-07T13:13:59.000-0500

    Patch without crass comment attached. Apologies for the early-morning red fog 😛

  • Justin Obara commented 2010-01-19T16:50:02.000-0500

    tested the patch with the renderer demo, and it appears to working properly now in safari 4

  • Antranig Basman commented 2010-01-19T21:35:40.000-0500

    Fixed at revision 9045. Depending on the Chrome release schedule, we may need to make a detect for that browser too.