ICEfaces
  1. ICEfaces
  2. ICE-4059

Inconsistent/improper styling of panelCollapsible component when disabled=true

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2 SP1b, 1.8DR#2
    • Fix Version/s: 1.8RC2, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Styles

      Description

      Setting disabled=true on a panelCollapsible yields unpredictable and incorrect results depending on the style used. The original forum post has screen snapshots showing that;

      Rime: Appears to apply NO style to disabled panels
      XP: Applies a consistent visual style BUT no visual cue as to its disabled state
      Royale: Applies a consistent visual style and applies a visual cue to the disabled state BUT does not appear to change the cursor appropriately.

      The forum poster identified this problem under 1.7 but I was able to replicate it using 1.8.

        Activity

        Deryk Sinotte created issue -
        Deryk Sinotte made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 1.8RC1 [ 10143 ]
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] Arturo Zambrano [ artzambrano ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18335 Wed Feb 11 16:33:40 MST 2009 art.zambrano ICE-4059 Inconsistent/improper styling of panelCollapsible component when disabled=true
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp-portlet.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/royale/royale.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime-portlet.css
        Hide
        Arturo Zambrano added a comment -

        fixed at revision 18335

        Show
        Arturo Zambrano added a comment - fixed at revision 18335
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ken Fyten added a comment -

        We should review the styling for Rime theme as it's currently a little lacking. Need to preserve the "bar" at the top somehow. I'll consult with design on this.

        Show
        Ken Fyten added a comment - We should review the styling for Rime theme as it's currently a little lacking. Need to preserve the "bar" at the top somehow. I'll consult with design on this.
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ] Ken Fyten [ ken.fyten ]
        Hide
        Mandeep Hayher added a comment -

        There also seems to be no difference in styling of enabled and disabled collapsible panel on IE browser for 'Royale' & 'XP' theme.

        Show
        Mandeep Hayher added a comment - There also seems to be no difference in styling of enabled and disabled collapsible panel on IE browser for 'Royale' & 'XP' theme.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Ken Fyten [ ken.fyten ] Arturo Zambrano [ artzambrano ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 1.8 [ 10161 ]
        Fix Version/s 1.8RC1 [ 10143 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18420 Tue Feb 24 13:46:28 MST 2009 art.zambrano Fix related to ICE-4059 (adjusted CSS styles for IE)
        Files Changed
        Commit graph ADD /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/css-images/table_col_header_dis.gif
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp-portlet.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css
        Commit graph ADD /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/royale/css-images/table_col_header_dis.gif
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/royale/royale.css
        Hide
        Arturo Zambrano added a comment -

        Fixed at revision 18420

        Opacity just won't work properly in IE in conjunction with our current markup and other CSS rules. Added new background images for the disabled classes instead.

        Show
        Arturo Zambrano added a comment - Fixed at revision 18420 Opacity just won't work properly in IE in conjunction with our current markup and other CSS rules. Added new background images for the disabled classes instead.
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #18428 Wed Feb 25 11:25:27 MST 2009 art.zambrano Fix related to ICE-4059 (fixed panelCollapsible styles in rime)
        Files Changed
        Commit graph ADD /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/css-images/table_col_header_dis.gif
        Commit graph ADD /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/css-images/PnlHdr_down_dis.gif
        Commit graph ADD /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/css-images/PnlHdr_collapsed_dis.gif
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime.css
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime-portlet.css
        Ken Fyten made changes -
        Fix Version/s 1.8RC2 [ 10163 ]
        Fix Version/s 1.8 [ 10161 ]
        Ken Fyten made changes -
        Fix Version/s 1.8 [ 10161 ]
        Assignee Priority P1
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Arturo Zambrano [ artzambrano ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Deryk Sinotte
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: