#195543 by hass and yhager: CSS inherit was completely misused in RTL CSS files

5 files changed, 19 insertions, 21 deletions

diff --git a/themes/bluemarine/style-rtl.css b/themes/bluemarine/style-rtl.css
index eb3131149..ef3c46905 100644
--- a/themes/bluemarine/style-rtl.css
+++ b/themes/bluemarine/style-rtl.css
@@ -17,7 +17,7 @@ body {
 }
 ul.links li {
   border-right: 1px solid #9cf;
-  border-left: inherit;
+  border-left: none;
 }
 .block, .box {
   padding: 0 1.5em 0 0;
diff --git a/themes/chameleon/marvin/style-rtl.css b/themes/chameleon/marvin/style-rtl.css
index 033001329..6da7ca6ba 100644
--- a/themes/chameleon/marvin/style-rtl.css
+++ b/themes/chameleon/marvin/style-rtl.css
@@ -10,7 +10,7 @@ p {
 }
 
 ul.links li {
-  border-left: inherit;
+  border-left: none;
   border-right: 1px solid #888;
 }
 
@@ -25,7 +25,7 @@ ul.links li {
   float: left;
 }
 .node .content {
-  padding-left: inherit;
+  padding-left: 0;
   padding-right: 1em;
 }
 .node .links {
diff --git a/themes/chameleon/style-rtl.css b/themes/chameleon/style-rtl.css
index 9e47579cd..c9c43a479 100644
--- a/themes/chameleon/style-rtl.css
+++ b/themes/chameleon/style-rtl.css
@@ -8,20 +8,20 @@ body {
  * Welcome to design by tables..
  */
 #sidebar-left {
-  border-right: inherit;
+  border-right: none;
   border-left: 1px solid gray;
 }
 #sidebar-right {
-  border-left: inherit;
+  border-left: none;
   border-right: 1px solid gray;
 }
 
 ul.links li {
-  border-left: inherit;
+  border-left: none;
   border-right: 1px solid #000;
 }
 ul.links li.first {
-  border-left: inherit;
+  border-left: none;
   border-right: none;
 }
 div.links {
diff --git a/themes/garland/style-rtl.css b/themes/garland/style-rtl.css
index c59302519..791abf243 100644
--- a/themes/garland/style-rtl.css
+++ b/themes/garland/style-rtl.css
@@ -20,7 +20,7 @@ ul.menu, .item-list ul {
 }
 
 ul.menu ul, .item-list ul ul {
-  margin-left: inherit;
+  margin-left: 0;
   margin-right: 0em;
 }
 
@@ -34,7 +34,7 @@ ul li, ul.menu li, .item-list ul li, li.leaf {
 }
 
 ol li {
-  margin-left: inherit;
+  margin-left: 0;
   margin-right: 2em;
 }
 
@@ -51,7 +51,7 @@ ul.inline li {
 }
 
 ol.task-list {
-  margin-left: inherit;
+  margin-left: 0;
   margin-right: 0;
 }
 
@@ -68,7 +68,7 @@ ol.task-list li.done {
 }
 
 ol.task-list li.active {
-  margin-right: inherit;
+  margin-right: 0;
   margin-left: 1em;
 }
 
@@ -96,10 +96,9 @@ dl dd {
 }
 
 #wrapper #container #header h1 img {
-  padding-right: inherit;
+  padding-right: 0;
+  padding-left: 20px;
   float: right;
-  padding-right: inherit;
-  padding-left : 20px;
 }
 
 #sidebar-left .block-region {
@@ -118,7 +117,7 @@ dl dd {
 #wrapper #container .breadcrumb {
   position: absolute;
   top: 15px;
-  left: inherit;
+  left: 0;
   right: 35px;
   z-index: 3;
 }
@@ -139,7 +138,6 @@ ul.primary-links li {
  * Secondary navigation
  */
 ul.secondary-links {
-  padding: 20px 0 0;
   float: left;
   clear: left;
 }
@@ -186,7 +184,7 @@ ul.links li, ul.inline li {
 }
 
 .picture, .comment .submitted {
-  padding-left: inherit;
+  padding-left: 0;
   float: left;
   clear: left;
   padding-right: 1em;
@@ -201,12 +199,12 @@ ul.links li, ul.inline li {
 }
 
 .indented {
-  margin-left: 0; /* an should be inherit, but force left margin to be 0  */
+  margin-left: 0;
   margin-right: 25px;
 }
 
 html.js fieldset.collapsible legend a {
-  padding-left: inherit;
+  padding-left: 0;
   padding-right: 2em;
   background: url("images/menu-expanded.gif") no-repeat 100% 50%;
 }
@@ -220,7 +218,7 @@ html.js fieldset.collapsed legend a {
  */
 #block-node-0 h2 {
   float: right;
-  padding-right: inherit;
+  padding-right: 0;
   padding-left: 20px;
 }
 
diff --git a/themes/garland/style.css b/themes/garland/style.css
index 4210932f6..473d3d6c1 100644
--- a/themes/garland/style.css
+++ b/themes/garland/style.css
@@ -577,7 +577,7 @@ ul.primary-links li a:hover, ul.primary-links li a.active {
  */
 ul.secondary-links {
   margin: 0;
-  padding: 18px 0 0; /* LTR */
+  padding: 18px 0 0;
   float: right; /* LTR */
   clear: right; /* LTR */
   position: relative;