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

offer theme #1236

Merged
merged 5 commits into from
Dec 6, 2024
Merged

offer theme #1236

merged 5 commits into from
Dec 6, 2024

Conversation

SallyMcGrath
Copy link
Member

@SallyMcGrath SallyMcGrath commented Dec 6, 2024

(i had broken the count in the summary divs but now I fixed)

things i changed

I moved the scripts into the assets file so they can be piped
I made a default layout
I fixed all the broken HTML
I added a very cursory BEM, we would improve this later
I fixed the improper nesting of the list items
Then I realised I'd broken the counts so I dropped your divs back in until we can do a nice/polishing pass

SallyMcGrath and others added 4 commits December 6, 2024 21:35
rm import
fix improper li nesting
break counts apparently
feel free to move around/rename
I can imagine wanting a "reviews" dashboard that displays broader
information in the future, e.g. things that have been reviewed.
1. move selection to [data-attributes]
1. mark up with BEM
@illicitonion illicitonion merged commit af9f095 into review-dashboard Dec 6, 2024
2 checks passed
@illicitonion illicitonion deleted the theme-dashboard branch December 6, 2024 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants