Return-Path: 
 <commits-return-3769-apmail-roller-commits-archive=roller.apache.org@roller.apache.org>
Delivered-To: apmail-roller-commits-archive@www.apache.org
Received: (qmail 46270 invoked from network); 6 Jun 2007 03:51:59 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2)
  by minotaur.apache.org with SMTP; 6 Jun 2007 03:51:59 -0000
Received: (qmail 6293 invoked by uid 500); 6 Jun 2007 03:52:03 -0000
Delivered-To: apmail-roller-commits-archive@roller.apache.org
Received: (qmail 6265 invoked by uid 500); 6 Jun 2007 03:52:03 -0000
Mailing-List: contact commits-help@roller.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@roller.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@roller.apache.org>
List-Post: <mailto:commits@roller.apache.org>
List-Id: <commits.roller.apache.org>
Reply-To: dev@roller.apache.org
Delivered-To: mailing list commits@roller.apache.org
Received: (qmail 6254 invoked by uid 99); 6 Jun 2007 03:52:02 -0000
Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Jun 2007 20:52:02 -0700
X-ASF-Spam-Status: No, hits=-99.5 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, 05 Jun 2007 20:51:58 -0700
Received: by eris.apache.org (Postfix, from userid 65534)
	id 5FAD11A981A; Tue,  5 Jun 2007 20:51:38 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r544720 -
 /roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp
Date: Wed, 06 Jun 2007 03:51:38 -0000
To: commits@roller.apache.org
From: agilliland@apache.org
X-Mailer: svnmailer-1.1.0
Message-Id: <20070606035138.5FAD11A981A@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: agilliland
Date: Tue Jun  5 20:51:37 2007
New Revision: 544720

URL: http://svn.apache.org/viewvc?view=rev&rev=544720
Log:
fix some browser compatability issues with theme selector.


Modified:
    roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp

Modified: roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp
URL: http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp?view=diff&rev=544720&r1=544719&r2=544720
==============================================================================
--- roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp (original)
+++ roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp Tue Jun  5 20:51:37 2007
@@ -30,28 +30,22 @@
 
 function updateThemeChooser(selected) {
     if(selected.value == 'shared') {
-        selectedChooser = $('sharedChooser');
-        selectedOptioner = $('sharedOptioner');
+        $('sharedChooser').style.backgroundColor="#CCFFCC";
+        $('sharedChooser').style.border="1px solid #008000";
+        $('sharedOptioner').show();
         
-        otherChooser = $('customChooser');
-        otherOptioner = $('customOptioner');
+        $('customChooser').style.backgroundColor="#eee";
+        $('customChooser').style.border="1px solid gray";
+        $('customOptioner').hide();
     } else {
-        selectedChooser = $('customChooser');
-        selectedOptioner = $('customOptioner');
+        $('customChooser').style.backgroundColor="#CCFFCC";
+        $('customChooser').style.border="1px solid #008000";
+        $('customOptioner').show();
         
-        otherChooser = $('sharedChooser');
-        otherOptioner = $('sharedOptioner');
+        $('sharedChooser').style.backgroundColor="#eee";
+        $('sharedChooser').style.border="1px solid gray";
+        $('sharedOptioner').hide();
     }
-    
-    // update styling on chooser
-    selectedChooser.style.backgroundColor="#CCFFCC";
-    selectedChooser.style.border="1px solid #008000";
-    otherChooser.style.backgroundColor="#eee";
-    otherChooser.style.border="1px solid grey";
-    
-    // update display of selected optioner
-    otherOptioner.hide();
-    selectedOptioner.show();
 }
 -->
 </script>
@@ -69,13 +63,13 @@
         <tr>
             <td width="50%">
                 <div id="sharedChooser" class="chooser">
-                    <h2><input id="sharedRadio" type="radio" name="themeType" value="shared" <s:if test="!customTheme">checked="true"</s:if> onchange="updateThemeChooser(this)" />&nbsp;<s:text name="themeEditor.sharedTheme" /></h2>
+                    <h2><input id="sharedRadio" type="radio" name="themeType" value="shared" <s:if test="!customTheme">checked="true"</s:if> onclick="updateThemeChooser(this)" />&nbsp;<s:text name="themeEditor.sharedTheme" /></h2>
                     <s:text name="themeEditor.sharedThemeDescription" />
                 </div>
             </td>
             <td width="50%">
                 <div id="customChooser" class="chooser">
-                    <h2><input id="customRadio" type="radio" name="themeType" value="custom" <s:if test="customTheme">checked="true"</s:if> onchange="updateThemeChooser(this)" />&nbsp;<s:text name="themeEditor.customTheme" /></h2>
+                    <h2><input id="customRadio" type="radio" name="themeType" value="custom" <s:if test="customTheme">checked="true"</s:if> onclick="updateThemeChooser(this)" />&nbsp;<s:text name="themeEditor.customTheme" /></h2>
                     <s:text name="themeEditor.customThemeDescription" />
                 </div>
             </td>