Currently, we only display the "Signup/Login" links on the site homepage to anonymous users, but nowhere else. Let's provide more opportunities to signup/login.

CommentFileSizeAuthor
#8 commons.signup_login_utility_links.patch797 bytesjpontani
#6 commons.signup_login_utility_links.patch757 bytesjpontani
#4 commons.signup_login_utility_links.patch692 bytesjpontani

Comments

ezra-g’s picture

ezra-g commented
Issue tags: -Needs design +Commons 7.x-3.1 radar

Tagging for 3.1.

ezra-g’s picture

ezra-g commented
Component: Private groups » Code
jpontani’s picture

jpontani commented
Assigned: Unassigned » jpontani
jpontani’s picture

jpontani commented
Status: Active » Needs review
StatusFileSize
new commons.signup_login_utility_links.patch692 bytes

Patch attached.

ezra-g’s picture

ezra-g commented
Status: Needs review » Needs work

Solid start! We should verify that users have permission to register, before displaying the Signup link.

jpontani’s picture

jpontani commented
StatusFileSize
new commons.signup_login_utility_links.patch757 bytes

Ah that would help, wouldn't it.

ezra-g’s picture

ezra-g commented
Status: Needs work » Needs review

:D. Thanks!

jpontani’s picture

jpontani commented
Status: Needs review » Needs work
StatusFileSize
new commons.signup_login_utility_links.patch797 bytes

Actually this one, just copied from site_homepage

jpontani’s picture

jpontani commented
Status: Needs work » Needs review

Posting at same time! Setting as needs review.

Zarabadoo’s picture

Zarabadoo commented
Status: Needs review » Reviewed & tested by the community

This looks fine from a front end perspective. I say put it in.

rosemeria’s picture

rosemeria commented

Tested on 7.x-3.x-dev - works great. Thanks

Need to delete css for ".not-logged-in .block-search" - margin-top not needed anymore on anonymous display.

Delete -- line#2805 (global.styles.css)
.not-logged-in .block-search {
margin-top: 15px;
}

davidhunter’s picture

davidhunter commented

I put the Signup/Login buttons into the main content of my site using blocks.
Then I said "Display on all pages, except these listed" :
user
user/login
user/password
user/register
user/reset/*

For the anonymous user Role.

Works well like this.

jpontani’s picture

jpontani commented
Status: Reviewed & tested by the community » Fixed

Committed!

http://drupalcode.org/project/commons_utility_links.git/commitdiff/3155f4c

Automatically closed -- issue fixed for 2 weeks with no activity.