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

              Ascending order - Click to sort in descending order
              Jack Van Ooststroom created issue -
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #13718 Wed May 02 14:34:22 MDT 2007 jack.van.ooststroom Fix for ICE-1519: OutputChartRenderer should add width and height attributes to the image element.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/outputchart/OutputChartRenderer.java
              Jack Van Ooststroom made changes -
              Field Original Value New Value
              Assignee Jack van Ooststroom [ jack.van.ooststroom ]
              Hide
              Permalink
              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.
              Jack Van Ooststroom made changes -
              Fix Version/s 1.6DR#4 [ 10060 ]
              Affects Version/s 1.6DR#3 [ 10050 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #13729 Wed May 02 17:02:55 MDT 2007 jack.van.ooststroom Undo fix for ICE-1519
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/outputchart/OutputChartRenderer.java
              Ken Fyten made changes -
              Fix Version/s 1.6 [ 10031 ]
              Fix Version/s 1.6DR#4 [ 10060 ]
              Ken Fyten made changes -
              Fix Version/s 1.7 [ 10080 ]
              Fix Version/s 1.6 [ 10031 ]
              Hide
              Permalink
              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
              Permalink
              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.
              Ken Fyten made changes -
              Status Open [ 1 ] Closed [ 6 ]
              Resolution Won't Fix [ 2 ]

                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: