ICEfaces
  1. ICEfaces
  2. ICE-5973

menupopup sample image not displayed in portlet

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Beta1
          • Fix Version/s: 2.0-Beta2, 2.0.0
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            ICEfaces 2.0, Liferay 6.0.4

            Description


            menuPopup sample contains the following for inserting the image:

                        <ice:graphicImage value="./images/icefaces-menupop.png"
                            alt="ICEfaces" title="ICEfaces" width="352" height="122"/>

            This is likely not the correct way to resolve an image resource in a JSF 2.0 Portlet.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Neil Griffin added a comment -

              It's fine to have a portlet request an image resource directly from the context, like "./images/icefaces-menupop.png"

              But I think the reason why it's not rendering is because of the relative "." preceding the path. A long time ago, I think the behavior of ice:graphicImage was modified so that if a leading "/" is found in the attribute value, then the full context path will be rendered, which helps images render properly in Liferay at least. (not sure about other portals).

              Show
              Neil Griffin added a comment - It's fine to have a portlet request an image resource directly from the context, like "./images/icefaces-menupop.png" But I think the reason why it's not rendering is because of the relative "." preceding the path. A long time ago, I think the behavior of ice:graphicImage was modified so that if a leading "/" is found in the attribute value, then the full context path will be rendered, which helps images render properly in Liferay at least. (not sure about other portals).
              Hide
              Permalink
              Neil Griffin added a comment -

              Assigning to Deryk since he has commit rights to the trunk, so that the path can be fixed to have a leading slash.

              Show
              Neil Griffin added a comment - Assigning to Deryk since he has commit rights to the trunk, so that the path can be fixed to have a leading slash.

                People

                • Assignee:
                  Deryk Sinotte
                  Reporter:
                  Ted Goddard
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: