ICEfaces
  1. ICEfaces
  2. ICE-9952

Styling of showcase examples problematic in Liferay 6.2

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P01, 4.0.BETA
    • Fix Version/s: EE-3.3.0.GA_P02, 4.0
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      Liferay 6.2
    • Assignee Priority:
      P1

      Description

      While there have always been some relatively minor styling conflicts between our showcase portlet examples and Liferay, the current version (Liferay 6.2 and ICEfaces 4.0 Beta) is significant.

        Activity

        Hide
        Deryk Sinotte added a comment -

        Putting all portlets in a single column prevents most of the styling issues but, as seen in the screen snapshot, once multiple columns are enabled, the layout and styling makes the portlets nearly impossible to use.

        Show
        Deryk Sinotte added a comment - Putting all portlets in a single column prevents most of the styling issues but, as seen in the screen snapshot, once multiple columns are enabled, the layout and styling makes the portlets nearly impossible to use.
        Hide
        Arturo Zambrano added a comment - - edited

        Are there any CSS style class naming specifications for portlet applications? I ask this because the main issue (examples having more width than they should) can simply be fixed by changing a style class name in our component showcase facelets. In main-template.xhtml we only need to change the 'container' class name to something else, like 'ice-container' (and of course change the name in the CSS file as well). It seems that Liferay 6.2 is using a class named 'container' as well for their UI, which happens to be the same name we're using. So, if there's no requirement to name CSS style classes in a certain way we can simply change this class name.

        Show
        Arturo Zambrano added a comment - - edited Are there any CSS style class naming specifications for portlet applications? I ask this because the main issue (examples having more width than they should) can simply be fixed by changing a style class name in our component showcase facelets. In main-template.xhtml we only need to change the 'container' class name to something else, like 'ice-container' (and of course change the name in the CSS file as well). It seems that Liferay 6.2 is using a class named 'container' as well for their UI, which happens to be the same name we're using. So, if there's no requirement to name CSS style classes in a certain way we can simply change this class name.
        Hide
        Deryk Sinotte added a comment -

        The Portlet specifications (http://download.oracle.com/otn-pub/jcp/portlet-2.0-edr-oth-JSpec/portlet-2_0-edr.pdf) do outline a few CSS style naming guidelines but they are fairly basic and all names start with "portlet-" so in this case, container should be find to change.

        Show
        Deryk Sinotte added a comment - The Portlet specifications ( http://download.oracle.com/otn-pub/jcp/portlet-2.0-edr-oth-JSpec/portlet-2_0-edr.pdf ) do outline a few CSS style naming guidelines but they are fairly basic and all names start with "portlet-" so in this case, container should be find to change.
        Hide
        Arturo Zambrano added a comment -

        Committed fix to 4.0 trunk at revision 41223 and to 3.3 EE maintenance branch at revision 41222. Renamed 'container' CSS class name to 'ice-container' in the portlet showcase, in order to avoid conflicts with Liferay 6.2.

        Show
        Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 41223 and to 3.3 EE maintenance branch at revision 41222. Renamed 'container' CSS class name to 'ice-container' in the portlet showcase, in order to avoid conflicts with Liferay 6.2.
        Hide
        Deryk Sinotte added a comment -

        Confirm that this looks much better now and that portlets are usable in multicolumn layouts.

        Show
        Deryk Sinotte added a comment - Confirm that this looks much better now and that portlets are usable in multicolumn layouts.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: