ContextInstitute / bfcom

http://bfn.context.org
GNU General Public License v3.0
4 stars 1 forks source link

Add Search to menu #41

Open rgilman opened 5 years ago

rgilman commented 5 years ago

At some point we will need to get search functioning on the site but for now we just need a placeholder in the top menu.

Since the top menu uses WordPress's Appearance > Menus functionality, each one of us needs to do this manually (for now). Here's what I've done.

  1. Go to https://dev.bfcom.local/wp-admin/nav-menus.php or Dashboard > Appearance > Menus – however you like to think about it.
  2. Under Pages, choose View All and find Site-Wide Activity and add that to the end of bfcom-main-menu.
  3. Once added, open that menu pane and change the name to Search.
  4. Save it.

You could point this menu to any page you like – for now it's just a placeholder.

Once added, we can include it in our design work for both desktop and mobile.

iangilman commented 5 years ago

@DavidScottBernstein @wunluv please follow the instructions above and then post here to let everyone know that you've done so. :-)

DavidScottBernstein commented 5 years ago

@rgilman I'm a bit confused about what step 2 means. I see the page Site-Wide Activity at the bottom of Pages in Wordpress but I'm not sure what you mean by "add that to the end of bfcom-main-menu."

rgilman commented 5 years ago

Apologies for not being as clear as I might have been.

Here's a screenshot:

screen shot 2018-10-01 at 1 27 44 pm

In it, I have checked the checkbox for Site-Wide Activity. The next step is to click the "Add to Menu" button.

Once added, the lower part of the menu list will look like this:

screen shot 2018-10-01 at 1 32 32 pm

Click the little disclose triangle next to the word "Page" on the right. The panel will open to look like this:

screen shot 2018-10-01 at 1 32 48 pm

Change the "Navigation Label" from Site-Wide Activity to Search. The end result should look like this:

screen shot 2018-10-01 at 1 33 01 pm

Once you've done all of that, remember to "Save Menu" (blue buttons on the right of the screen).

Hope that helps.

DavidScottBernstein commented 5 years ago

Yes, big help. Thank you. Done.

iangilman commented 5 years ago

Done!

@DavidScottBernstein would you be up for adding a search icon to that new menu icon, just like you did with the others?

DavidScottBernstein commented 5 years ago

Yes. However, I don’t see a search menu item after I made the above change in Wordpress.

rgilman commented 5 years ago

@DavidScottBernstein do you mean that you don't see it on bfn.context.org or that you don't see it on your local dev site? As for bfn.context.org, that was my oversight. I've now added the Search menu. I also activated the footer menu.

DavidScottBernstein commented 5 years ago

@rgilman I didn't see Search on either. Now I see it in the top menu on bfn.context.org but still not on my local site after I fetched from Master.

rgilman commented 5 years ago

Hmm... Curious. The inclusion of Search in the menu should be controlled by the database and not the theme files so I'm puzzle about why it's disappeared from you local site. In any case, I suggest you go back to Admin > Dashboard > Appearance > Menus to see what's going on and redo the above procedure if need be.

iangilman commented 5 years ago

@DavidScottBernstein Sounds like maybe you never fully got the search menu on in the first place? Perhaps you missed one of the steps in the instructions above? Anyway, yes, trying again sounds like a good idea. Keep in mind that it's controlled entirely by what you do in the admin and should have nothing to do with the code.

iangilman commented 5 years ago

Okay, @DavidScottBernstein has added the search icon to the menu as part of #57.

The one remaining thing is for @wunluv to make the change on his local setup and confirm here.