-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding Material Dashboard bootstrap template #2
Conversation
index.html
Outdated
</a> | ||
</li> | ||
<li class="nav-item"> | ||
<a class="nav-link text-white " href="about.html"> |
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.
Should we include a blank about.html page or take out this ref since it currently doesn't exist?
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.
I added it more as a reference for having another manu item, but I'll remove to avoid confusion :)
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.
Yeah I figured hahah no worries in that case :)
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.
Other than the small comment I made, I think this looks good to me as the initial UI template! I made some tickets in clickup as some follows ups that seem like logical next steps to take on for us:
- Port Project Template to Next.js front end
- Figure out static checks for JavaScript or TypeScript portion of the app
Feel free to modify these or change them if you do not agree :)
I had a ticket for the first one already (https://app.clickup.com/t/86877agzf) so I'll close it, but the second one is a great idea. Thanks! |
My bad I missed that one cause you already had it in progress! |
PR Type
Feature
Short Description
Clickup Ticket(s): https://app.clickup.com/t/86877ahk0
Creating a basic template based on https://www.creative-tim.com/product/material-dashboard#
Currently looking like this:
Tests Added
NA