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

Unified Diff: static/scss/base/_utilities.scss

Issue 29485575: Issue 5385 - Create Site Header Component for Help Center (Closed)
Patch Set: Fix alignment of search form, reformat JS Created Aug. 22, 2017, 12:14 p.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/base/_utilities.scss
===================================================================
new file mode 100644
--- /dev/null
+++ b/static/scss/base/_utilities.scss
@@ -0,0 +1,88 @@
+// 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/>.
+
+// Screen reader ///////////////////////////////////////////////////////////////
+
+// Use this class to add screen reader / web crawler only content
+//
+// EG:
+// - An h1 for SEO that is not visible
+// - A skip-to link for screen readers
+
+.sr-only
+{
+ position: absolute;
+ overflow: hidden;
+ clip: rect(0, 0, 0, 0);
+ width: 1px;
+ height: 1px;
+ margin: -1px;
+ padding: 0;
+ border: 0;
+}
+
+// Clearfix ///////////////////////////////////////////////////////////////
+
+.clearfix:after
+{
+ display: table;
+ clear: both;
+ content: "";
+}
+
+// Responsive ///////////////////////////////////////////////////////////////
juliandoucette 2017/08/29 16:17:23 It may be worth proposing that we rename $size-bre
+
+.mobile-only
+{
+ @media (min-width: $mobile-breakpoint)
juliandoucette 2017/08/23 14:39:05 See comment [here](https://codereview.adblockplus.
ire 2017/08/25 08:27:08 Acknowledged.
juliandoucette 2017/08/29 16:17:24 Note: This should be min-width $tablet-breakpoint
ire 2017/09/01 10:57:18 Done.
+ {
+ display: none;
+ }
+}
+
+.tablet-and-mobile-only
juliandoucette 2017/08/23 14:39:05 The website-default grid considers anything less t
ire 2017/08/25 08:27:08 Ack. See my reply on the quoted comment.
juliandoucette 2017/08/29 16:17:23 On 2017/08/25 08:27:08, ire wrote: Ack. See my rep
ire 2017/09/01 10:57:17 This is changed to .phablet-and-mobile-only, width
+{
+ @media (min-width: $tablet-breakpoint)
+ {
+ display: none;
+ }
+}
+
+.tablet-and-desktop-only
+{
+ @media (max-width: $mobile-breakpoint)
+ {
+ display: none;
+ }
+}
+
+.desktop-only
juliandoucette 2017/08/23 14:39:05 The website-default grid considers anything more t
ire 2017/08/25 08:27:09 Ack. See my reply on the quoted comment.
juliandoucette 2017/08/29 16:17:24 Note: This should be min-width: $desktop-breakpoin
ire 2017/09/01 10:57:17 This is max-width: $desktop-breakpoint, because I'
juliandoucette 2017/09/01 11:43:44 Acknowledged. But it should be $tablet-breakpoint
ire 2017/09/01 12:12:03 Done.
+{
+ @media (max-width: $tablet-breakpoint)
+ {
+ display: none;
+ }
+}
+
+// Unstyled ///////////////////////////////////////////////////////////////
+
+.unstyled
+{
+ margin: 0;
+ padding: 0;
+ border: 0;
+ background: none;
+}

Powered by Google App Engine
This is Rietveld