ICEfaces
  1. ICEfaces
  2. ICE-2227

scrollHeight, scrollWidth, and columnWidths not working as expected

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: 1.7DR#1
    • Fix Version/s: 1.6.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      'scrollHeight' and 'scrollWidth' in <ice:panelGroup>, and 'scrollHeight' and 'columnWidths' in <ice:dataTable> do not accomplish their purpose of modifying the dimensions of these components and making them scrollable. The problem is just that the CSS rules aren't rendered with units (px or em), which causes the browsers to ignore them. This is how scrollHeight="50" in <ice:panelGroup> is currently being rendered:

      <div class="icePnlGrp" id="_id1:_id2" style="height:50;overflow:auto;">

      This is how it should be rendered:

      <div class="icePnlGrp" id="_id1:_id2" style="height:50px;overflow:auto;">

        Activity

        Hide
        Adnan Durrani added a comment -

        We don't hard code any of the measurement unit to the rendererd HTML, as it can be changed based on the design. This attribute requires to the developer to specify a measurement unit along with the width and height (e.g.)

        <ice:panelGroup scrollHeight="50em" scrollWidth="20px" >

        I have added the comments to the TLD.

        branch-1.6 = revision: 15014
        trunk = revision: 15015

        Show
        Adnan Durrani added a comment - We don't hard code any of the measurement unit to the rendererd HTML, as it can be changed based on the design. This attribute requires to the developer to specify a measurement unit along with the width and height (e.g.) <ice:panelGroup scrollHeight="50em" scrollWidth="20px" > I have added the comments to the TLD. branch-1.6 = revision: 15014 trunk = revision: 15015

          People

          • Assignee:
            Unassigned
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: