👋 Hello, codeandcoffeelb! I know @karimamer is assigned to the original issue, so I'm happy to yield to their changes and you're more than welcome to close this out.
creates a YAML file in the _data defining the navigation items
updates includes/navigation.html to use one conditional to omit the home page at root and a for iterator to render all navigation items
I tested my changes locally by spinning up the Dockerfile defined in this repo to ensure what I've proposed is functional. Here's a motion picture, if it helps!
Next steps
Requesting review from @davidnuon as the maintainer of this organization (please correct me if I'm wrong 🙇 ). I've allowed edits from maintainers in the case there's anything you'd like to modify. I'm open to receive any feedback about my implementation––I'm not tied down to anything in particular. Ultimately, I'd just like to see Navigation fixed for your site! 😎
Closes https://github.com/codeandcoffeelb/codeandcoffeelb.github.io/issues/95.
Summary
👋 Hello, codeandcoffeelb! I know @karimamer is assigned to the original issue, so I'm happy to yield to their changes and you're more than welcome to close this out.
Approach
This is my take drawing from the approach recommended in https://jekyllrb.com/tutorials/navigation/:
_datadefining the navigation itemsincludes/navigation.htmlto use one conditional to omit the home page at root and aforiterator to render all navigation itemsI tested my changes locally by spinning up the
Dockerfiledefined in this repo to ensure what I've proposed is functional. Here's a motion picture, if it helps!Next steps
Requesting review from @davidnuon as the maintainer of this organization (please correct me if I'm wrong 🙇 ). I've allowed edits from maintainers in the case there's anything you'd like to modify. I'm open to receive any feedback about my implementation––I'm not tied down to anything in particular. Ultimately, I'd just like to see Navigation fixed for your site! 😎