ICEfaces
  1. ICEfaces
  2. ICE-2233

<ice:panelTab/> label property doesn't render properly under XP theme in IE within Liferay

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7DR#3, 1.7
    • Component/s: None
    • Labels:
      None
    • Environment:
      Tomcat 6.0.13 / Liferay 4.3.3

      Description

      This only happens under IE, and not under FF.

      It has to do with this setting in xp-portlet.css:

      .icePnlTbOn .MidMid a {
          display: block;
      }

      Because when I change it to "inline" the problem goes away, but then the padding isn't right.

      In order to reproduce, download the sample-icefaces-sun-portlet from the Liferay website, and edit the JobApplication.jspx file, so that there are two panelTabs instead of only one.

      Then, when you place the portlet on a Liferay portal page, and then click back and forth between tabs, you will see the labels of the tabs shifted way to the right, and getting clipped along the right hand side.
      1. screenshot-1.jpg
        13 kB
      2. screenshot-2.jpg
        111 kB
      3. screenshot-3.jpg
        96 kB

        Activity

        Hide
        Neil Griffin added a comment -

        Oops, sorry about that.

        I'm running IE7. Also, please clear your browser cache just in case.

        Thanks,

        Neil

        Show
        Neil Griffin added a comment - Oops, sorry about that. I'm running IE7. Also, please clear your browser cache just in case. Thanks, Neil
        Hide
        Neil Griffin added a comment -

        If it can't be reproduced with ICEfaces 1.6 (bundled with the sample portlet), then please replace the ICEFaces JARs in WEB-INF/lib with the ones in the trunk.

        I'm working primary with the trunk right now on a real-world application, trying to put it to the test.

        Show
        Neil Griffin added a comment - If it can't be reproduced with ICEfaces 1.6 (bundled with the sample portlet), then please replace the ICEFaces JARs in WEB-INF/lib with the ones in the trunk. I'm working primary with the trunk right now on a real-world application, trying to put it to the test.
        Hide
        yip.ng added a comment -

        OK. I have installed IE7 and can reproduce the bug now with the downloaded sample app.

        Show
        yip.ng added a comment - OK. I have installed IE7 and can reproduce the bug now with the downloaded sample app.
        Hide
        yip.ng added a comment -

        Testing the fix. Both tabs have been clicked on many times.

        Show
        yip.ng added a comment - Testing the fix. Both tabs have been clicked on many times.
        Hide
        Neil Griffin added a comment -

        Tested and confirmed fixed. Recommend closing it out.

        Show
        Neil Griffin added a comment - Tested and confirmed fixed. Recommend closing it out.

          People

          • Assignee:
            Unassigned
            Reporter:
            Neil Griffin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: