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

Issues on Foundations | Personality #72

Open
1 of 2 tasks
zeroedin opened this issue Jul 17, 2023 · 1 comment
Open
1 of 2 tasks

Issues on Foundations | Personality #72

zeroedin opened this issue Jul 17, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@zeroedin
Copy link
Collaborator

zeroedin commented Jul 17, 2023

Couple issues I noticed on the personality page:

  • Fixed in Fix cta links #75 CTA looks like the HTML has some errors tag closed before contents of the slotted <a>
  • When two rhb-grid are used in the tab panel looks like they might need a sibling selector to add margin between them:
rhb-grid + rhb-grid {
  margin-block-start: var(--rh-space-2xl, 32px);
}

Need to evaluate the rest of the site looking for the same pattern and see if that is what we need to do here, or a more custom rule.

@zeroedin zeroedin added the bug Something isn't working label Jul 17, 2023
@zeroedin zeroedin added this to the Production Launch milestone Jul 17, 2023
@amandapearlswig
Copy link
Contributor

CTA has been fixed, will look into adding the rbh-grid spacing suggestion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants