ICEfaces
  1. ICEfaces
  2. ICE-1519

OutputChartRenderer should add width and height attributes to the image element.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.6DR#3
    • Fix Version/s: 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Tested environment: Linux, Liferay/Tomcat 4.3, Firefox 2.0

      Description

      Using a simple ICEfaces Chart Portlet the actual Portlet space given by the Portal seems to be smaller than the image contained inside the Portlet. Adding the width and height attributes with the correct values (I hardcoded it to test it) seems to solve this problem.

        Activity

        Hide
        Jack Van Ooststroom added a comment -

        Initially the width and height attributes were being set on the img element of the inputChart component. However, when this image was dynamically being changed it showed some "flickering" when using Internet Explorer. It was decided to move the width and height attributes to the td element instead to resolve this IE-specific issue.

        To solve the Portlet-related issue the width and height attributes are now being set on both the img and td elements.

        Show
        Jack Van Ooststroom added a comment - Initially the width and height attributes were being set on the img element of the inputChart component. However, when this image was dynamically being changed it showed some "flickering" when using Internet Explorer. It was decided to move the width and height attributes to the td element instead to resolve this IE-specific issue. To solve the Portlet-related issue the width and height attributes are now being set on both the img and td elements.
        Hide
        Jack Van Ooststroom added a comment -

        The original fix was backed out. We should have a quick look if this is still an issue with ICEfaces 1.7-DR#3. Leaving the Fix Version at 1.7.

        Show
        Jack Van Ooststroom added a comment - The original fix was backed out. We should have a quick look if this is still an issue with ICEfaces 1.7-DR#3. Leaving the Fix Version at 1.7.
        Hide
        Ken Fyten added a comment -

        This turns out to not be required.

        Show
        Ken Fyten added a comment - This turns out to not be required.

          People

          • Assignee:
            Jack Van Ooststroom
            Reporter:
            Jack Van Ooststroom
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: