Merge pull request #12845 from Snuffleupagus/cleanup-more-CSS-dir

Remove unnecessary `dir`-dependent CSS rules, and simplify som findbar-button rules (PR 11077 follow-up)
This commit is contained in:
Tim van der Meij 2021-01-10 13:53:30 +01:00 committed by GitHub
commit f224a0eb9f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -367,15 +367,6 @@ html[dir="rtl"] #sidebarResizer {
position: relative; position: relative;
height: 32px; height: 32px;
background-color: var(--toolbar-bg-color); background-color: var(--toolbar-bg-color);
}
html[dir="ltr"] #toolbarContainer,
.findbar,
.secondaryToolbar {
box-shadow: 0 1px 0 var(--toolbar-border-color);
}
html[dir="rtl"] #toolbarContainer,
.findbar,
.secondaryToolbar {
box-shadow: 0 1px 0 var(--toolbar-border-color); box-shadow: 0 1px 0 var(--toolbar-border-color);
} }
@ -494,58 +485,42 @@ html[dir="rtl"] .findbar {
right: 64px; right: 64px;
} }
.findbar .splitToolbarButton {
margin-top: 3px;
}
html[dir="ltr"] .findbar .splitToolbarButton { html[dir="ltr"] .findbar .splitToolbarButton {
margin-left: 0px; margin-left: 0px;
margin-top: 3px; margin-right: 5px;
} }
html[dir="rtl"] .findbar .splitToolbarButton { html[dir="rtl"] .findbar .splitToolbarButton {
margin-left: 5px;
margin-right: 0px; margin-right: 0px;
margin-top: 3px;
} }
.findbar .splitToolbarButton .findNext { .findbar .splitToolbarButton > .toolbarButton {
width: 29px;
}
html[dir="ltr"] .findbar .splitToolbarButton .findNext {
border-right: 1px solid var(--field-border-color);
}
html[dir="rtl"] .findbar .splitToolbarButton .findNext {
border-left: 1px solid var(--field-border-color);
}
.findbar .splitToolbarButton .toolbarButton {
background-color: var(--findbar-nextprevious-btn-bg-color); background-color: var(--findbar-nextprevious-btn-bg-color);
border-radius: 0px; border-radius: 0;
height: 26px; height: 26px;
border-top: 1px solid var(--field-border-color); border-top: 1px solid var(--field-border-color);
border-bottom: 1px solid var(--field-border-color); border-bottom: 1px solid var(--field-border-color);
} }
.findbar .splitToolbarButton .toolbarButton::before { .findbar .splitToolbarButton > .toolbarButton::before {
top: 5px; top: 5px;
} }
html[dir="ltr"] .findbar .splitToolbarButton > .findPrevious { .findbar .splitToolbarButton > .findNext {
border-radius: 0; width: 29px;
} }
html[dir="ltr"] .findbar .splitToolbarButton > .findNext { html[dir="ltr"] .findbar .splitToolbarButton > .findNext {
border-bottom-left-radius: 0;
border-bottom-right-radius: 2px; border-bottom-right-radius: 2px;
border-top-left-radius: 0;
border-top-right-radius: 2px; border-top-right-radius: 2px;
} border-right: 1px solid var(--field-border-color);
html[dir="rtl"] .findbar .splitToolbarButton > .findPrevious {
border-radius: 0;
} }
html[dir="rtl"] .findbar .splitToolbarButton > .findNext { html[dir="rtl"] .findbar .splitToolbarButton > .findNext {
border-bottom-left-radius: 2px; border-bottom-left-radius: 2px;
border-bottom-right-radius: 0;
border-top-left-radius: 2px; border-top-left-radius: 2px;
border-top-right-radius: 0; border-left: 1px solid var(--field-border-color);
} }
.findbar input[type="checkbox"] { .findbar input[type="checkbox"] {
@ -565,7 +540,6 @@ html[dir="ltr"] #findInput {
border-top-right-radius: 0; border-top-right-radius: 0;
border-bottom-right-radius: 0; border-bottom-right-radius: 0;
} }
html[dir="rtl"] #findInput { html[dir="rtl"] #findInput {
border-top-left-radius: 0; border-top-left-radius: 0;
border-bottom-left-radius: 0; border-bottom-left-radius: 0;
@ -737,11 +711,9 @@ html[dir="rtl"] .findbar * {
display: inline-block; display: inline-block;
} }
html[dir="ltr"] .splitToolbarButton > .toolbarButton { html[dir="ltr"] .splitToolbarButton > .toolbarButton {
border-radius: 2px;
float: left; float: left;
} }
html[dir="rtl"] .splitToolbarButton > .toolbarButton { html[dir="rtl"] .splitToolbarButton > .toolbarButton {
border-radius: 2px;
float: right; float: right;
} }
@ -792,12 +764,10 @@ html[dir="rtl"] .splitToolbarButton > .toolbarButton {
} }
html[dir="ltr"] .splitToolbarButton > .toolbarButton:first-child, html[dir="ltr"] .splitToolbarButton > .toolbarButton:first-child,
html[dir="rtl"] .splitToolbarButton > .toolbarButton:last-child { html[dir="rtl"] .splitToolbarButton > .toolbarButton:last-child {
position: relative;
margin: 0; margin: 0;
} }
html[dir="ltr"] .splitToolbarButton > .toolbarButton:last-child, html[dir="ltr"] .splitToolbarButton > .toolbarButton:last-child,
html[dir="rtl"] .splitToolbarButton > .toolbarButton:first-child { html[dir="rtl"] .splitToolbarButton > .toolbarButton:first-child {
position: relative;
margin: 0; margin: 0;
} }
.splitToolbarButtonSeparator { .splitToolbarButtonSeparator {
@ -932,20 +902,6 @@ html[dir="rtl"] .dropdownToolbarButton > select {
height: 1px; height: 1px;
} }
html[dir="ltr"] #findPrevious {
margin-left: 0;
}
html[dir="ltr"] #findNext {
margin-right: 3px;
}
html[dir="rtl"] #findPrevious {
margin-right: 0;
}
html[dir="rtl"] #findNext {
margin-left: 3px;
}
.toolbarButton::before, .toolbarButton::before,
.secondaryToolbarButton::before, .secondaryToolbarButton::before,
.dropdownToolbarButton::after, .dropdownToolbarButton::after,