Issue 2292 - Add a disclaimer text on all landing pages in French language that provide the user li…

static/css/index.css

Index: static/css/index.css
===================================================================
--- a/static/css/index.css
+++ b/static/css/index.css
@@ -458,75 +458,62 @@
 }
 
 #content.edge #other-platform > div,
 #content.edge h3.edge-message
 {
   display: block;
 }
 
-.warning-platform,
-.disclaimer,
 #maxthon-notification
 {
   display: none;
   color: #D00;
   text-align: center;
   border: 1px solid #D00;
   padding: 5px;
   border-radius: 5px;
 }
 
-.disclaimer
+.only
 {
-  font-size: 14px;
-  color: #000;
-  text-align: left;
+  display: none;
 }
 
-#content.ua-firefox .warning-platform.firefox,

[saroyanm, 2016/10/26 17:31:18]

I'd not recommend removing the #content selector, because it looks like all the
platform specific selectors are declared through #content ID selector on the
page, I assume it's done so for readability and I'll suggest to leave this
change out of current review, we can adapt similar fix globally to the current
file in separate review I guess, in case there is a need of doing so.

-#content.ua-chrome .warning-platform.chrome,
-#content.ua-android .warning-platform.android,
-#content.ua-opera .warning-platform.opera,
-#content.ua-internet-explorer .warning-platform.internet-explorer,
-#content.ua-safari .warning-platform.safari,
-#content.ua-yandex-browser .warning-platform.yandex-browser,
-#content.ua-maxthon .warning-platform.maxthon,
-html[lang^="zh_"] #maxthon-notification
+.ua-firefox .only.firefox,
+.ua-chrome .only.chrome,
+.ua-android .only.android,
+.ua-opera .only.opera,
+.ua-internet-explorer .only.internet-explorer,
+.ua-safari .only.safari,
+.ua-yandex-browser .only.yandex-browser,
+.ua-maxthon .only.maxthon
 {
   display: block;
 }
 
-#content.firefox.ua-firefox .warning-platform.firefox,
-#content.chrome.ua-chrome .warning-platform.chrome,
-#content.android.ua-android .warning-platform.android,
-#content.opera.ua-opera .warning-platform.opera,
-#content.internet-explorer.ua-internet-explorer .warning-platform.internet-explorer,
-#content.safari.ua-safari .warning-platform.safari,
-#content.yandex-browser.ua-yandex-browser .warning-platform.yandex-browser,
-#content.maxthon.ua-maxthon .warning-platform.maxthon,
-#content.ua-maxthon #maxthon-notification,
-#content.maxthon #maxthon-notification,
-/* IE6 fixes */

[saroyanm, 2016/10/26 17:31:18]

You removed the comment, but left the implementation.
Same as previous comment, we can remove IE6 support separately, because
currently it's inconsistent, we do have this solution for other selectors, I'd
do that in separate review to keep it consistent.
Note: I'm not sure whether this solution even work for IE, it doens't makes any
sense for until if we are not setting up special class for IE6.

-#content.firefox_ua-firefox .warning-platform.firefox,
-#content.chrome_ua-chrome .warning-platform.chrome,
-#content.opera_ua-opera .warning-platform.opera,
-#content.internet-explorer_ua-internet-explorer .warning-platform.internet-explorer,
-#content.safari_ua-safari .warning-platform.safari,
-#content.yandex-browser_ua-yandex-browser .warning-platform.yandex-browser,
-#content.maxthon_ua-maxthon .warning-platform.maxthon
+.firefox.ua-firefox .only.firefox,
+.chrome.ua-chrome .only.chrome,
+.android.ua-android .only.android,
+.opera.ua-opera .only.opera,
+.internet-explorer.ua-internet-explorer .only.internet-explorer,
+.safari.ua-safari .only.safari,
+.yandex-browser.ua-yandex-browser .only.yandex-browser,
+.maxthon.ua-maxthon .only.maxthon,
+.firefox_ua-firefox .only.firefox,
+.chrome_ua-chrome .only.chrome,
+.opera_ua-opera .only.opera,
+.internet-explorer_ua-internet-explorer .only.internet-explorer,
+.safari_ua-safari .only.safari,
+.yandex-browser_ua-yandex-browser .only.yandex-browser,
+.maxthon_ua-maxthon .only.maxthon
 {
   display: none;
 }
 
-html[lang="fr"] .disclaimer.lang-fr
-{
-  display: block;
-}
-
 #key-facts section
 {
   display: inline-block;
 }
 
 #key-facts section h1
 {
   margin-top: 0px;
@@ -667,9 +654,9 @@
 #more-container.android #more-android,
 #more-container.internet-explorer #more-internet-explorer,
 #more-container.safari #more-safari,
 #more-container.yandex-browser #more-yandex-browser,
 #more-container.maxthon #more-maxthon
 #more-container.edge #more-edge
 {
   background-color: #dbd4c7;
-}
+}