ICEfaces
  1. ICEfaces
  2. ICE-8000

PanelPositioned renders a visible xml comment containing a random number

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P04
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Chrome, IE

      Description

      The PanelPositioned component renders a visible xml comment containing a random number. The rendered comment only appears visible on some browsers, such as Chrome and IE, but not others, such as Firefox. The xml comment is being rendered in the following section of PanelPositionedRenderer:

                   if (isChanged(facesContext)) {
                          // Force the re rendering of the entire component. This is due to a strange quick with positioned
                          // panel. When an element is moved in the same list then it container element moves with it
                          // When the update occurs the elements are replaced but because the containers have moved
                          // then the result looks the same. (But a refresh shows otherwise)
                          Node node = domContext.createTextNode(
                                  "<!-- " + (new Random().nextInt(1000)) + "-->");
                          root.appendChild(node);
                      }

        Issue Links

          Activity

          Hide
          Philip Breau added a comment -

          screen shot showing the ee comps richTabs (which uses the panelPositioned) rendering the visible xml comment

          Show
          Philip Breau added a comment - screen shot showing the ee comps richTabs (which uses the panelPositioned) rendering the visible xml comment
          Hide
          Philip Breau added a comment -

          blocks ipck-396

          Show
          Philip Breau added a comment - blocks ipck-396
          Hide
          Philip Breau added a comment -

          blocks ipck-396

          Show
          Philip Breau added a comment - blocks ipck-396
          Hide
          Deryk Sinotte added a comment -

          Assigning to Art to take a look at.

          Show
          Deryk Sinotte added a comment - Assigning to Art to take a look at.
          Hide
          Deryk Sinotte added a comment -

          Just wanted to mention that I see this happening on the portlet version also.

          Show
          Deryk Sinotte added a comment - Just wanted to mention that I see this happening on the portlet version also.
          Hide
          Arturo Zambrano added a comment -

          Committed fix at revision 28650.

          Just needed to create an unescaped text node for the XMLcomment.

          Show
          Arturo Zambrano added a comment - Committed fix at revision 28650. Just needed to create an unescaped text node for the XMLcomment.

            People

            • Assignee:
              Arturo Zambrano
              Reporter:
              Philip Breau
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: