Bug 646595 - Add RTL support to about:firstrun in Fennec [r=vingtetun]
authorAnas Husseini <linux.anas@gmail.com>
Thu, 31 Mar 2011 17:35:11 +0200
changeset 2958 bfd90c18b93914077249540913cec926a1ac9897
parent 2957 e0842af24ca7f905037dd792ac1bc2e0bda2d68f
child 2959 8167d57cab8ee7f15f6350f9ff03c58ce472731d
push id2494
push uservnicolas@mozilla.com
push dateThu, 31 Mar 2011 15:36:11 +0000
reviewersvingtetun
bugs646595
Bug 646595 - Add RTL support to about:firstrun in Fennec [r=vingtetun]
chrome/content/firstrun/firstrun.xhtml
themes/core/firstRun.css
--- a/chrome/content/firstrun/firstrun.xhtml
+++ b/chrome/content/firstrun/firstrun.xhtml
@@ -54,17 +54,17 @@
 <head>
   <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
   <meta name="viewport" content="width=480; initial-scale=.6667; user-scalable=0;" />
   <title>&firstrun.title;</title>
   <link rel="icon" type="image/png" href="chrome://branding/content/favicon32.png" />
   <link rel="stylesheet" href="chrome://browser/skin/firstRun.css" type="text/css"/>
 </head>
 
-<body id="firstrun" class="">
+<body id="firstrun" dir="&locale.dir;" class="">
   <div id="wrapper">
     <div id="header">
       <a href="http://mozilla.com/m"><img src="chrome://browser/content/firstrun/mozilla.png" width="64" height="17"/></a>
     </div>
 
     <div id="head">
       <h1><img src="chrome://branding/content/logoWordmark.png"/></h1>
       <p>&firstrun.heading2;</p>
@@ -73,17 +73,17 @@
     <div id="content">
       <h2></h2>
       <ul id="recommended">
         <li id="sync"><a href="javascript:loadSync();"><b>&firstrun.sync2.title;</b> <span>&firstrun.sync2.description;</span></a></li>
         <li id="addons"><a href="javascript:loadAddons();"><b>&firstrun.addons2.title;</b> <span>&firstrun.addons2.description;</span></a></li>
         <li id="home"><a href="about:home"><b>&firstrun.home2.title;</b> <span>&firstrun.home2.description;</span></a></li>
       </ul>
 
-      <p id="features"><a id="featuresURL">&firstrun.features;</a></p>
+      <p id="features"><a id="featuresURL"><span>&firstrun.features;</span><img id="nav-arrow" src="chrome://browser/content/firstrun/nav-arrow.png" /></a></p>
       <script type="application/javascript;version=1.8"><![CDATA[
         let Ci = Components.interfaces, Cc = Components.classes, Cu = Components.utils;
         let gChromeWin = null;
 
         let links = [
           {id: "releaseNotesURL", pref: "app.releaseNotesURL"},
           {id: "featuresURL", pref: "app.featuresURL"},
           {id: "faqURL", pref: "app.faqURL"},
--- a/themes/core/firstRun.css
+++ b/themes/core/firstRun.css
@@ -218,16 +218,26 @@ img {
 
 #firstrun #features {
   clear: both;
   text-align: center;
 }
 
 #firstrun #features a {
   display: inline-block;
-  padding: 5px 35px 5px 0;
-  background: url("chrome://browser/content/firstrun/nav-arrow.png") right center no-repeat;
   font-size: 16px;
 }
 
+#firstrun #features a span {
+  vertical-align: 35%;
+}
+
+#firstrun #features a #nav-arrow {
+  -moz-margin-start: 5px; 
+}
+
+#firstrun[dir="rtl"] #features a #nav-arrow {
+  -moz-transform: scaleX(-1);
+}
+
 #firstrun #demo {
   text-align: center;
 }