Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: static/css/index.css

Issue 29749588: Fixes #53 - Added detection and install links for iOS safari (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Separated samsung browser Created April 12, 2018, 11:54 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« includes/index.tmpl ('K') | « includes/index.tmpl ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/css/index.css
===================================================================
--- a/static/css/index.css
+++ b/static/css/index.css
@@ -238,17 +238,17 @@
bottom: 4px;
}
#content.android .install-button,
#content.firefox.ua-firefox .install-button,
#content.chrome.ua-chrome .install-button,
#content.opera.ua-opera .install-button,
#content.internet-explorer.ua-internet-explorer .install-button,
-#content.safari.ua-safari .install-button,
+#content.safari .install-button,
ire 2018/04/12 14:30:32 What's the reason for these changes? (where you re
juliandoucette 2018/04/12 15:02:23 I think that #content.safari applies to #content.s
ire 2018/04/12 15:13:50 But these styles should only be applied if the bot
juliandoucette 2018/04/12 15:30:39 Oops! (I had things backwards.)
#content.yandex-browser.ua-yandex-browser .install-button,
#content.edge.ua-edge .install-button,
/* IE6 fixes */
#content.firefox_ua-firefox .install-button,
#content.chrome_ua-chrome .install-button,
#content.opera_ua-opera .install-button,
#content.internet-explorer_ua-internet-explorer .install-button,
#content.safari_ua-safari .install-button,
@@ -265,17 +265,17 @@
text-shadow: 2px 2px 2px #327d14;
}
#content.android .install-button:hover,
#content.firefox.ua-firefox .install-button:hover,
#content.chrome.ua-chrome .install-button:hover,
#content.opera.ua-opera .install-button:hover,
#content.internet-explorer.ua-internet-explorer .install-button:hover,
-#content.safari.ua-safari .install-button:hover,
+#content.safari .install-button:hover,
#content.yandex-browser.ua-yandex-browser .install-button:hover,
#content.edge.ua-edge .install-button:hover,
/* IE6 fixes */
#content.firefox_ua-firefox .install-button:hover,
#content.chrome_ua-chrome .install-button:hover,
#content.opera_ua-opera .install-button:hover,
#content.internet-explorer_ua-internet-explorer .install-button:hover,
#content.safari_ua-safari .install-button:hover,
@@ -285,17 +285,17 @@
box-shadow: 0px 2px 7px #486c15, 0 1px 0 0 #486c15;
}
#content.android .install-button:active,
#content.firefox.ua-firefox .install-button:active,
#content.chrome.ua-chrome .install-button:active,
#content.opera.ua-opera .install-button:active,
#content.internet-explorer.ua-internet-explorer .install-button:active,
-#content.safari.ua-safari .install-button:active,
+#content.safari .install-button:active,
#content.yandex-browser.ua-yandex-browser .install-button:active,
#content.edge.ua-edge .install-button:active,
/* IE6 fixes */
#content.firefox_ua-firefox .install-button:active,
#content.chrome_ua-chrome .install-button:active,
#content.opera_ua-opera .install-button:active,
#content.internet-explorer_ua-internet-explorer .install-button:active,
#content.safari_ua-safari .install-button:active,
@@ -309,17 +309,18 @@
background: -o-linear-gradient(bottom, #62c52b, #4f9d22);
background: linear-gradient(bottom, #62c52b, #4f9d22);
box-shadow: 0px 0px 7px #486c15;
}
#content.firefox #install-firefox, #content.chrome #install-chrome,
#content.opera #install-opera, #content.android #install-android,
#content.internet-explorer #install-internet-explorer,
-#content.safari #install-safari,
+#content.ua-safari #install-safari,
+#content.ua-ios #install-ios,
#content.yandex-browser #install-yandex-browser,
#content.edge #install-edge
{
display: inline-block;
}
#other-platform ul
{
« includes/index.tmpl ('K') | « includes/index.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld