-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix/adjust navigation bar and font size in layout #239
Fix/adjust navigation bar and font size in layout #239
Conversation
✅ Deploy Preview for openstartervillage-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for openstartervillage ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Please remove the unused CSS instead of comment them out. |
It's very good to have dynamic navigation links. 🥳 Nice work! May I know your reason not to replace the hard code approach? |
Yes, Sir! |
I think it will be better to wait until the content for the resource page is completely updated. |
1. Add filter to check if the path and name in each page are empty 2. Add a hard-coded link to cards page 3. Remove the link to activities page 4. Replace page navigation list with dynamic page navigation list
There are some duplicate lines of code on generating the navigation list. Please create a shared function and reuse it. |
Good work!!! |
Description
minor
Impaction
Screenshots
Performance
Build and deploy time
N/A
LightHouse
N/A