svn commit: r544720 - /roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/ThemeEdit.jsp

[agilliland]

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>