ICEfaces
  1. ICEfaces
  2. ICE-10975

icecore:focusManager - Support setting focus to buttons and links

    Details

      Description

      The icecore:focusManager tag only supports setting focus to components that are an instance of UIInput. This request is to see if support for buttons and links can be added so that on load of a page, a button or link can receive focus.

      Testing the following code:
                  <icecore:focusManager for="buttonTwo">
                      <ace:pushButton id="buttonOne" value="Button One"/>
                      <ace:pushButton id="buttonTwo" value="Button Two"/>
                      <ace:pushButton id="buttonThree" value="Button Three"/>
                  </icecore:focusManager>
      Shows the following warning: The "for" attribute points to a component that is not an instance of UIInput

        Activity

        Hide
        Mircea Toma added a comment -

        Implemented Focusable functionality for push and link buttons.

        Show
        Mircea Toma added a comment - Implemented Focusable functionality for push and link buttons.
        Hide
        Ken Fyten added a comment -

        Should also add support for ace:radioButtons and ace:checkboxButtons.

        Show
        Ken Fyten added a comment - Should also add support for ace:radioButtons and ace:checkboxButtons.
        Hide
        Carmen Cristurean added a comment - - edited

        Added tests for pushButton, linkButton, radioButtons, checkboxButtons with focusManger.
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/focusManager
        Test page: /focusManagerFor.jsf

        A regression was found for both ace:pushButton, and ace:linkButton: the actionListener does not appear to fire once linkButton's 'value', or pushButton's 'label' attribute is changed dynamically.
        To reproduce, use the Dynamic Attribute QA tests for these components:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/linkButton
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/pushButton
        Test pages: /pushButtonDynamic.jsf, or /linkButtonDynamicAttribute.jsf

        These regressions cannot be reproduced with ICEfaces 4 Trunk Jenkins Build #1848 (r.48494).

        Show
        Carmen Cristurean added a comment - - edited Added tests for pushButton, linkButton, radioButtons, checkboxButtons with focusManger. http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/focusManager Test page: /focusManagerFor.jsf A regression was found for both ace:pushButton, and ace:linkButton: the actionListener does not appear to fire once linkButton's 'value', or pushButton's 'label' attribute is changed dynamically. To reproduce, use the Dynamic Attribute QA tests for these components: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/linkButton http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/pushButton Test pages: /pushButtonDynamic.jsf, or /linkButtonDynamicAttribute.jsf These regressions cannot be reproduced with ICEfaces 4 Trunk Jenkins Build #1848 (r.48494).
        Hide
        Mircea Toma added a comment -

        Assign ID to the element that needs to be focused, but do it programmatically to ensure that the DOM updates for the component propagates to the root of its markup.

        Show
        Mircea Toma added a comment - Assign ID to the element that needs to be focused, but do it programmatically to ensure that the DOM updates for the component propagates to the root of its markup.
        Hide
        Carmen Cristurean added a comment -

        ICEfaces4 trunk r.48505: verified pushButton and linkButton using the QA test apps > Dynamic Attribute tests, on IE11, Chrome48, FF41.

        Show
        Carmen Cristurean added a comment - ICEfaces4 trunk r.48505: verified pushButton and linkButton using the QA test apps > Dynamic Attribute tests, on IE11, Chrome48, FF41.
        Hide
        Carmen Cristurean added a comment -

        Re-opening to add support for ace:radioButtons and ace:checkboxButtons.

        Show
        Carmen Cristurean added a comment - Re-opening to add support for ace:radioButtons and ace:checkboxButtons.
        Hide
        Mircea Toma added a comment -

        Added focus support for ace:radioButtons and ace:checkboxButtons.

        Show
        Mircea Toma added a comment - Added focus support for ace:radioButtons and ace:checkboxButtons .

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: