ICEfaces
  1. ICEfaces
  2. ICE-5062

ice:panelPopup header is rendered as a HTML Table with two columns <td> but it is not applied corectly custom style class

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2
          • Fix Version/s: 1.8.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            <ice:panelPopup styleClass="myPnlPop" >
            - in ICEfaces 1.8.1 was rendered as
                <table>
                    <tr>
                        <td class='icePnlPopHdr myPnlPopHdr'>
                        ....
                        </td>
                    </tr>
                </table>

            - in ICEfaces 1.8.2 si rendered with two <td> elements
                  <table>
                    <tr>
                        <td class='icePnlPopHdr myPnlPopHdr'>
                           ............
                        </td>
                        <td class='icePnlPopHdr'>
                            <div style="width: 1px;"/>
                        </td>
                    </tr>
                </table>
                
            Why the custom style "myPnlPopHdr" is not applied to the element number two <td> ?

              Activity

              Ascending order - Click to sort in descending order
              Calin Cosmin created issue -
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #19578 Mon Nov 02 14:24:44 MST 2009 yip.ng ICE-5062,ICE-4700: Added missing style class for column header.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelpopup/PanelPopupRenderer.java
              Hide
              Permalink
              yip.ng added a comment - - edited

              Extra column added in ICE-4700.

              Show
              yip.ng added a comment - - edited Extra column added in ICE-4700 .
              Hide
              Permalink
              yip.ng added a comment -

              Missing style class added.

              Show
              yip.ng added a comment - Missing style class added.
              yip.ng made changes -
              Field Original Value New Value
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 1.8.3 [ 10211 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #19949 Thu Dec 10 15:45:41 MST 2009 judy.guglielmin ICE-4481, update PanelPopup for ICE-5062, ICE-4778, ICE-4781, ICE-4700, ICE-4710, ICE-4578, ICE-4595, ICE-4475, ICE-3356, ICE-4495,ICE-4450
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/panelpopup/PanelPopup.java
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/component-metadata/src/main/resources/conf/ice_cust_properties/cust-panelpopup-props.xml
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/panelpopup/PanelPopupRenderer.java
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/outputconnectionstatus/OutputConnectionStatusRenderer.java
              Hide
              Permalink
              Mike Geier added a comment -

              What exactly was the fix?

              I have this similar problem and found a work-around for IE8, but it doesn't work in IE7 or lower. My problem is that I override the styles to hide the border and create rounded popup. It worked fine in 1.8.1 but not in 1.8.2. See attached.

              Can I get the patched file(s)? My work-around was to add the following to the .css file:
              /* IceFaces 1.8.2 added a 1px empty cell with a div, need to hide this */
              .icePnlPopHdr div

              { display:none; }
              Show
              Mike Geier added a comment - What exactly was the fix? I have this similar problem and found a work-around for IE8, but it doesn't work in IE7 or lower. My problem is that I override the styles to hide the border and create rounded popup. It worked fine in 1.8.1 but not in 1.8.2. See attached. Can I get the patched file(s)? My work-around was to add the following to the .css file: /* IceFaces 1.8.2 added a 1px empty cell with a div, need to hide this */ .icePnlPopHdr div { display:none; }
              Hide
              Permalink
              Mike Geier added a comment -

              Extra column in popup header

              Show
              Mike Geier added a comment - Extra column in popup header
              Mike Geier made changes -
              Attachment popup.gif [ 12130 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Calin Cosmin
                • Votes:
                  2 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: