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

Unified Diff: static/scss/layout/_header.scss

Issue 29485575: Issue 5385 - Create Site Header Component for Help Center (Closed)
Patch Set: Implement navbar-collapse Created Aug. 25, 2017, 8:25 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/layout/_header.scss
===================================================================
new file mode 100644
--- /dev/null
+++ b/static/scss/layout/_header.scss
@@ -0,0 +1,74 @@
+// 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/>.
+
+#site-header
+{
+ min-height: $site-header-height;
+ font-size: $small-font;
+ line-height: $site-header-content-line-height;
+}
+
+#site-title
+{
+ float: left;
+ margin-right: $xl;
+}
+
+#site-title img
+{
+ height: $lg - 5px;
juliandoucette 2017/08/29 16:17:31 NIT: It seem strange to shrink the logo and not th
ire 2017/09/01 10:57:23 You're right. I've kept the logo the same size. Di
+ vertical-align: middle;
+
+ @media (min-width: $tablet-breakpoint)
+ {
+ height: $lg;
+ }
+}
+
+.toggle-navbar-collapse
+{
+ width: $site-header-content-line-height;
+ float: right;
+ line-height: $site-header-content-line-height;
+ text-align: right;
+ cursor: pointer;
+
+ img
+ {
+ width: $md;
+ vertical-align: middle;
+ }
+}
+
+html[dir="rtl"]
+{
+ #site-title
juliandoucette 2017/08/29 16:17:32 NIT: Move rtl #site-title above .toggle-navbar-col
ire 2017/09/01 10:57:24 Done.
+ {
+ float: right;
+ margin-right: 0;
+ margin-left: $xl;
+ }
+
+ .toggle-navbar-collapse
+ {
+ float: left;
+ }
+}
+
+.no-js .toggle-navbar-collapse
+{
+ display: none;
+}

Powered by Google App Engine
This is Rietveld