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

Side by Side Diff: static/scss/components/_search-form.scss

Issue 29485575: Issue 5385 - Create Site Header Component for Help Center (Closed)
Patch Set: Unfix header, implement standard spacing units Created Aug. 16, 2017, 2:37 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // This file is part of help.eyeo.com.
2 // Copyright (C) 2017 Eyeo GmbH
3 //
4 // help.eyeo.com is free software: you can redistribute it and/or modify
5 // it under the terms of the GNU General Public License as published by
6 // the Free Software Foundation, either version 3 of the License, or
7 // (at your option) any later version.
8 //
9 // help.eyeo.com is distributed in the hope that it will be useful,
10 // but WITHOUT ANY WARRANTY; without even the implied warranty of
11 // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 // GNU General Public License for more details.
13 //
14 // You should have received a copy of the GNU General Public License
15 // along with help.eyeo.com. If not, see <http://www.gnu.org/licenses/>.
16
17 #site-search
18 {
19 display: none;
20 clear: both;
21
22 &.open
23 {
24 display: block;
25 }
26
27 @media (min-width: $tablet-breakpoint)
28 {
29 display: block;
30 clear: none;
31 width: auto;
32 }
33 }
34
35 .no-js #site-search
36 {
37 display: block;
38 }
39
40 // Product Website Link
41
42 #product-website-link
43 {
44 float: right;
45 }
46
47 #product-website-link img
48 {
49 height: 1em;
50 margin-left: 0.5em;
51 }
52
53 #site-search.open #product-website-link,
54 .no-js #product-website-link
55 {
56 float: none;
57
58 @media (min-width: $mobile-breakpoint)
59 {
60 float: right;
61 margin-top: 20px;
62 }
63
64 @media (min-width: $tablet-breakpoint)
65 {
66 margin-top: 0;
67 }
68 }
69
70 // Search Form
71
72 #search-form
73 {
74 display: inline-block;
75 position: relative;
76 overflow: hidden;
77 width: 100%;
78 height: $search-form-height;
79 border-radius: 3px;
80 color: $gray;
81 background: #fff;
82 line-height: 1;
83 vertical-align: middle;
84
85 @media (min-width: $mobile-breakpoint)
86 {
87 max-width: $search-form-width;
88 }
89 }
90
91 #site-search.open #search-form,
92 .no-js #search-form
93 {
94 margin-top: 20px;
95
96 @media (min-width: $tablet-breakpoint)
97 {
98 margin-top: 0;
99 }
100 }
101
102 #search-form input
103 {
104 width: 100%;
105 height: $search-form-height - 2px;
juliandoucette 2017/08/18 15:03:57 Why -2px?
ire 2017/08/21 15:23:22 It was slightly misaligned if I used the same heig
juliandoucette 2017/08/21 16:28:40 This seems to ~cause the placeholder text alignmen
ire 2017/08/22 12:15:16 Oh okay, I've removed it. Doesn't seem to cause a
106 padding-right: $search-form-height + 5px;
juliandoucette 2017/08/18 15:03:57 NIT: It seems strange to calculate an x value usin
ire 2017/08/21 15:23:22 This padding is used to create an exact square aro
juliandoucette 2017/08/21 16:28:40 Acknowledged.
107 padding-left: 10px;
108 border: 0;
109 font-family: $primary-font;
110 font-size: $small-font;
111 }
112
113 #search-form button
114 {
115 position: absolute;
116 top: 0;
117 right: 0;
118 width: $search-form-height;
119 height: 100%;
120 border: 0;
121 background-color: transparent;
122 }
123
124 #search-form button img
125 {
126 height: 15px;
juliandoucette 2017/08/18 15:03:56 NIT: Shouldn't this be an em value to match the fo
ire 2017/08/21 15:23:22 Yes you're right. Done.
127 }
128
129 html[dir="rtl"]
130 {
131 #search-form input
132 {
133 padding-right: 10px;
134 padding-left: $search-form-height + 5px;
135 }
136
137 #search-form button
138 {
139 right: auto;
140 left: 0;
141 }
142
143 #product-website-link
144 {
145 float: left;
146 }
147
148 #product-website-link img
149 {
150 margin-right: 0.5em;
151 margin-left: 0;
152 }
153 }
OLDNEW

Powered by Google App Engine
This is Rietveld