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

Unified Diff: skin/desktop-options.css

Issue 29673576: Make new options page compatible with edge (Closed)
Patch Set: dropped automatically inserted extra line Created Feb. 5, 2018, 9:44 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
« desktop-options.js ('K') | « desktop-options.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skin/desktop-options.css
===================================================================
--- a/skin/desktop-options.css
+++ b/skin/desktop-options.css
@@ -225,15 +225,39 @@
.side-controls button
{
margin: 0rem;
- -moz-margin-start: 1rem;
- -webkit-margin-start: 1rem;
+}
+
+/*
+ Due to Edge adoption as new target browser
+ we cannot use -moz/webkit-margin-start
+ or -moz/webkit-margin-end because
+ these lack Edge support.
+ Yet we need to preserve html direction
+ and potential UI that might swap right to left.
+*/
+html:not([dir="rtl"]) .side-controls button
+{
+ margin-left: 1rem;
+}
+
+html[dir="rtl"] .side-controls button
+{
+ margin-right: 1rem;
}
.side-controls.wrap button
{
margin: 0.6rem 0rem;
- -moz-margin-start: auto;
- -webkit-margin-start: auto;
+}
+
+html:not([dir="rtl"]) .side-controls.wrap button
+{
+ margin-left: auto;
+}
+
+html[dir="rtl"] .side-controls.wrap button
+{
+ margin-right: auto;
}
/*
@@ -544,7 +568,7 @@
#sidebar
{
- flex-shrink: 0;
+ flex-shrink: 0;
}
#sidebar .fixed
@@ -615,10 +639,8 @@
display: flex;
margin-top: -1px;
padding: 1rem 0.8rem;
- -moz-margin-end: -1px;
- -webkit-margin-end: -1px;
- -moz-margin-start: -1px;
- -webkit-margin-start: -1px;
+ margin-left: -1px;
+ margin-right: -1px;
border-color: #CDCDCD transparent;
border-style: solid;
border-width: 1px;
@@ -629,12 +651,20 @@
li a[role="tab"][aria-selected]
{
- -moz-border-start-color: #CDCDCD;
- -webkit-border-start-color: #CDCDCD;
font-weight: 700;
background-color: #FFF;
}
+html:not([dir="rtl"]) li a[role="tab"][aria-selected]
+{
+ border-left-color: #CDCDCD;
+}
+
+html[dir="rtl"] li a[role="tab"][aria-selected]
+{
+ border-right-color: #CDCDCD;
+}
+
#sidebar footer
{
width: 100%;
@@ -735,8 +765,16 @@
section.cols > *:first-child
{
flex: 1;
- -moz-margin-end: 2rem;
- -webkit-margin-end: 2rem;
+}
+
+html:not([dir="rtl"]) section.cols > *:first-child
+{
+ margin-right: 2rem;
+}
+
+html[dir="rtl"] section.cols > *:first-child
+{
+ margin-left: 2rem;
}
section.cols > *:last-child
@@ -811,8 +849,16 @@
{
font-weight: 700;
font-size: 1rem;
- -moz-margin-end: 0.5rem;
- -webkit-margin-end: 0.5rem;
+}
+
+html:not([dir="rtl"]) #acceptable-ads label
+{
+ margin-right: 0.5rem;
+}
+
+html[dir="rtl"] #acceptable-ads label
+{
+ margin-left: 0.5rem;
}
#dnt
@@ -976,10 +1022,20 @@
.table .col5:nth-of-type(5)
{
flex: 1;
- -moz-margin-start: 0;
- -webkit-margin-start: 0;
- -moz-margin-end: 1.8rem;
- -webkit-margin-end: 1.8rem;
+}
+
+html:not([dir="rtl"]) .th .col5:nth-of-type(5),
+html:not([dir="rtl"]) .table .col5:nth-of-type(5)
+{
+ margin-left: 0;
+ margin-right: 1.8rem;
+}
+
+html[dir="rtl"] .th .col5:nth-of-type(5),
+html[dir="rtl"] .table .col5:nth-of-type(5)
+{
+ margin-right: 0;
+ margin-left: 1.8rem;
}
.table.cols > span
@@ -992,10 +1048,14 @@
padding: 0.5rem 0rem;
}
-.table.cols .state
+html:not([dir="rtl"]) .table.cols .state
{
- -moz-margin-start: 1rem;
- -webkit-margin-start: 1rem;
+ margin-left: 1rem;
+}
+
+html[dir="rtl"] .table.cols .state
+{
+ margin-right: 1rem;
}
.table.cols .gear
@@ -1052,13 +1112,21 @@
{
position: relative;
margin: 0rem;
- -moz-margin-end: 1rem;
- -webkit-margin-end: 1rem;
line-height: 1.5rem;
text-decoration: none;
cursor: help;
}
+html:not([dir="rtl"]) .tooltip
+{
+ margin-right: 1rem;
+}
+
+html[dir="rtl"] .tooltip
+{
+ margin-left: 1rem;
+}
+
/*
Used for translatable screen reader only content.
e.g.: Use instead of aria-label to avoid complex attribute value translation
@@ -1110,10 +1178,14 @@
border: 2px solid #077CA6;
}
-#content-whitelist form button
+html:not([dir="rtl"]) #content-whitelist form button
{
- -moz-margin-start: 0.7rem;
- -webkit-margin-start: 0.7rem;
+ margin-left: 0.7rem;
+}
+
+html[dir="rtl"] #content-whitelist form button
+{
+ margin-right: 0.7rem;
}
#whitelisting-table li
@@ -1346,8 +1418,16 @@
#social ul li
{
display: inline-block;
- -moz-margin-end: 1rem;
- -webkit-margin-end: 1rem;
+}
+
+html:not([dir="rtl"]) #social ul li
+{
+ margin-right: 1rem;
+}
+
+html[dir="rtl"] #social ul li
+{
+ margin-left: 1rem;
}
#social ul li a
« desktop-options.js ('K') | « desktop-options.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld