Skip to content

Commit

Permalink
feat: themed logos [CFE-75] (#220)
Browse files Browse the repository at this point in the history
* feat: themed logos

* Update README.md

Co-authored-by: Maria Eguiluz <[email protected]>

* feat: add fallback hex code

* Update README.md

---------

Co-authored-by: Maria Eguiluz <[email protected]>
  • Loading branch information
artmsilva and atomictangerine authored Dec 10, 2024
1 parent 5745ba4 commit fb70645
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 4 deletions.
8 changes: 7 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,10 @@
3. Click the "Upload" button
4. Visit Github to see your changes

.
## Theming

Branded logos will use WPDS Theme Colors like so where a CSS property/variable is used with a fallback hex code.

```html
fill="var(--wpds-colors-primary, #111)"
```
4 changes: 2 additions & 2 deletions app/app.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ const App = () => {
{Object.keys(AllAssets).map((Asset) => {
const Component = AllAssets[Asset];
return (
<article key={Asset}>
<h2>{Asset}</h2>
<article key={Asset} href={`#${Asset}`}>
<h2 id={`#${Asset}`}>{Asset}</h2>
<AssetContainer>
<Component />
</AssetContainer>
Expand Down
2 changes: 1 addition & 1 deletion src/washington-post.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit fb70645

Please sign in to comment.