ICEmobile
  1. ICEmobile
  2. MOBI-1152

mobi:tabSet - Tab content height not changing for other tabs

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE 1.3.1.GA
    • Fix Version/s: EE 1.3.1.GA_P04
    • Component/s: Faces
    • Labels:
      None
    • Environment:
      All, Desktop

      Description

      With the mobi:tabSet the height used to show the content of the tab is set to what the first tabs content is. So if the first tab has only 19px; of content, the height for all of the tabs gets set to this even if the content of other tabs is larger.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case that shows this issue.

        Steps:

        • Load welcomeICEfaces.jsf
        • The second tab should show an ace:gMap component. Once I change the height used by the tabs via the browser dev tools I can see the map content.
        Show
        Arran Mccullough added a comment - Attached test case that shows this issue. Steps: Load welcomeICEfaces.jsf The second tab should show an ace:gMap component. Once I change the height used by the tabs via the browser dev tools I can see the map content.
        Hide
        Judy Guglielmin added a comment -

        when I modify the markup from
        <mobi:tabSet id="stackOne" selectedId="#

        {testBean.selectedId}" autoheight="true">
        to
        <mobi:tabSet id="stackOne" selectedId="#{testBean.selectedId}

        " height="500px">
        then, I do see the gmap.
        If there is not a problem with this workaround, I would suggest it.
        Looking at the autoheight issue in the meantime....

        Show
        Judy Guglielmin added a comment - when I modify the markup from <mobi:tabSet id="stackOne" selectedId="# {testBean.selectedId}" autoheight="true"> to <mobi:tabSet id="stackOne" selectedId="#{testBean.selectedId} " height="500px"> then, I do see the gmap. If there is not a problem with this workaround, I would suggest it. Looking at the autoheight issue in the meantime....
        Hide
        Judy Guglielmin added a comment -

        rev 48528 autoHeight now takes into account the style value for height on any div when calculating height. Does not accumulate values, just used a max value on any div within the tabcontents. This is done on a timeout when the page loads (not done when tabs are changing), so if a facelet="true" and using autoHeight, may want to place this in showContents (or user should be ware and use facelet="false" if this is important. Can always be overridden by the"height" attribute on the <mobi:tabSet> component.

        Show
        Judy Guglielmin added a comment - rev 48528 autoHeight now takes into account the style value for height on any div when calculating height. Does not accumulate values, just used a max value on any div within the tabcontents. This is done on a timeout when the page loads (not done when tabs are changing), so if a facelet="true" and using autoHeight, may want to place this in showContents (or user should be ware and use facelet="false" if this is important. Can always be overridden by the"height" attribute on the <mobi:tabSet> component.
        Hide
        Judy Guglielmin added a comment -

        test case showing height attribute for <mobi:tabSet> being used with gmap as well as autoHeight parameter also working by using the height style value used on inner div of content.

        Show
        Judy Guglielmin added a comment - test case showing height attribute for <mobi:tabSet> being used with gmap as well as autoHeight parameter also working by using the height style value used on inner div of content.

          People

          • Assignee:
            Judy Guglielmin
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: