Skip to content

Commit 5598086

Browse files
committed
Fix to mobile splash bg color and brand colors, remove brand colors from theme system in favor of primary / secondary colors
1 parent 28eab3f commit 5598086

15 files changed

Lines changed: 36 additions & 122 deletions

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,3 +71,6 @@ build/Release
7171
# Python bytecode
7272
__pycache__/
7373
*.py[cod]
74+
75+
# Planning docs
76+
CONTENT_PLANNING*.md

.markdownlint.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
"MD004": false,
33
"MD013": false,
44
"MD025": false,
5+
"MD029": false,
56
"MD031": false,
67
"MD033": false,
78
"MD035": false,

_TODO.md

Lines changed: 18 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,23 @@
11
<!-- markdownlint-disable-file -->
22
# TODO
33

4-
## Refactor E2E Test Suite
5-
6-
### Files with Skipped Tests
4+
## E2E Files with Skipped Tests
75

86
Blocked Categories (44 tests):
97

10-
Visual regression testing (18) - Needs Percy/Chromatic
11-
Lighthouse audits (6) - Integration pending
12-
Newsletter double opt-in (6) - Email testing infrastructure
13-
Axe accessibility (2) - axe-core integration
8+
Visual regression testing (18)
9+
10+
Playwright's Built-in Visual Testing
11+
12+
Playwright has a built-in visual comparison feature using await expect(page).toHaveScreenshot().
13+
14+
How it works: On the first run, Playwright saves a baseline screenshot. Subsequent runs compare the actual screenshot to this baseline, failing the test if there are pixel differences.
1415

15-
### Axe tags
16-
17-
cat.aria: Rules related to Accessible Rich Internet Applications (ARIA) attributes and roles.
18-
cat.color: Rules related to color contrast and meaning conveyed by color.
19-
cat.controls: Rules for interactive controls, such as form elements and links.
20-
cat.forms: Rules specifically for forms, form fields, and their labels.
21-
cat.keyboard: Rules related to keyboard operability.
22-
cat.links: Rules for links, including their names and destinations.
23-
cat.name-role-value: Rules that check if an element has a name, role, and value that can be correctly interpreted by assistive technologies.
24-
cat.semantics: Rules related to the semantic structure of a document, such as headings and landmarks.
25-
cat.sensory-and-visual-cues: Rules that deal with information conveyed by sensory or visual characteristics.
26-
cat.structure: Rules related to the document's overall structure, like the proper nesting of elements.
27-
cat.tables: Rules for data tables, including headers and associations.
28-
cat.text-alternatives: Rules for ensuring that text alternatives are provided for non-text content, such as images.
29-
30-
## Content Redo - Tag Groups
31-
32-
- Platform Engineering
33-
- DevOps
34-
- Kubernetes
35-
- Infrastructure as Code
36-
- Containers
37-
- API Management
38-
- SRE
16+
Pros: It's free, everything stays local (no third-party services needed), setup is simple, and you retain full control over the baseline images in your repository.
17+
18+
Cons: Browser rendering can be inconsistent across different operating systems and machines, leading to "flaky" tests or false positives. Managing baselines for multiple browsers and resolutions manually can also be challenging at scale.
19+
20+
Axe accessibility (2) - axe-core integration
3921

4022
## Search UI
4123

@@ -81,7 +63,7 @@ window.addEventListener('afterprint', (event) => {
8163
})
8264
```
8365

84-
or listen for changes:
66+
Or listen for changes:
8567

8668
```typescript
8769
if (window.matchMedia) {
@@ -97,3 +79,7 @@ if (window.matchMedia) {
9779
});
9880
}
9981
```
82+
83+
Please analyze the document CONTENT_PLANNING_OPUS.md
84+
85+
Please add a suggested set of tag groups that would be appropriate for the content goals of the site, as described in the document. These are top-level content organization categories like "DevOps". Then, sketch out 30+ article ideas organized by the tag group they most relate to (they can fall under more than one tag group).

src/components/Brand/Logo.astro

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -12,20 +12,6 @@ const { class: className } = Astro.props
1212
aria-hidden="true"
1313
focusable="false"
1414
>
15-
<style>
16-
.logo {
17-
position: relative;
18-
z-index: var(--z-content-overlay);
19-
}
20-
21-
.logo-block-outer {
22-
fill: var(--color-content-inverse);
23-
}
24-
25-
.logo-block-inner {
26-
fill: var(--color-primary);
27-
}
28-
</style>
2915
<g id="logo-group" transform="translate(-64.234499,-91.339148)">
3016
<path
3117
class="logo-block-outer"

src/components/Brand/WordmarkBuilders.astro

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,6 @@ const { class: className } = Astro.props
99
aria-hidden="true"
1010
focusable="false"
1111
>
12-
<style>
13-
.wordmark__svg-path {
14-
fill: var(--color-primary-offset);
15-
}
16-
</style>
1712
<path
1813
class="wordmark__svg-path"
1914
d="M 12.384,11.52 C 13.984,10.752 15.2,8.864 15.2,6.688 15.2,3.488 12.768,1.152 9.536,1.152 H 0 v 22.08 h 10.304 c 3.456,0 6.08,-2.528 6.08,-5.952 0,-2.752 -1.632,-5.088 -4,-5.76 z M 8.864,3.808 c 1.92,0 3.328,1.44 3.328,3.36 0,1.92 -1.504,3.36 -3.328,3.36 H 2.848 V 3.808 Z M 9.632,20.576 H 2.848 v -7.52 h 6.784 c 2.144,0 3.744,1.6 3.744,3.744 0,2.144 -1.6,3.776 -3.744,3.776 z"

src/components/Brand/WordmarkWebstack.astro

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,6 @@ const { class: className } = Astro.props
99
aria-hidden="true"
1010
focusable="false"
1111
>
12-
<style>
13-
.wordmark__svg-path {
14-
fill: var(--color-primary-offset);
15-
}
16-
</style>
1712
<path
1813
class="wordmark__svg-path"
1914
d="M 27.936,0.96 22.432,19.168 16.832,0.96 H 14.208 L 8.608,19.168 3.104,0.96 H 0 l 7.04,22.08 h 2.976 L 15.488,5.248 20.96,23.04 h 2.976 l 7.04,-22.08 z"

src/components/Brand/index.astro

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import Logo from '@components/Brand/Logo.astro'
66
import WordmarkBuilders from '@components/Brand/WordmarkBuilders.astro'
77
import WordmarkWebstack from '@components/Brand/WordmarkWebstack.astro'
8+
import './index.css'
89
---
910

1011
<a
@@ -14,7 +15,7 @@ import WordmarkWebstack from '@components/Brand/WordmarkWebstack.astro'
1415
aria-label="Webstack Builders home"
1516
>
1617
<span class="brand-logo">
17-
<Logo class="h-(--header-icon-size) w-(--header-icon-size)" />
18+
<Logo class="h-(--header-icon-size) w-(--header-icon-size) relative z-(--z-content-overlay)" />
1819
</span>
1920
<span class="flex flex-col justify-between items-center ml-2 sm:ml-3">
2021
<span class="brand-wordmark-webstack">

src/components/Brand/index.css

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
.logo-block-outer {
2+
fill: var(--color-primary);
3+
}
4+
5+
.logo-block-inner {
6+
fill: var(--color-secondary);
7+
}
8+
9+
.wordmark__svg-path {
10+
fill: var(--color-primary);
11+
}

src/components/Navigation/menu.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@
120120
}
121121

122122
:global(.mobile-splash::after) {
123-
background-color: var(--color-page-base);
123+
background-color: var(--color-primary);
124124
border-radius: 50%;
125125
content: '';
126126
height: 300vmax;

src/styles/print.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@
1717
--theme-color-trim: var(--a11y-theme-color-trim);
1818
--theme-color-spotlight: var(--a11y-theme-color-spotlight);
1919
--theme-color-spotlight-offset: var(--a11y-theme-color-spotlight-offset);
20-
--theme-color-primary-brand: var(--a11y-theme-color-primary-brand);
21-
--theme-color-secondary-brand: var(--a11y-theme-color-secondary-brand);
2220
--theme-color-success: var(--a11y-theme-color-success);
2321
--theme-color-success-offset: var(--a11y-theme-color-success-offset);
2422
--theme-color-success-inverse: var(--a11y-theme-color-success-inverse);

0 commit comments

Comments
 (0)