ICEfaces
  1. ICEfaces
  2. ICE-7283

Showcase templates should not have ids on non-component markup

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0.2
          • Fix Version/s: 3.1.0.RC1, 3.1
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            ICEfaces 2 Comp Suite
          • Assignee Priority:
            P2
          • Affects:
            Sample App./Tutorial

            Description

            The Comp Suite sample application uses non-JSF-component markup in the templates files where an id is provided as an attribute. For example, in main-template.xhtml:

                <span id="primary">
                ...
                    <ui:insert name="component-example"/>
                </span>

            This is deadly to portlet rendering because the ids cannot be namespaced and therefore cannot be guaranteed to be unique. While it may work in non-portlet applications, having ids that are not under JSF control could lead to other problems. The recommendation is to either:

            - Remove the ids from the various <span> and <div> container tags. If they are being used for CSS, then consider style and class attributes instead.
            - Switch to using JSF components like h:panelGroup so that the ids can be properly generated and namespaced by JSF and the PortletFaces Bridge.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #29753 Thu Jun 28 14:02:56 MDT 2012 carlo.guglielmin ICE-7283 - Replaced all HTML components that had IDs with their h: equivalent. Was all divs and spans that got moved to panelGroups.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/showcase.xhtml
              Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/templates/main-template.xhtml
              Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/showcase-nonredirect.xhtml

                People

                • Assignee:
                  Carlo Guglielmin
                  Reporter:
                  Deryk Sinotte
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: