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

Side by Side Diff: scss/_button-links.scss

Issue 29365594: Issue 4633 - Default form styles (Closed)
Patch Set: Created Nov. 29, 2016, 4:29 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
« no previous file with comments | « no previous file | scss/_forms.scss » ('j') | scss/_forms.scss » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*!
2 * This file is part of website-defaults
3 * Copyright (C) 2016 Eyeo GmbH
4 *
5 * website-defaults is free software: you can redistribute it and/or
6 * modify it under the terms of the GNU General Public License as published by
7 * the Free Software Foundation, either version 3 of the License, or
8 * (at your option) any later version.
9 *
10 * website-defaults is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with website-defaults. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19 /*******************************************************************************
20 * Button links
juliandoucette 2017/01/10 13:40:26 TODO: - change to button.scss (move native button
21 ******************************************************************************/
22
23 .button,
24 .button:visited
25 {
26 display: inline-block;
27 padding: 0em 0.5em;
28 border: 1px solid $primary;
29 color: $primary;
30 background-color: $primary-light;
31 line-height: 2em;
saroyanm 2016/12/06 18:26:53 Detail: I think we decided to use line-heights wit
juliandoucette 2016/12/07 17:31:07 Acknowledged. I will change this.
32 cursor: pointer;
33 }
34
35 .button:hover,
36 .button:active,
37 .button:focus
38 {
39 border-color: $accent;
40 color: $accent;
41 text-decoration: none;
42 }
43
44 .submit,
saroyanm 2016/12/06 18:26:53 Weak suggrestion: I think we can move this style t
juliandoucette 2016/12/07 17:31:07 Acknowledged. Some websites may have no forms or
45 .submit:visited
46 {
47 border-color: $primary;
48 color: $primary-light;
49 background-color: $primary;
50 }
51
52 .submit:hover,
53 .submit:active,
54 .submit:focus
55 {
56 border-color: $accent;
57 color: $primary-light;
58 background-color: $accent;
59 }
OLDNEW
« no previous file with comments | « no previous file | scss/_forms.scss » ('j') | scss/_forms.scss » ('J')

Powered by Google App Engine
This is Rietveld