ICEfaces
  1. ICEfaces
  2. ICE-6451

Add scrollbar to list of selections for selectInputText

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.8.2-EE-GA_P02
    • Fix Version/s: None
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

      Description

      It would be useful to have the selectInputText component render a scroll bar when displaying items. At the moment it is possible to restrict the number of matches made. If there are more matches than what the rows will show then these matches aren't shown. Having this implemented will allow the matches to be displayed in a set section but also be able to show all options.

      Suggest adding a new attribute called "scrollable=true/false" which would display a vertical scrollbar inside the popup list if there were more possible matches available that would fit in the "rows"-specified number of rows in the popup.

        Issue Links

          Activity

          Hide
          yip.ng added a comment - - edited

          Two approaches:

          1. Specify height of div in theme or by user.

          2. Dynamically calculate height from rows attribute value. Inaccurate even if possible, depending on how the rows are styled with padding, border, margin, etc.

          What if list is less than height? Blank space at bottom OK?

          Show
          yip.ng added a comment - - edited Two approaches: 1. Specify height of div in theme or by user. 2. Dynamically calculate height from rows attribute value. Inaccurate even if possible, depending on how the rows are styled with padding, border, margin, etc. What if list is less than height? Blank space at bottom OK?
          Hide
          yip.ng added a comment - - edited

          Prototyped with fixed height and overflow:auto for the list.

          OK in FF4 (see screenshot 1), except that div won't scroll with arrow keys.

          Problems in IE9:

          • no response to arrow keys at all, i.e. highlight doesn't appear at all
          • sometimes JS error triggered, see screenshot 2
          • after scrolling with mouse, the div just disappears altogether
          Show
          yip.ng added a comment - - edited Prototyped with fixed height and overflow:auto for the list. OK in FF4 (see screenshot 1), except that div won't scroll with arrow keys. Problems in IE9: no response to arrow keys at all, i.e. highlight doesn't appear at all sometimes JS error triggered, see screenshot 2 after scrolling with mouse, the div just disappears altogether
          Hide
          yip.ng added a comment - - edited

          In IE9: Problems 1 and 2 occur even in the original showcase.

          In IE8: Only has problem 3, plus same problem as FF4 of div not scrolling with arrow keys.

          JS error in IE is from bridge\lib\prototype\dom.js.

          Show
          yip.ng added a comment - - edited In IE9: Problems 1 and 2 occur even in the original showcase. In IE8: Only has problem 3, plus same problem as FF4 of div not scrolling with arrow keys. JS error in IE is from bridge\lib\prototype\dom.js.
          Hide
          yip.ng added a comment - - edited

          IE9 problems 1 and 2 gone after upgrade of prototype.

          The problems that remain are:

          • div won't scroll with arrow keys (FF and IE) (because focus is never really in the div, it's just the highlight that moves )
          • div disappears on clicking scroll bar (IE)
          • calculation of height if we decide to calculate height dynamically
          Show
          yip.ng added a comment - - edited IE9 problems 1 and 2 gone after upgrade of prototype. The problems that remain are: div won't scroll with arrow keys (FF and IE) (because focus is never really in the div, it's just the highlight that moves ) div disappears on clicking scroll bar (IE) calculation of height if we decide to calculate height dynamically

            People

            • Assignee:
              Unassigned
              Reporter:
              Arran Mccullough
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: