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
        Ken Fyten added a comment -

        Here's a video link that demonstrates some of the issues:

        http://screencast.com/t/x1nxb9N3

        Show
        Ken Fyten added a comment - Here's a video link that demonstrates some of the issues: http://screencast.com/t/x1nxb9N3
        Hide
        Ken Fyten added a comment -

        Please investigate, this seems to happen consistently on all browsers, makes me think it is a component issue.

        Show
        Ken Fyten added a comment - Please investigate, this seems to happen consistently on all browsers, makes me think it is a component issue.
        Hide
        Adnan Durrani added a comment -

        Command: Commit
        Modified: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\resources\org.icefaces.component.sliderentry\slider.js
        Sending content: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\resources\org.icefaces.component.sliderentry\slider.js
        Completed: At revision: 23300

        Show
        Adnan Durrani added a comment - Command: Commit Modified: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\resources\org.icefaces.component.sliderentry\slider.js Sending content: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\resources\org.icefaces.component.sliderentry\slider.js Completed: At revision: 23300
        Hide
        Mandeep Hayher added a comment - - edited

        icefaces2 revision# 23411
        server Tomcat6

        Chrome8 & safari5, FF3.6
        Issue1 fixed

        Issue2 clicking on the slider bar does not get focus on the slider component.

        Show
        Mandeep Hayher added a comment - - edited icefaces2 revision# 23411 server Tomcat6 Chrome8 & safari5, FF3.6 Issue1 fixed Issue2 clicking on the slider bar does not get focus on the slider component.
        Hide
        Ken Fyten added a comment -

        Need to look at: "Issue2 clicking on the slider bar does not get focus on the slider component. " Basically, if the slider doesn't have focus, and you click on the bar, it jumps to the clicked bar value, but doesn't also assign focus to the slider, which it should. Currently, you have to click on the arrow image for the focus to be set, but clicking on the bar should also work.

        Show
        Ken Fyten added a comment - Need to look at: "Issue2 clicking on the slider bar does not get focus on the slider component. " Basically, if the slider doesn't have focus, and you click on the bar, it jumps to the clicked bar value, but doesn't also assign focus to the slider, which it should. Currently, you have to click on the arrow image for the focus to be set, but clicking on the bar should also work.
        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: