Skip to content
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

About.php and Dynamically Generated Menu #15

Open
rcass opened this issue Jun 22, 2018 · 3 comments
Open

About.php and Dynamically Generated Menu #15

rcass opened this issue Jun 22, 2018 · 3 comments
Labels
if time permits Not urgent but nice to have

Comments

@rcass
Copy link

rcass commented Jun 22, 2018

In about.php I noticed that you are just hard coding the menu.

<nav>
<ul class="aboutNavigation">
<li><a href="#aboutJump">About Wav</a></li>
<li><a href="#festivalJump">Festival</a></li>
<li><a href="#teamJump">Team</a></li>
<li><a href="#collaboratorsJump">Collaborators</a></li>
<li><a href="#workJump">Work</a></li>
<li><a href="#pressJump">Press</a></li>
</ul>
</nav>

This might be a good interm solution, but if the client ever wants to change the menu, they will need to go into the php :/

Is there a reason for not dynamically generating the menu here? 🤔

@pinaska
Copy link
Collaborator

pinaska commented Jun 22, 2018

This about-page-only menu is only used for navigation through this page sections. It does not contain external links.

@rcass
Copy link
Author

rcass commented Jun 22, 2018

Oh I see. Okay in that case I would still create (register in functions.php) a new menu, so the client has access to this menu and can make changes.

Then under the menu items just create custom links. As a custom link to can chose the link label (ex: Press) and also the link (ex: set it to #pressJump)

Basically even through it is just jumping to different sections on the page, it still can be a dynamically generated menu (and accessible to the client).

@pinaska
Copy link
Collaborator

pinaska commented Jun 23, 2018

ok, we will address this. Thank you

@pinaska pinaska added the if time permits Not urgent but nice to have label Jun 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
if time permits Not urgent but nice to have
Projects
None yet
Development

No branches or pull requests

2 participants