Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: EE-1.8.2.GA_P07, EE-3.3.0.GA_P02
-
Fix Version/s: EE-3.3.0.GA_P03, EE-1.8.2.GA_P08
-
Component/s: ICE-Components
-
Labels:None
-
Environment:All
-
Assignee Priority:P2
-
Support Case References:Support Case #13081 - https://icesoft.my.salesforce.com/5007000000otjJS
Description
When the ice:panelCollapsible header is first rendered, it renders an internal div with a padding set to 0px. When this panel is shown/expanded, this div gets an inline styling that uses a padding: 1px;. In more complex apps this simple change can cause major changes to the layout of other components.
Either this added padding needs to be removed or it needs to be set when the component is first rendered to prevent the changes in layout.
Either this added padding needs to be removed or it needs to be set when the component is first rendered to prevent the changes in layout.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Change padding to 0px in the client code just as the renderer does.
Show
Mircea Toma
added a comment - Change padding to 0px in the client code just as the renderer does.
Verified using component-showcase and ICEfaces 1.8.2 r43383. Tomcat 7, IE 11, Chrome 38, FF 33 DOM inspections.
Show
Liana Munroe
added a comment - Verified using component-showcase and ICEfaces 1.8.2 r43383. Tomcat 7, IE 11, Chrome 38, FF 33 DOM inspections.
It looks like the change mentioned in
ICE-4674was implemented inICE-5308. The issue is still seen with this change though. When the panelCollapsible is rendered it renders the inner div with padding: 0px; Upon expand or contract, this div's padding is changed to 1px;Looking at the update returned, the div has padding: 0px; still set, so this additional padding is being set from something else. I did a quick test and setting padding: 1px; in the PanelCollapsibleRendered.java does keep the padding consistent but this might not be best for backwards compatibility.
This issue can be seen on the show case demo's as well. Load the panel collapsible demo and inspect the header div code. Expand or contract the panel and inspecting the same div shows the change in padding.