ContextInstitute / bfcom

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

#21, 32: Replaced 'user' with user photo with dropdown menu #36

Closed DavidScottBernstein closed 5 years ago

DavidScottBernstein commented 5 years ago

21, #32 User photo currently linked to user profile and no user menu

31 Created drop down but no items have been implemented yet.

iangilman commented 5 years ago

Looks like @rgilman has found some of the links you'll need for the menu:

<a href="/wp-admin/">Admin Dashboard</a>
<a href="/wp-login.php?action=logout">Logout</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/profile/edit/">Edit My Profile</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/profile/change-avatar/">Change Avatar</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/messages/">Messages</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/compose/">Compose Message</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/settings/">Email/Password Settings</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/settings/notifications/">Notification Settings</a>
<a href="/members/<?php bp_loggedin_user_username(); ?>/settings/profile/">Profile Settings</a>
iangilman commented 5 years ago

... And you might look at the documentation for the Foundation dropdown:

https://foundation.zurb.com/sites/docs/dropdown-menu.html

Looks like you're doing it a little differently than that, but it's working just fine! I'm curious what the differences are.

rgilman commented 5 years ago

It looks to me that what @DavidScottBernstein has done with dropdown-pane is actually better in this case than my attempt with dropdown menu. Looks good to me for merging. I can then PR/merge the clean-up and recode work I've been doing.

iangilman commented 5 years ago

@DavidScottBernstein What do you think? Are you in progress on additions to this pull request, or shall we merge it? Sounds like you should at least push the changes you made to the indentation.

iangilman commented 5 years ago

This is superseded by #36; closing.