ICEfaces
  1. ICEfaces
  2. ICE-6329

ACE: sliderEntry focus behaviour issues

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      IF 2, Adv. Comps, ace-showcase sample, FF 3.6 or Chrome/Safari.
    • Affects:
      Sample App./Tutorial

      Description

      When testing the sliderEntry demo in the ace-showcase app, some strange focus-mgmt issues are apparent:

      1. Try tabbing into the horiz. slider entry, then using the arrow keys to increase/decrease the slider value. SingleSubmit occurs, focus stays on slider, all good. Now tab onto the "Submit" button at bottom of demo, then Shift-Tab back up to the horiz. slider, use arrows to change value. Single-submit occurs, and focus is moved back to the Submit button! This will keep happening unless you focus on the inputText field above the horiz. slider, then back to the slider, then it's fine again. Is there something about the focus in the slider that makes it 2nd class element vs. the inputText and Submit button? It seems like the bridge's focus-retention feature is somehow sticking on the submit button once it's been focussed, until another native widget like the inputText is focussed.

      2. Use the mouse to click on the horiz. slider bar (not the arrow itself, the bar beside it), slider moves to new position and gets focus, now keyboard arrows will adjust slider position. All good. Then click on vertical slider bar (not arrow). Focus is not set on vertical slider after clicking on bar for some reason. Not good.

      Happens the same way on Safari 5, Chrome, and FF 3.6 ( all I tested).




        Activity

        Hide
        yip.ng added a comment - - edited

        Clicking too close to the bar doesn't get focus. (This is true even for the horizontal slider, not just the vertical slider as implied in the description.) But if you click a little further away from the bar (on the bottom for horizontal slider and on the right for vertical slider), you will get focus.

        Show
        yip.ng added a comment - - edited Clicking too close to the bar doesn't get focus. (This is true even for the horizontal slider, not just the vertical slider as implied in the description.) But if you click a little further away from the bar (on the bottom for horizontal slider and on the right for vertical slider), you will get focus.
        Hide
        yip.ng added a comment - - edited

        In YUI 3 examples, clicking anywhere won't get us focus at all. Therefore, focusing may have been done by us, not YUI.

        Show
        yip.ng added a comment - - edited In YUI 3 examples, clicking anywhere won't get us focus at all. Therefore, focusing may have been done by us, not YUI.
        Hide
        yip.ng added a comment - - edited

        There is already code (by us, not YUI) to do the focusing on the click event, but somehow this only works when you click not too close to the bar.

        Show
        yip.ng added a comment - - edited There is already code (by us, not YUI) to do the focusing on the click event, but somehow this only works when you click not too close to the bar.
        Hide
        yip.ng added a comment -

        Tried other means of listening to click event. No luck. Need to look deeper into YUI source code.

        Show
        yip.ng added a comment - Tried other means of listening to click event. No luck. Need to look deeper into YUI source code.
        Hide
        yip.ng added a comment -

        Fixed.

        Revision: 23825


        Modified : /icefaces2/trunk/icefaces/ace/component/resources/org.icefaces.component.sliderentry/slider.js

        Show
        yip.ng added a comment - Fixed. Revision: 23825 Modified : /icefaces2/trunk/icefaces/ace/component/resources/org.icefaces.component.sliderentry/slider.js

          People

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

            Dates

            • Created:
              Updated:
              Resolved: