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

Update theme with the one from the style guide #37

Closed
1 task done
Tracked by #22
wilwade opened this issue Oct 28, 2024 · 0 comments · Fixed by #38
Closed
1 task done
Tracked by #22

Update theme with the one from the style guide #37

wilwade opened this issue Oct 28, 2024 · 0 comments · Fixed by #38

Comments

@wilwade
Copy link
Collaborator

wilwade commented Oct 28, 2024

  • Remove extras from the tailwind config and app.css
@wilwade wilwade mentioned this issue Oct 28, 2024
8 tasks
@wilwade wilwade assigned wilwade and claireolmstead and unassigned wilwade Oct 28, 2024
@claireolmstead claireolmstead linked a pull request Oct 28, 2024 that will close this issue
4 tasks
claireolmstead added a commit that referenced this issue Oct 28, 2024
# Description

Update theme to use the styleguide.

- Closes #37 

## Type of change

- [x] New Feature

# How to Test?

Pull branch and see that styles look the same and are not broken in any
screen size.

1. Pull and run locally
2. See that css is coming through as well as the theme. Best examples
are the underline on hover in the nav (showing css) and the colors
properly coming through (tailwind config)

# Checklist:

- [x] I have performed a self-review of my code
- [x] I have commented my code & PR, particularly in hard-to-understand
areas
- [x] I have checked at all the breakpoints to make sure it works on all
screen sizes

---------

Co-authored-by: Claire Olmstead <olmsteadclaire@gmail.com>
@wilwade wilwade added this to the Developer portal updates milestone Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants