http://gwt-code-reviews.appspot.com/912801/diff/1/2 File user/src/com/google/gwt/cell/client/IconCellDecorator.java (right):
http://gwt-code-reviews.appspot.com/912801/diff/1/2#newcode53 user/src/com/google/gwt/cell/client/IconCellDecorator.java:53: @Template("<div style=\"position:absolute;{0}:0px;top:0px;height:100%;" Thats cool - we can take advantage of the fact that we know the height of the image. Thanks! http://gwt-code-reviews.appspot.com/912801/diff/1/2#newcode99 user/src/com/google/gwt/cell/client/IconCellDecorator.java:99: this.imageWidth = icon.getWidth() + 6; On 2010/09/23 08:33:22, tbroyer wrote:
I know it's not the purpose of this patch but... should this "6" be
"spacing"? Done. http://gwt-code-reviews.appspot.com/912801/diff/1/2#newcode175 user/src/com/google/gwt/cell/client/IconCellDecorator.java:175: return template.imageWrapper(direction, res.getWidth(), image); Good catch. I'll fix this. http://gwt-code-reviews.appspot.com/912801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
