ICEfaces
  1. ICEfaces
  2. ICE-5309

ice:panelTab - title visible with decoration in IE6

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2
          • Fix Version/s: 1.8.2-EE-GA_P01, 1.8.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            ICefaces from trunk
          • Workaround Description:
            Changes of xp.css from revisions 19005 and 18990 can be reverted.

            Description

            With revision 19005 some modifications of xp.css was created. Some of them was not backward compatibile.
            After update titles of panel tab's apperas with decoration's typical for 'A" hyperlink elements.

            Was css modifications really necessary?

            In my opinion css changes from revisions 18990 and 19005 can be reverted because I see some problems
            - without this changes label on tab works well
            - syntax modification is not backward compatibile, tabs are not correctly displayed in IE6
              instead
                  .icePnlTbOn .MidMid>a, .icePnlTbOn .MidMid>div {
              can be old
                  .icePnlTbOn .MidMid a, .icePnlTbOn .MidMid div {
            - css syntax like
                  .icePnlTbOn .MidMid a, .icePnlTbOn .MidMid div {
               increase twice size of tab area because div and a got the same big padding
                 <a class="icePnlTbLblLnk">
                        <div><table cellspacing="0" cellpadding="0"><tbody><tr><td>Tab title</td></tr></tbody></table></div></a>

            Finnaly xp.css from revision 18990 was better than 19005.
            I don't see any problem's with xp.css without these changes.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment -

              rime.css had the same changes at rev. 19005 but was later reverted at rev. 19104 (ICE-4737). Somehow xp.css was missed in the reversion.

              Reversion applied similar to rime.css. (xp-portlet.css was never changed in the first place.)

              Revision: 20899

              Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css

              Revision: 20900

              Modified : /icefaces-ee/branches/icefaces-ee-1.8.2/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css

              Show
              yip.ng added a comment - rime.css had the same changes at rev. 19005 but was later reverted at rev. 19104 (ICE-4737). Somehow xp.css was missed in the reversion. Reversion applied similar to rime.css. (xp-portlet.css was never changed in the first place.) Revision: 20899 Modified : /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css Revision: 20900 Modified : /icefaces-ee/branches/icefaces-ee-1.8.2/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css
              Hide
              Permalink
              Joanne Bai added a comment -

              On ICEfaces-1.8.2-ee revision 20900, following .icePnlTbOn style is observed in xp.css

              .icePnlTbOn .MidMid a, .icePnlTbOn .MidMid div {

              Close it.

              Show
              Joanne Bai added a comment - On ICEfaces-1.8.2-ee revision 20900, following .icePnlTbOn style is observed in xp.css .icePnlTbOn .MidMid a, .icePnlTbOn .MidMid div { Close it.

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Krashan Brahmanjara
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: