-
Notifications
You must be signed in to change notification settings - Fork 31
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
Restructure the homepage code #387
Conversation
@YoshithaRathnayake The aim of the mentioned issue is to create a better code structure. You can refer the material kit starter template for that.
And there are some custom styles that we could directly add using bootstrap classes without using custom CSS. |
I think that one is making from this pr #388 @Piumal1999 |
Now @Piumal1999 |
Is that the way wanted |
index.html
Outdated
<!-- End of Resource Section --> |
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.
The end of line is missing
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.
Now @anjula-sack
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.
Thank you! @YoshithaRathnayake
Purpose
The purpose of this PR is to fix #358
Goals
Restructuring the homepage code
Approach
I restructured the homepage code
Screenshots
No screenshots
Preview Link
https://yoshitharathnayake.github.io/sack-site/
Checklist
Related PRs
Test environment
Learning