ICEfaces
  1. ICEfaces
  2. ICE-10100

Change default ajax event to 'valueChange' for input components

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Labels:
      None
    • Environment:
      Any

      Description

      According to the JSF specification, the 'valueChange' ajax event is the default event for input components such as h:inputText and h:selectOneMenu. Our input components use 'blur' as the default event. In order to better align to the JSF specification, our input components should have 'valueChange' as their default ajax event.

        Activity

        Hide
        Arturo Zambrano added a comment -

        The problem with the richTextEntry test is in the app itself. The valueChangeListener is firing, but that test is missing render="@all" execute="@all" in the ajax tag to render the text fields under the editor, and that's why the message is not displayed on the page. Please fix.

        Show
        Arturo Zambrano added a comment - The problem with the richTextEntry test is in the app itself. The valueChangeListener is firing, but that test is missing render="@all" execute="@all" in the ajax tag to render the text fields under the editor, and that's why the message is not displayed on the page. Please fix.
        Hide
        Arturo Zambrano added a comment -

        Corrected default execute and render values for valueChange event to execute=@this, render=@all. No need to fix the test app now.

        Show
        Arturo Zambrano added a comment - Corrected default execute and render values for valueChange event to execute=@this, render=@all. No need to fix the test app now.
        Hide
        Liana Munroe added a comment -

        Re-opening for possible regression.
        Showcase ace:textEntry/textAreaEntry
        > All demos: validation messages generated on any textEntry/textAreaEntry demo are rendered on the other textEntry/textAreaEntry pages.
        To reproduce
        Navigate to ace:textEntry > Overview
        Insert cursor in first field then tab through 2 more fields.
        Note validation messages for required fields.
        Navigate to ace:textEntry > Auto Tab
        Note that the messages rendered in the Overview demo are now displayed in the Auto Tab demo.

        Show
        Liana Munroe added a comment - Re-opening for possible regression. Showcase ace:textEntry/textAreaEntry > All demos: validation messages generated on any textEntry/textAreaEntry demo are rendered on the other textEntry/textAreaEntry pages. To reproduce Navigate to ace:textEntry > Overview Insert cursor in first field then tab through 2 more fields. Note validation messages for required fields. Navigate to ace:textEntry > Auto Tab Note that the messages rendered in the Overview demo are now displayed in the Auto Tab demo.
        Hide
        Arturo Zambrano added a comment -

        I couldn't reproduce the last issue. Please re-test.

        Show
        Arturo Zambrano added a comment - I couldn't reproduce the last issue. Please re-test.
        Hide
        Liana Munroe added a comment -

        No longer able to reproduce Icefaces 4 trunk r42039. Tomcat7, all browsers.

        Show
        Liana Munroe added a comment - No longer able to reproduce Icefaces 4 trunk r42039. Tomcat7, all browsers.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: