Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#69] Styling the breadcrumbs #81

Merged

Conversation

nathan-schmidt-viget
Copy link
Contributor

Summary

Styles the breadcrumbs with TW

Issues

Testing Instructions

  1. Make sure the breadcrumbs look like the screenshots.

Screenshots

Rank Math

Screenshot 2024-05-21 at 2 40 50 PM

Default WP

Screenshot 2024-05-21 at 2 40 34 PM

Comment on lines +4 to +10
p {
@apply flex flex-wrap gap-8;
}

a {
@apply text-sky-700 underline hover:no-underline focus:no-underline;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a fan of just tag elements but that is all Rank Math gives us 😞

@nathan-schmidt-viget nathan-schmidt-viget self-assigned this May 21, 2024
Copy link
Contributor

@bd-viget bd-viget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM!

@nathan-schmidt-viget nathan-schmidt-viget merged commit 7995fe2 into bd/69-page-header-component May 22, 2024
@nathan-schmidt-viget nathan-schmidt-viget deleted the ns/69-page-header-style branch May 22, 2024 18:59
bd-viget pushed a commit that referenced this pull request May 23, 2024
bd-viget added a commit that referenced this pull request May 23, 2024
* [#69] Page Header Component

* [#69] Vertical Alignment

* [#69] Column top margin

* [#69] Added RankMath & Wordfence

* [#69] Added breadcrumb block

* [#69] Styling the breadcrumbs (#81)

---------

Co-authored-by: Nathan Schmidt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants