-
Notifications
You must be signed in to change notification settings - Fork 28
Refactor main navigation #146
Comments
@danhannigan is this project still participating in hacktoberfest? Either way, I would like to take a crack at refactoring the navigation. |
Hey @rhyn0bytes it is! I just dropped the tags to prevent some spam I was seeing. I'll still flag it as hacktoberfest-approved once we get it all worked out! |
Going to start a |
Right on, thanks @rhyn0bytes! Feel free to ping me with questions at any time - let's keep the styling & functionality as close to the original as possible. And yeah, you'll likely encounter goofy bugs (if you see anything particularly troubling, go ahead and pop open an issue ticket), the website update has been more of a side-side project of mine and hasn't gotten the due diligence it deserves! |
Hey @rhyn0bytes was this something you were still tinkering on? If not that's totally okay, I'll likely just hop in and work on the refactor myself! |
Hi @danhannigan, I'm so sorry. I let the whole month just run away from me. I started a refactor a few weeks ago. Let me clean up and get you a PR to look at. Would this upcoming weekend be alright? |
@rhyn0bytes oh no need to apologize! There's no rush at all, I just wanted to check in on it and make sure I wasn't going to start on work if you already had something in a good spot, feel free to sling it on up whenever you'd like! |
I had pulled the main navigation from some online source when I was building the site initially, and I have to admit it's not exactly as clean or as friendly as I would like in terms of updating and maintaining - I'd like to refactor it, and I'm open to any thoughts or ideas there. I like the general style of it.
The text was updated successfully, but these errors were encountered: