-fix css rules

- comment_list.tpl style was broken
- picture_nav_buttons.tpl shorter javascript

git-svn-id: http://piwigo.org/svn/trunk@7877 68402e56-0260-453c-a942-63ccdbb3a9ee

themes/Sylvia/theme.css

@@ -7,7 +7,7 @@
 #comments .thumbnailCategories hr {display:none;}
 #comments form.filter input[type="submit"] { margin-top:1.8em; }
 /*  Sylvia Theme is close to the Roma Admin theme */
-BODY, H1, H3, DT, .rateButtonSelected, H2, #menubar DT { 
+BODY, H1, H3, DT, INPUT.rateButtonSelected, H2, #menubar DT { 
 	color:#666; 
 }
 #imageHeaderBar, #imageToolBar A:hover { 

themes/clear/theme.css

@@ -1,7 +1,7 @@
 
 /* text color */
 BODY, H1, H2, H3, DT,
-.rateButtonSelected /* <= why IE doesn't inherit this ? */ {
+INPUT.rateButtonSelected /* <= why IE doesn't inherit this ? */ {
 	color:#696969;	/* dimgray */
 }
 
@@ -49,12 +49,12 @@ UL.thumbnails SPAN.wrap2:hover,
 }
 
 /* links */
-A, .rateButton {
+A, INPUT.rateButton {
     color: #005e89;
     background: transparent;
 }
 
-A:hover, .rateButton:hover {
+A:hover, INPUT.rateButton:hover {
     color: #858460;
 }
 

themes/dark/theme.css

@@ -1,7 +1,7 @@
 
 /* text color */
 BODY, H1, H3,
-.rateButtonSelected /* <= why IE doesn't inherit this ? */ {
+INPUT.rateButtonSelected /* <= why IE doesn't inherit this ? */ {
 	color:#d0d0d0;
 }
 
@@ -65,7 +65,7 @@ UL.thumbnails SPAN.wrap2:hover,
 
 
 /* links */
-A, .rateButton {
+A, INPUT.rateButton {
 	color: #fff;
 	border: 0;
 }

themes/default/template/comment_list.tpl

@@ -10,7 +10,7 @@
 		</a>
 	</div>
 	{/if}
-    <div class="description" style="height:{if isset($comment.IN_EDIT)}200{/if}px">
+	<div class="description"{if isset($comment.IN_EDIT)} style="height:200px"{/if}>
 		{if isset($comment.U_DELETE) or isset($comment.U_VALIDATE) or isset($comment.U_EDIT) }
 		<ul class="actions" style="float:right">
 		{if isset($comment.U_DELETE)}
@@ -36,6 +36,7 @@
 		{/if}
 		</ul>
 		{/if}
+
 		<span class="author">{$comment.AUTHOR}</span> - <span class="date">{$comment.DATE}</span>
 		{if isset($comment.IN_EDIT)}
 		<a name="edit_comment"></a>

themes/default/template/picture_nav_buttons.tpl

@@ -63,42 +63,42 @@
 
   </div>
 
-<script type="text/javascript">// <![CDATA[
-{literal}
-function keyboardNavigation(e)
+<script type="text/javascript">// <![CDATA[ {literal}
+document.onkeydown = function(e)
 {
-	if(!e) e=window.event;
+	e=e||window.event;
 	if (e.altKey) return true;
-	var target = e.target || e.srcElement;
+	var target = e.target||e.srcElement;
 	if (target && target.type) return true; //an input editable element
-	var keyCode=e.keyCode || e.which;
+	var keyCode = e.keyCode||e.which;
 	var docElem = document.documentElement;
+	var url;
 	switch(keyCode) {
 {/literal}
 {if isset($next)}
-	case 63235: case 39: if (e.ctrlKey || docElem.scrollLeft==docElem.scrollWidth-docElem.clientWidth ){ldelim}window.location="{$next.U_IMG}".replace( "&amp;", "&" ); return false; } break;
+	case 63235: case 39: if (e.ctrlKey || docElem.scrollLeft==docElem.scrollWidth-docElem.clientWidth)url="{$next.U_IMG|@escape:jasvascript}";  break;
 {/if}
 {if isset($previous)}
-	case 63234: case 37: if (e.ctrlKey || docElem.scrollLeft==0){ldelim}window.location="{$previous.U_IMG|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	case 63234: case 37: if (e.ctrlKey || docElem.scrollLeft==0)url="{$previous.U_IMG|@escape:jasvascript}"; break;
 {/if}
 {if isset($first)}
-	/*Home*/case 36: if (e.ctrlKey){ldelim}window.location="{$first.U_IMG|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	/*Home*/case 36: if (e.ctrlKey)url="{$first.U_IMG|@escape:jasvascript}"; break;
 {/if}
 {if isset($last)}
-	/*End*/case 35: if (e.ctrlKey){ldelim}window.location="{$last.U_IMG|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	/*End*/case 35: if (e.ctrlKey)url="{$last.U_IMG|@escape:jasvascript}"; break;
 {/if}
 {if isset($U_UP) and !isset($slideshow)}
-	/*Up*/case 38: if (e.ctrlKey){ldelim}window.location="{$U_UP|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	/*Up*/case 38: if (e.ctrlKey)url="{$U_UP|@escape:jasvascript}"; break;
 {/if}
 
 {if isset($slideshow.U_START_PLAY)}
-	/*Pause*/case 32: {ldelim}window.location="{$slideshow.U_START_PLAY|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	/*Pause*/case 32: url="{$slideshow.U_START_PLAY|@escape:jasvascript}"; break;
 {/if}
 {if isset($slideshow.U_STOP_PLAY)}
-	/*Play*/case 32: {ldelim}window.location="{$slideshow.U_STOP_PLAY|@escape:jasvascript}".replace("&amp;","&"); return false; } break;
+	/*Play*/case 32: url="{$slideshow.U_STOP_PLAY|@escape:jasvascript}"; break;
 {/if}
 	}
+	if (url) {ldelim}window.location=url.replace("&amp;","&"); return false;}
 	return true;
 }
-document.onkeydown=keyboardNavigation;
 // ]]></script>

themes/default/theme.css

@@ -300,7 +300,7 @@ UL.thumbnails SPAN.wrap2 {
 	margin: 0;			/* important reset the margins */
 	display: table-cell;/* block prevents vertical-align here */
 	vertical-align: middle;/* Ok with Opera and Geko not IE6 */
-	border-radius: 4px 4px;	/* round corners with CSS3 compliant browsers */
+	border-radius: 4px;	/* round corners with CSS3 compliant browsers */
 	-moz-border-radius: 4px;	/* round corners with Geko */
 	-webkit-border-radius: 4px; /* Safari webkit project */
 }
@@ -399,6 +399,8 @@ TABLE.infoTable TD.value UL {
 .rateButton, .rateButtonSelected, .rateButtonStarFull, .rateButtonStarEmpty  {
 	padding:0;
 	border:0;
+	color:inherit;
+	background-color:transparent !important;	/* Konqueror doesn't accept transparent here */
 }
 
 .rateButton, .rateButtonStarFull, .rateButtonStarEmpty {
@@ -418,6 +420,7 @@ TABLE.infoTable TD.value UL {
 	background: url('icon/rating-stars.gif') no-repeat scroll; background-position:0 center; width:16px;
 }
 
+
 /**
  * Default Layout
  */
@@ -651,17 +654,12 @@ UL.tagSelection LI {
 	display: none;
 }
 
-#thePopuphelpPage P {
-	text-align: justify;
+#thePopuphelpPage P, #theNotificationPage P {
 	padding: 0.5em;
 }
 
-#thePopuphelpPage LI {
-	margin-bottom: 0.5em;
-}
-
-#theNotificationPage P {
-	padding: 0.5em;
+#theNotificationPage DL, #thePopuphelpPage DL {
+	margin: 0 25px 25px; 
 }
 
 TR.tagLine {
@@ -696,10 +694,6 @@ TABLE.tagLetterContent {
 
 #theHeader {text-align: center;}
 
-#theNotificationPage DL, #thePopuphelpPage DL {
-	margin: 0 25px 25px; 
-}
-
 .content #comments UL.thumbnailCategories LI { width:99%; }
 
 /* jQuery datepicker */
@@ -771,11 +765,6 @@ INPUT.radio, INPUT.checkbox {
 	border: none;	/* <= IE6 needs this */
 }
 
-/* rate buttons displayed like links */
-.rateButton, .rateButtonSelected, .rateButtonStarFull, .rateButtonStarEmpty {
-	color:inherit;
-	background-color:transparent !important;	/* Konqueror doesn't accept transparent here */
-}
 
 .errors { /* Errors display */
 	color: red;