Skip to content

Commit

Permalink
Poetry: Add links in site title and post title (#186)
Browse files Browse the repository at this point in the history
* Poetry: Update header and index links

- Make site title link to home (in header template part).
- Make post title a link (in index template).
- Fix missing quotes in theme.json.
- Update link colors.

* Addressing PR Reviews - Adds links in patterns

- Fix templates header.html and index.html to link to the respective patterns, as it was originally.
- Adds the links in site title and post-title in the respective patterns instead.

* Add the footer back in index template
  • Loading branch information
adrianaleites authored May 21, 2024
1 parent dfce47d commit 8ebac36
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 7 deletions.
2 changes: 1 addition & 1 deletion poetry/patterns/header.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<!-- wp:group {"style":{"spacing":{"padding":{"top":"32px","bottom":"64px"}}},"layout":{"type":"constrained"}} -->
<div class="wp-block-group" style="padding-top:32px;padding-bottom:64px"><!-- wp:group {"style":{"spacing":{"blockGap":"64px"}},"layout":{"type":"flex","flexWrap":"nowrap"}} -->
<div class="wp-block-group"><!-- wp:group {"style":{"spacing":{"blockGap":"88px"}},"layout":{"type":"flex","orientation":"vertical"}} -->
<div class="wp-block-group"><!-- wp:site-title {"level":0,"isLink":false,"style":{"typography":{"fontStyle":"normal","fontWeight":"600"}},"fontSize":"medium"} /-->
<div class="wp-block-group"><!-- wp:site-title {"level":0,"isLink":true,"style":{"typography":{"fontStyle":"normal","fontWeight":"600"}},"fontSize":"medium"} /-->

<!-- wp:group {"style":{"spacing":{"blockGap":"48px"}},"layout":{"type":"flex","orientation":"vertical"}} -->
<div class="wp-block-group"><!-- wp:paragraph {"style":{"typography":{"lineHeight":"0.8"}},"fontSize":"xx-large","fontFamily":"rozha-one"} -->
Expand Down
2 changes: 1 addition & 1 deletion poetry/patterns/posts.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
<figure class="wp-block-image size-full is-resized"><img src="<?php echo esc_url( get_template_directory_uri() ); ?>/assets/images/feather-small.png" alt="" width="56" height="56"/></figure>
<!-- /wp:image -->

<!-- wp:post-title {"style":{"spacing":{"padding":{"right":"24%"}}}} /-->
<!-- wp:post-title {"isLink":true,"style":{"spacing":{"padding":{"right":"24%"}}}} /-->

<!-- wp:group {"style":{"spacing":{"blockGap":"8px"},"elements":{"link":{"color":{"text":"var:preset|color|vivid-red"}}}},"layout":{"type":"flex","flexWrap":"nowrap"}} -->
<div class="wp-block-group has-link-color"><!-- wp:paragraph {"textColor":"custom-contrast"} -->
Expand Down
20 changes: 15 additions & 5 deletions poetry/theme.json
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@
},
"styles": {
"color": {
"background": "var:preset|color|white"
"background": "var(--wp--preset--color--white)"
},
"elements": {
"h1": {
Expand All @@ -131,7 +131,7 @@
"fontSize": "var(--wp--preset--font-size--xx-large)",
"fontStyle": "normal",
"fontWeight": "400",
"lineHeight": 0.8
"lineHeight": "0.8"
}
},
"h2": {
Expand All @@ -145,11 +145,21 @@
"typography": {
"fontFamily": "Rozha One"
}
},
"link": {
":hover": {
"color": {
"text": "var(--wp--preset--color--custom-contrast)"
}
},
"color": {
"text": "var(--wp--preset--color--custom-text)"
}
}
},
"typography": {
"fontFamily": "var:preset|font-family|inter",
"lineHeight": 2
"fontFamily": "var(--wp--preset--font-family--inter)",
"lineHeight": "2"
}
},
"templateParts": [
Expand All @@ -163,5 +173,5 @@
}
],
"version": 2,
"$schema": "https://schemas.wp.org/wp/6.1/theme.json"
"$schema": "https://schemas.wp.org/wp/6.5/theme.json"
}

0 comments on commit 8ebac36

Please sign in to comment.