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
              Ted Goddard created issue -
              Ted Goddard made changes -
              Field Original Value New Value
              Assignee Neil Griffin [ ngriffin7a ]
              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).
              Neil Griffin made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              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.
              Neil Griffin made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Neil Griffin [ ngriffin7a ] Deryk Sinotte [ deryk.sinotte ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22228 Wed Aug 25 08:34:41 MDT 2010 deryk.sinotte ICE-5973: made path for graphicImage absolute
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/compat/samples/component-showcase/web/WEB-INF/includes/examples/custom/menuPopup.xhtml
              Deryk Sinotte made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Fix Version/s 2.0-Beta2 [ 10242 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10230 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: