Tapestry
  1. Tapestry
  2. TAPESTRY-2349

FormFragment should be tied to the 'click' event of the associated checkbox

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0.11
    • Fix Version/s: 5.0.13
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      IE6 (and Firefox, but less of an issue there)

      Description

      The FormFragment is currently tied to the 'change' method of the checkbox it is associated to. This causes a problem in IE6 - the user has to tab out of or click out of the checkbox before the 'change' event gets fired. Generally the desired effect should be that the form fragment shows up immediately after clicking the checkbox and not being required to cause the checkbox to lose focus.

      The quick hack is to simply change 'change' to 'click' in tapestry.js inside the linkCheckboxToFormFragment method. I'm not sure if there are other negative side effects to doing so, but it seems to work in FF and IE.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        58d 7h 9m 1 Howard M. Lewis Ship 09/Jun/08 00:20
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12567044 ] jira [ 12589788 ]
        Mark Thomas made changes -
        Workflow jira [ 12428800 ] Default workflow, editable Closed status [ 12567044 ]
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Fix Version/s 5.0.13 [ 12313205 ]
        Status Open [ 1 ] Closed [ 6 ]
        Assignee Howard M. Lewis Ship [ hlship ]
        Hide
        Howard M. Lewis Ship added a comment -

        This was actually fixed in 5.0.12.

        Show
        Howard M. Lewis Ship added a comment - This was actually fixed in 5.0.12.
        Skow created issue -

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Skow
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development