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

Unified Diff: static/scss/_navbar.scss

Issue 29587584: Issue 5635 - Implement website-default navbar component (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Demo of both methods Created Nov. 6, 2017, 8:18 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
Index: static/scss/_navbar.scss
===================================================================
new file mode 100644
--- /dev/null
+++ b/static/scss/_navbar.scss
@@ -0,0 +1,231 @@
+// This file is part of help.eyeo.com.
+// Copyright (C) 2017 Eyeo GmbH
+//
+// help.eyeo.com is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// help.eyeo.com is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with help.eyeo.com. If not, see <http://www.gnu.org/licenses/>.
+
+/*******************************************************************************
+ * Navbar component (Table Layout Method)
+ ******************************************************************************/
+
+#table-navbar .navbar .container
+{
+ display: table;
+ width: 100%;
+ max-width: $large-desktop-width;
+}
+
+#table-navbar .navbar button
juliandoucette 2017/11/06 12:36:43 NIT: Perhaps button should normally inherit color?
ire 2017/11/08 17:14:46 Done.
+{
+ color: inherit;
+}
+
+#table-navbar .navbar-branding,
+#table-navbar .toggle-navbar-collapse,
+#table-navbar .navbar-collapse
+{
+ display: table-cell;
+ padding-top: $small-space;
+ padding-bottom: $small-space;
+ vertical-align: middle;
+}
+
+/* .navbar .toggle-navbar-collapse
+****************************************************************************/
+
+#table-navbar .toggle-navbar-collapse
+{
+ display: none;
+}
+
+.js #table-navbar .toggle-navbar-collapse
+{
+ display: table-cell;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: none;
+ }
+}
+
+#table-navbar .toggle-navbar-collapse button
juliandoucette 2017/11/06 12:36:43 TOL: Perhaps we could use (extend) a (link button
ire 2017/11/08 17:14:46 Done.
+{
+ min-width: 100px;
+ border: 0;
+ background-color: transparent;
+ cursor: pointer;
+}
+
+.js #table-navbar .toggle-navbar-collapse .close-label
+{
+ display: none;
+}
+
+#table-navbar .expanded .toggle-navbar-collapse .open-label
+{
+ display: none;
+}
+
+#table-navbar .expanded .toggle-navbar-collapse .close-label
+{
+ display: inline;
+}
+
+/* .navbar .navbar-collapse
+****************************************************************************/
+
+#table-navbar .navbar-collapse
+{
+ display: table-row;
+ text-align: left;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: table-cell;
+ text-align: right;
+ }
+}
+
+[dir="rtl"] #table-navbar .navbar-collapse
+{
+ text-align: right;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ text-align: left;
+ }
+}
+
+.js #table-navbar .navbar-collapse
+{
+ display: none;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: table-cell;
+ }
+}
+
+#table-navbar .expanded .navbar-collapse
+{
+ display: table-row;
+}
+
+.no-js #table-navbar .navbar-collapse:after,
+#table-navbar .expanded .navbar-collapse:after
+{
+ @media (max-width: $tablet-breakpoint + 1px)
+ {
+ display: block;
+ height: $small-space;
+ content: "";
+ }
+}
+
+
+/*******************************************************************************
+ * Navbar component (Float Method)
+ ******************************************************************************/
+
+#floated-navbar .navbar
+{
+ @extend .clearfix;
+}
+
+#floated-navbar .navbar button
+{
+ border: 0;
+ color: inherit;
+ background-color: transparent;
+ cursor: pointer;
+}
+
+#floated-navbar .navbar-branding,
+#floated-navbar .toggle-navbar-collapse,
+#floated-navbar .navbar-collapse
+{
+ padding-top: $small-space;
+ padding-bottom: $small-space;
+}
+
+/* .navbar .toggle-navbar-collapse
+ ****************************************************************************/
+
+#floated-navbar .toggle-navbar-collapse
+{
+ display: none;
+ border: 0;
+ background-color: transparent;
+ cursor: pointer;
+}
+
+.js #floated-navbar .toggle-navbar-collapse
+{
+ display: block;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: none;
+ }
+}
+
+.js #floated-navbar .toggle-navbar-collapse .close-label
+{
+ display: none;
+}
+
+#floated-navbar .expanded .toggle-navbar-collapse .open-label
+{
+ display: none;
+}
+
+#floated-navbar .expanded .toggle-navbar-collapse .close-label
+{
+ display: inline;
+}
+
+/* .navbar .navbar-collapse
+ ****************************************************************************/
+
+#floated-navbar .navbar-collapse
+{
+ clear: both;
+ padding-top: 0;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ float: right;
+ clear: none;
+ padding-top: $small-space;
+ }
+}
+
+[dir="rtl"] #floated-navbar .navbar-collapse
+{
+ float: left;
+}
+
+.js #floated-navbar .navbar-collapse
+{
+ display: none;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: block;
+ }
+}
+
+#floated-navbar .expanded .navbar-collapse
+{
+ display: block;
+}
« static/js/navbar.js ('K') | « static/js/navbar.js ('k') | static/scss/_utilities.scss » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld