ICEfaces
  1. ICEfaces
  2. ICE-6500

The h:message component rendering causes a full page reload.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha2
          • Fix Version/s: 2.0.1
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces 2 JSF 2
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Enclose <h:message> in a < h:panelGroup>.
            Show
            Enclose <h:message> in a < h:panelGroup>.

            Description

            The MessageRenderer, responsible for rendering the h:message component, does not render an ID around the dynamically generated text. It's also not clear if any containing element is rendered even if the message is empty (the containing element is required by ICEfaces for Ajax update).

            1. screenshot-01.png
              229 kB
            2. screenshot-02.png
              172 kB
            3. screenshot-03.png
              172 kB

              Activity

              Ascending order - Click to sort in descending order
              4 older comments
              Hide
              Permalink
              yip.ng added a comment -

              Problem reproducible in ACE showcase. See screenshot 2. But the rendering isn't done by com.icesoft.faces.renderkit.dom_html_basic.MessageRenderer. Is it done by com.sun.faces.renderkit.html_basic.MessageRenderer? How are we going to modify that? Change the renderer mapping for <h:message> and create a new class for it? How to do that?

              Show
              yip.ng added a comment - Problem reproducible in ACE showcase. See screenshot 2. But the rendering isn't done by com.icesoft.faces.renderkit.dom_html_basic.MessageRenderer. Is it done by com.sun.faces.renderkit.html_basic.MessageRenderer? How are we going to modify that? Change the renderer mapping for <h:message> and create a new class for it? How to do that?
              Hide
              Permalink
              Ken Fyten added a comment -

              Yip, looks like http://jira.icefaces.org/browse/ICE-5716 was the original fix for h:messages component. The task now is to take that same fix and implement one for h:message as well.

              Show
              Ken Fyten added a comment - Yip, looks like http://jira.icefaces.org/browse/ICE-5716 was the original fix for h:messages component. The task now is to take that same fix and implement one for h:message as well.
              Hide
              Permalink
              yip.ng added a comment -

              Changes in faces-config for ICE-5716 gone. Need to track down another way of mapping renderer to component.

              Show
              yip.ng added a comment - Changes in faces-config for ICE-5716 gone. Need to track down another way of mapping renderer to component.
              Hide
              Permalink
              yip.ng added a comment - - edited

              Mapping in ICE-5716 was moved to DOMRenderKit.java in ICE-5868.

              Show
              yip.ng added a comment - - edited Mapping in ICE-5716 was moved to DOMRenderKit.java in ICE-5868 .
              Hide
              Permalink
              yip.ng added a comment - - edited

              Done. See screenshot 3.

              Revision: 23991

              Modified : /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/renderkit/DOMRenderKit.java
              Added : /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/renderkit/html_basic/MessageRenderer.java

              Show
              yip.ng added a comment - - edited Done. See screenshot 3. Revision: 23991 Modified : /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/renderkit/DOMRenderKit.java Added : /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/renderkit/html_basic/MessageRenderer.java

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Deryk Sinotte
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: