Skip to content

Commit dee5f6e

Browse files
Clean comments and fix spacing
1 parent 9de9e96 commit dee5f6e

File tree

3 files changed

+12
-5
lines changed

3 files changed

+12
-5
lines changed

assets/sass/protocol/base/elements/_links.scss

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ a:where(:visited) {
3232
--link-color: var(--link-color-inverse);
3333
--link-color-hover: var(--link-color-hover-inverse);
3434
--link-color-visited: var(--link-color-visited-inverse);
35-
--link-color-visited-hover:var(--link-color-visited-hover-inverse);
35+
--link-color-visited-hover: var(--link-color-visited-hover-inverse);
3636
}
3737

3838
// any descendant link will inherit white theming (replace white-links mixin)
@@ -44,9 +44,7 @@ a:where(:visited) {
4444
--link-color-visited-hover: #{$color-white};
4545
}
4646

47-
// this might belong in a different CTA file?
48-
// it's not meant to apply to `a` elements unless they have the class added directly
49-
// the other classes in this file are theme containers that select `a` elements inside them
47+
// apply directly to link
5048
.mzp-c-cta-link {
5149
font-family: var(--button-font-family);
5250
font-weight: bold;

assets/sass/protocol/components/_button.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@
7676
// * -------------------------------------------------------------------------- */
7777
// Button shape and size
7878

79-
.mzp-c-button /* stylelint-disable-line no-duplicate-selectors */ {
79+
.mzp-c-button {
8080
@include border-box;
8181
@include font-size(16px);
8282
@include transition(background-color 100ms, box-shadow 100ms, color 100ms);

components/button/button.config.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,55 @@
11
context:
22
label: Primary Button
3+
link: './button'
34
default: Primary
45
variants:
56
- name: Primary Dark
67
preview: '@preview-dark'
78
notes: Add the class `mzp-t-dark` for use on dark backgrounds (the button itself is light).
9+
link: './button'
810
context:
911
class: 'mzp-t-dark'
1012
- name: Secondary
1113
notes: A less prominent button with a hollow, transparent background. Use this for any additional actions someone can take on any given page that is different from the primary action.
14+
link: './button'
1215
context:
1316
class: 'mzp-t-secondary'
1417
label: Secondary Button
1518
- name: Secondary Dark
1619
preview: '@preview-dark'
20+
link: './button'
1721
notes: A light variation of the secondary button for dark backgrounds, adding the class `mzp-t-dark` (the button itself is light).
1822
context:
1923
class: 'mzp-t-secondary mzp-t-dark'
2024
label: Secondary Button
2125
- name: Product
26+
link: './button'
2227
notes: Use this theme style for product-related transactions, such as downloading software or creating an account.
2328
context:
2429
class: 'mzp-t-product'
2530
label: Download Firefox
2631
- name: Product Secondary
32+
link: './button'
2733
notes: This secondary product button theme should be used when the CTA is a product-related transaction, but not the main action a person should take on a page.
2834
context:
2935
class: 'mzp-t-product mzp-t-secondary'
3036
label: Download Firefox
3137
- name: Product Secondary Dark
38+
link: './button'
3239
preview: '@preview-dark'
3340
notes: A light variation of the secondary product button for use on dark backgrounds. Add the class `mzp-t-dark` (the button itself is light).
3441
context:
3542
class: 'mzp-t-product mzp-t-secondary mzp-t-dark'
3643
label: Download Firefox
3744
- name: Neutral
3845
notes: A neutral button for less important actions.
46+
link: './button'
3947
context:
4048
class: 'mzp-t-neutral'
4149
label: Neutral Button
4250
- name: Neutral Dark
4351
preview: '@preview-dark'
52+
link: './button'
4453
notes: A light variation of the neutral button for use on dark backgrounds. Add the class `mzp-t-dark` (the button itself is light).
4554
context:
4655
class: 'mzp-t-neutral mzp-t-dark'

0 commit comments

Comments
 (0)