From adffaces-commits-return-1988-apmail-incubator-adffaces-commits-archive=incubator.apache.org@incubator.apache.org Tue Feb 06 23:16:06 2007 Return-Path: Delivered-To: apmail-incubator-adffaces-commits-archive@locus.apache.org Received: (qmail 1267 invoked from network); 6 Feb 2007 23:16:05 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 6 Feb 2007 23:16:05 -0000 Received: (qmail 84926 invoked by uid 500); 6 Feb 2007 23:16:12 -0000 Delivered-To: apmail-incubator-adffaces-commits-archive@incubator.apache.org Received: (qmail 84911 invoked by uid 500); 6 Feb 2007 23:16:12 -0000 Mailing-List: contact adffaces-commits-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: adffaces-dev@incubator.apache.org Delivered-To: mailing list adffaces-commits@incubator.apache.org Received: (qmail 84902 invoked by uid 99); 6 Feb 2007 23:16:12 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Feb 2007 15:16:12 -0800 X-ASF-Spam-Status: No, hits=-9.4 required=10.0 tests=ALL_TRUSTED,NO_REAL_NAME X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Feb 2007 15:16:03 -0800 Received: by eris.apache.org (Postfix, from userid 65534) id 7E9B41A981A; Tue, 6 Feb 2007 15:15:43 -0800 (PST) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r504361 [1/2] - in /incubator/adffaces/trunk/trinidad: trinidad-demo/src/main/webapp/skins/purple/ trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/ trinidad-impl/src/test/resources/org/apache/myfaces/tri... Date: Tue, 06 Feb 2007 23:15:42 -0000 To: adffaces-commits@incubator.apache.org From: jwaldman@apache.org X-Mailer: svnmailer-1.1.0 Message-Id: <20070206231543.7E9B41A981A@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: jwaldman Date: Tue Feb 6 15:15:40 2007 New Revision: 504361 URL: http://svn.apache.org/viewvc?view=rev&rev=504361 Log: ADFFACES-368 selectOrderShuttle displays reorder up down icons improperly. https://issues.apache.org/jira/browse/ADFFACES-368 This happened for me on IE only. The fix is to get rid of all the spacers but one, and that one only have a width of 1. Modified: incubator/adffaces/trunk/trinidad/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectOrderShuttleRenderer.java incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimal-golden.xml incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIE-golden.xml incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIERtl-golden.xml incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalInacc-golden.xml incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalSaf-golden.xml incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalScrRdr-golden.xml Modified: incubator/adffaces/trunk/trinidad/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css?view=diff&rev=504361&r1=504360&r2=504361 ============================================================================== --- incubator/adffaces/trunk/trinidad/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css (original) +++ incubator/adffaces/trunk/trinidad/trinidad-demo/src/main/webapp/skins/purple/purpleSkin.css Tue Feb 6 15:15:40 2007 @@ -341,6 +341,32 @@ padding:5px; } +/* shuttle */ +/** Icons unique to the selectOrderShuttle for re-ordering user selections **/ +af|selectOrderShuttle::reorder-top-icon { + content: url(/skins/purple/images/reordertop_enabled.png); + width: 16px; + height: 16px; +} + +af|selectOrderShuttle::reorder-up-icon { + content: url(/skins/purple/images/reorderup_enabled.png); + width: 16px; + height: 16px; +} + +af|selectOrderShuttle::reorder-down-icon { + content: url(/skins/purple/images/reorderdown_enabled.png); + width: 16px; + height: 16px; +} + +af|selectOrderShuttle::reorder-bottom-icon { + content: url(/skins/purple/images/reorderbottom_enabled.png); + width: 16px; + height: 16px; +} + /** panelHeader **/ /** ----------- **/ af|panelHeader Modified: incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectOrderShuttleRenderer.java URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectOrderShuttleRenderer.java?view=diff&rev=504361&r1=504360&r2=504361 ============================================================================== --- incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectOrderShuttleRenderer.java (original) +++ incubator/adffaces/trunk/trinidad/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/SelectOrderShuttleRenderer.java Tue Feb 6 15:15:40 2007 @@ -85,7 +85,6 @@ "af_selectOrderShuttle.REORDER_UP_ALL_TIP", disabled ? null : "javascript:TrShuttleProxy._orderTopBottomList(0,'" + listId + "');"); - renderSpacer(context, rc, "1", "5"); rw.startElement("br", null); rw.endElement("br"); @@ -95,7 +94,8 @@ "af_selectOrderShuttle.REORDER_UP_TIP", disabled ? null : "javascript:TrShuttleProxy._orderList(0,'" + listId + "');"); - renderSpacer(context, rc, "30", "15"); + // we render 15 vertical spaces to separate the up from the down icons. + renderSpacer(context, rc, "1", "15"); rw.startElement("br", null); rw.endElement("br"); @@ -105,7 +105,6 @@ "af_selectOrderShuttle.REORDER_DOWN_TIP", disabled ? null : "javascript:TrShuttleProxy._orderList(1,'" + listId + "');"); - renderSpacer(context, rc, "1", "5"); rw.startElement("br", null); rw.endElement("br"); Modified: incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimal-golden.xml URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimal-golden.xml?view=diff&rev=504361&r1=504360&r2=504361 ============================================================================== --- incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimal-golden.xml (original) +++ incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimal-golden.xml Tue Feb 6 15:15:40 2007 @@ -413,14 +413,6 @@ > Top - - -
@@ -434,7 +426,7 @@ @@ -449,14 +441,6 @@ > Down - - -
@@ -992,14 +976,6 @@ > Top - - -
@@ -1013,7 +989,7 @@ @@ -1028,14 +1004,6 @@ > Down - - -
@@ -1564,14 +1532,6 @@ > Top - - -
@@ -1584,7 +1544,7 @@ @@ -1598,14 +1558,6 @@ > Down - - -
@@ -2140,14 +2092,6 @@ > Top - - -
@@ -2161,7 +2105,7 @@ @@ -2176,14 +2120,6 @@ > Down - - -
@@ -2790,14 +2726,6 @@ > Top - - -
@@ -2811,7 +2739,7 @@ @@ -2826,14 +2754,6 @@ > Down - - -
@@ -3446,14 +3366,6 @@ > Top - - -
@@ -3467,7 +3379,7 @@ @@ -3482,14 +3394,6 @@ > Down - - -
@@ -4025,14 +3929,6 @@ > Top - - -
@@ -4046,7 +3942,7 @@ @@ -4061,14 +3957,6 @@ > Down - - -
@@ -4597,14 +4485,6 @@ > Top - - -
@@ -4617,7 +4497,7 @@ @@ -4631,14 +4511,6 @@ > Down - - -
@@ -5173,14 +5045,6 @@ > Top - - -
@@ -5194,7 +5058,7 @@ @@ -5209,14 +5073,6 @@ > Down - - -
@@ -5766,14 +5622,6 @@ > Top - - -
@@ -5787,7 +5635,7 @@ @@ -5802,14 +5650,6 @@ > Down - - -
@@ -6345,14 +6185,6 @@ > Top - - -
@@ -6366,7 +6198,7 @@ @@ -6381,14 +6213,6 @@ > Down - - -
@@ -6924,14 +6748,6 @@ > Top - - -
@@ -6945,7 +6761,7 @@ @@ -6960,14 +6776,6 @@ > Down - - -
@@ -7574,14 +7382,6 @@ > Top - - -
@@ -7595,7 +7395,7 @@ @@ -7610,14 +7410,6 @@ > Down - - -
@@ -8230,14 +8022,6 @@ > Top - - -
@@ -8251,7 +8035,7 @@ @@ -8266,14 +8050,6 @@ > Down - - -
@@ -8809,14 +8585,6 @@ > Top - - -
@@ -8830,7 +8598,7 @@ @@ -8845,14 +8613,6 @@ > Down - - -
@@ -9389,14 +9149,6 @@ > Top - - -
@@ -9410,7 +9162,7 @@ @@ -9425,14 +9177,6 @@ > Down - - -
Modified: incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIE-golden.xml URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIE-golden.xml?view=diff&rev=504361&r1=504360&r2=504361 ============================================================================== --- incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIE-golden.xml (original) +++ incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIE-golden.xml Tue Feb 6 15:15:40 2007 @@ -401,10 +401,6 @@ > Top -
@@ -417,7 +413,7 @@
@@ -429,10 +425,6 @@ > Down -
@@ -948,10 +940,6 @@ > Top -
@@ -964,7 +952,7 @@
@@ -976,10 +964,6 @@ > Down -
@@ -1488,10 +1472,6 @@ > Top -
@@ -1503,7 +1483,7 @@
@@ -1514,10 +1494,6 @@ > Down -
@@ -2032,10 +2008,6 @@ > Top -
@@ -2048,7 +2020,7 @@
@@ -2060,10 +2032,6 @@ > Down -
@@ -2642,10 +2610,6 @@ > Top -
@@ -2658,7 +2622,7 @@
@@ -2670,10 +2634,6 @@ > Down -
@@ -3258,10 +3218,6 @@ > Top -
@@ -3274,7 +3230,7 @@
@@ -3286,10 +3242,6 @@ > Down -
@@ -3805,10 +3757,6 @@ > Top -
@@ -3821,7 +3769,7 @@
@@ -3833,10 +3781,6 @@ > Down -
@@ -4345,10 +4289,6 @@ > Top -
@@ -4360,7 +4300,7 @@
@@ -4371,10 +4311,6 @@ > Down -
@@ -4889,10 +4825,6 @@ > Top -
@@ -4905,7 +4837,7 @@
@@ -4917,10 +4849,6 @@ > Down -
@@ -5446,10 +5374,6 @@ > Top -
@@ -5462,7 +5386,7 @@
@@ -5474,10 +5398,6 @@ > Down -
@@ -5993,10 +5913,6 @@ > Top -
@@ -6009,7 +5925,7 @@
@@ -6021,10 +5937,6 @@ > Down -
@@ -6540,10 +6452,6 @@ > Top -
@@ -6556,7 +6464,7 @@
@@ -6568,10 +6476,6 @@ > Down -
@@ -7150,10 +7054,6 @@ > Top -
@@ -7166,7 +7066,7 @@
@@ -7178,10 +7078,6 @@ > Down -
@@ -7766,10 +7662,6 @@ > Top -
@@ -7782,7 +7674,7 @@
@@ -7794,10 +7686,6 @@ > Down -
@@ -8313,10 +8201,6 @@ > Top -
@@ -8329,7 +8213,7 @@
@@ -8341,10 +8225,6 @@ > Down -
@@ -8861,10 +8741,6 @@ > Top -
@@ -8877,7 +8753,7 @@
@@ -8889,10 +8765,6 @@ > Down -
Modified: incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIERtl-golden.xml URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIERtl-golden.xml?view=diff&rev=504361&r1=504360&r2=504361 ============================================================================== --- incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIERtl-golden.xml (original) +++ incubator/adffaces/trunk/trinidad/trinidad-impl/src/test/resources/org/apache/myfaces/trinidadinternal/renderkit/golden/selectOrderShuttle-minimalIERtl-golden.xml Tue Feb 6 15:15:40 2007 @@ -401,10 +401,6 @@ > Top -
@@ -417,7 +413,7 @@
@@ -429,10 +425,6 @@ > Down -
@@ -948,10 +940,6 @@ > Top -
@@ -964,7 +952,7 @@
@@ -976,10 +964,6 @@ > Down -
@@ -1488,10 +1472,6 @@ > Top -
@@ -1503,7 +1483,7 @@
@@ -1514,10 +1494,6 @@ > Down -
@@ -2032,10 +2008,6 @@ > Top -
@@ -2048,7 +2020,7 @@
@@ -2060,10 +2032,6 @@ > Down -
@@ -2642,10 +2610,6 @@ > Top -
@@ -2658,7 +2622,7 @@
@@ -2670,10 +2634,6 @@ > Down -
@@ -3258,10 +3218,6 @@ > Top -
@@ -3274,7 +3230,7 @@
@@ -3286,10 +3242,6 @@ > Down -
@@ -3805,10 +3757,6 @@ > Top -
@@ -3821,7 +3769,7 @@
@@ -3833,10 +3781,6 @@ > Down -
@@ -4345,10 +4289,6 @@ > Top -
@@ -4360,7 +4300,7 @@
@@ -4371,10 +4311,6 @@ > Down -
@@ -4889,10 +4825,6 @@ > Top -
@@ -4905,7 +4837,7 @@
@@ -4917,10 +4849,6 @@ > Down -
@@ -5446,10 +5374,6 @@ > Top -
@@ -5462,7 +5386,7 @@
@@ -5474,10 +5398,6 @@ > Down -
@@ -5993,10 +5913,6 @@ > Top -
@@ -6009,7 +5925,7 @@
@@ -6021,10 +5937,6 @@ > Down -
@@ -6540,10 +6452,6 @@ > Top -
@@ -6556,7 +6464,7 @@
@@ -6568,10 +6476,6 @@ > Down -
@@ -7150,10 +7054,6 @@ > Top -
@@ -7166,7 +7066,7 @@
@@ -7178,10 +7078,6 @@ > Down -
@@ -7766,10 +7662,6 @@ > Top -
@@ -7782,7 +7674,7 @@
@@ -7794,10 +7686,6 @@ > Down -
@@ -8313,10 +8201,6 @@ > Top -
@@ -8329,7 +8213,7 @@
@@ -8341,10 +8225,6 @@ > Down -
@@ -8861,10 +8741,6 @@ > Top -
@@ -8877,7 +8753,7 @@
@@ -8889,10 +8765,6 @@ > Down -