Author: vmolotkov Date: 2008-04-24 07:13:30 -0400 (Thu, 24 Apr 2008) New Revision: 8112 Modified: trunk/ui/orderingList/src/main/java/org/richfaces/renderkit/OrderingComponentRendererBase.java trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/css/orderingList.xcss trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/LayoutManager.js Log: http://jira.jboss.com/jira/browse/RF-3223 Modified: trunk/ui/orderingList/src/main/java/org/richfaces/renderkit/OrderingComponentRendererBase.java =================================================================== --- trunk/ui/orderingList/src/main/java/org/richfaces/renderkit/OrderingComponentRendererBase.java 2008-04-24 10:04:47 UTC (rev 8111) +++ trunk/ui/orderingList/src/main/java/org/richfaces/renderkit/OrderingComponentRendererBase.java 2008-04-24 11:13:30 UTC (rev 8112) @@ -368,11 +368,10 @@ writer.startElement(HTML.IMG_ELEMENT, orderingList); //buttons sometimes don't work under IE if mouse cursor //is above some special areas of the button - writer.writeAttribute(HTML.style_ATTRIBUTE, "margin:0px;padding:0px;border:0px;", null); + writer.writeAttribute(HTML.class_ATTRIBUTE, "rich-ordering-control-img", null); - writer.writeAttribute(HTML.width_ATTRIBUTE, "15", null); - writer.writeAttribute(HTML.height_ATTRIBUTE, "15", null); - writer.writeAttribute(HTML.border_ATTRIBUTE, "0", null); + //writer.writeAttribute(HTML.width_ATTRIBUTE, "15", null); + //writer.writeAttribute(HTML.height_ATTRIBUTE, "15", null); writer.writeAttribute(HTML.alt_ATTRIBUTE, helper.getDefaultText(), null); writer.writeAttribute(HTML.src_ATTRIBUTE, getResource( Modified: trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/css/orderingList.xcss =================================================================== --- trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/css/orderingList.xcss 2008-04-24 10:04:47 UTC (rev 8111) +++ trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/css/orderingList.xcss 2008-04-24 11:13:30 UTC (rev 8112) @@ -5,6 +5,14 @@ <f:verbatim><![CDATA[ +.rich-ordering-control-img { + margin: 0px; + padding: 0px; + border:0px; + width: 15px; + height: 15px; +} + .rich-ordering-control-disabled { border : 1px solid; margin-bottom : 3px; Modified: trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/LayoutManager.js =================================================================== --- trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/LayoutManager.js 2008-04-24 10:04:47 UTC (rev 8111) +++ trunk/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/LayoutManager.js 2008-04-24 11:13:30 UTC (rev 8112) @@ -38,20 +38,17 @@ } } - if (Prototype.Browser.IE) { + if (Prototype.Browser.IE && this.contentDiv && this.contentTable && this.getScrollWidth()) { //IE displays unnecessary horizontal scroll //when vertical scroll's displayed - if (this.contentDiv && this.contentTable && this.getScrollWidth()) { - //alert(this.contentTable.offsetWidth +" "+ this.contentDiv.offsetWidth); - if (this.contentTable.offsetWidth && ((this.contentTable.offsetWidth <= this.contentDiv.clientWidth))) { - this.contentTable.style.width = this.contentDiv.clientWidth + "px"; - this.headerTable.style.width = this.contentDiv.offsetWidth + "px"; - this.contentDiv.style.overflowX = 'hidden'; - } - } else { - this.contentTable.style.width = "100%"; - this.headerTable.style.width = "100%"; + if (this.contentTable.offsetWidth && ((this.contentTable.offsetWidth <= this.contentDiv.clientWidth))) { + this.contentTable.style.width = this.contentDiv.clientWidth + "px"; + this.headerTable.style.width = this.contentDiv.offsetWidth + "px"; + this.contentDiv.style.overflowX = 'hidden'; } + } else { + this.contentTable.style.width = "100%"; + this.headerTable.style.width = "100%"; } }