- Notifications
You must be signed in to change notification settings - Fork0
Cw2-18 Improve Semantic HTML#7
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
* added events template* removed section* modified events to coordinate with navbar on click---------Co-authored-by: Stuart Cutbush <stuart@Stuarts-MacBook-Pro.local>Co-authored-by: Derek Xu <xuderek2004@gmail.com>
derekxu04 left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Cool, not too complicated.
Just make sure the pipeline is passing, will only take a second, you can runnpm run lint:fix for example for automatic linting
QuadAces commentedApr 17, 2024
Alrighty, should work now, prettier just wanted the code formatted :P |
derekxu04 left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
lgtm
* changed the h1 tags* changed image alt tags* changed tailwind* CW2-3 Add events section (#4)* added events template* removed section* modified events to coordinate with navbar on click---------Co-authored-by: Stuart Cutbush <stuart@Stuarts-MacBook-Pro.local>Co-authored-by: Derek Xu <xuderek2004@gmail.com>* changes to make linter approve* formatted code?---------Co-authored-by: Stuart Cutbush <stuart@Stuarts-MacBook-Pro.local>Co-authored-by: Derek Xu <xuderek2004@gmail.com>
Why the changes are required?
For SEO & Accessibility
Changes
Screenshots
No UI changes
Comments
No comments