ICEfaces
  1. ICEfaces
  2. ICE-6367

CLONE -add keyboardNavigationEnabled attribute to rowSelector component (Compat)

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-EE-Beta1
    • Fix Version/s: 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      keyboardNavigationEnabled

      Description

      add keyboardNavigationEnabled attribute to the rowSelector component which will be true by default and can be turn off by setting to false.

        Activity

        Hide
        Ken Fyten added a comment -

        Need to port this forward to the compat version.

        Show
        Ken Fyten added a comment - Need to port this forward to the compat version.
        Hide
        yip.ng added a comment -

        Cloned from ICE-5405.

        Show
        yip.ng added a comment - Cloned from ICE-5405 .
        Hide
        yip.ng added a comment -

        svn\ossrepo\icefaces2\trunk\icefaces\compat\component-metadata\src\main\resources\conf\ice_cust_properties\cust-rowselector-props.xml

        Changes already in svn.

        Show
        yip.ng added a comment - svn\ossrepo\icefaces2\trunk\icefaces\compat\component-metadata\src\main\resources\conf\ice_cust_properties\cust-rowselector-props.xml Changes already in svn.
        Hide
        yip.ng added a comment -

        svn\ossrepo\icefaces2\trunk\icefaces\compat\components\src\main\java\com\icesoft\faces\component\ext\RowSelector.java

        Changes already in svn.

        Show
        yip.ng added a comment - svn\ossrepo\icefaces2\trunk\icefaces\compat\components\src\main\java\com\icesoft\faces\component\ext\RowSelector.java Changes already in svn.
        Hide
        yip.ng added a comment -

        svn\ossrepo\icefaces2\trunk\icefaces\compat\components\src\main\java\com\icesoft\faces\component\ext\renderkit\TableRenderer.java

        Changes already in svn.

        Show
        yip.ng added a comment - svn\ossrepo\icefaces2\trunk\icefaces\compat\components\src\main\java\com\icesoft\faces\component\ext\renderkit\TableRenderer.java Changes already in svn.
        Hide
        Mandeep Hayher added a comment -

        Icefaces2 revision# 23643
        Server: Tomcat6
        Tested successfully on FF3.6.

        Focus is lost once a row is selected on IE8 browser.

        Can not tab in to row selection data-table on Opera10.60. Control + selection (using keyboard )does not work on Opera10.60. Also focus is often lost on Opera browser when using multiple selection by keyboard.

        Show
        Mandeep Hayher added a comment - Icefaces2 revision# 23643 Server: Tomcat6 Tested successfully on FF3.6. Focus is lost once a row is selected on IE8 browser. Can not tab in to row selection data-table on Opera10.60. Control + selection (using keyboard )does not work on Opera10.60. Also focus is often lost on Opera browser when using multiple selection by keyboard.
        Hide
        yip.ng added a comment -

        Focus is lost once a row is selected on IE8 browser: can't reproduce. See video at http://screencast.com/t/9WQkpfdQO.

        Show
        yip.ng added a comment - Focus is lost once a row is selected on IE8 browser: can't reproduce. See video at http://screencast.com/t/9WQkpfdQO .
        Hide
        yip.ng added a comment -

        Opera problems same in icefaces trunk. Therefore they are not compat regressions.

        Show
        yip.ng added a comment - Opera problems same in icefaces trunk. Therefore they are not compat regressions.
        Hide
        yip.ng added a comment - - edited

        Each row has a hidden link to simulate tabbing and focusing. In Opera you can't tab to links. Read this: http://stackoverflow.com/questions/2292016/is-there-a-workaround-to-safaris-operas-bug-that-you-cant-tab-through-hyperlin.

        And compare the tabbing on the Opera Portal page in Firefox and Opera itself: http://portal.opera.com/?utm_source=OperaBrowser&utm_medium=HomeButton&utm_campaign=PortalLinks.

        Show
        yip.ng added a comment - - edited Each row has a hidden link to simulate tabbing and focusing. In Opera you can't tab to links. Read this: http://stackoverflow.com/questions/2292016/is-there-a-workaround-to-safaris-operas-bug-that-you-cant-tab-through-hyperlin . And compare the tabbing on the Opera Portal page in Firefox and Opera itself: http://portal.opera.com/?utm_source=OperaBrowser&utm_medium=HomeButton&utm_campaign=PortalLinks .
        Hide
        yip.ng added a comment - - edited

        Ctrl + Space and Ctrl + Enter are Opera keyboard shortcuts. Therefore they don't work for selection.

        http://help.opera.com/Windows/11.00/en/keyboard.html

        Show
        yip.ng added a comment - - edited Ctrl + Space and Ctrl + Enter are Opera keyboard shortcuts. Therefore they don't work for selection. http://help.opera.com/Windows/11.00/en/keyboard.html
        Hide
        yip.ng added a comment -

        Given these Opera interference and deficiencies, we shouldn't expect keyboard navigation and selection to work at all.

        Show
        yip.ng added a comment - Given these Opera interference and deficiencies, we shouldn't expect keyboard navigation and selection to work at all.
        Hide
        Ken Fyten added a comment -

        Added Opera specific aspects to ICE-6395.

        Show
        Ken Fyten added a comment - Added Opera specific aspects to ICE-6395.

          People

          • Assignee:
            yip.ng
            Reporter:
            Adnan Durrani
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: