Buttons that require logged-in user should redirect to login page if you're not logged in #489

Closed
evanp opened this Issue Mar 27, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@evanp
Contributor

evanp commented Mar 27, 2013

Right now, we hide like, comment, share, follow buttons if you're not logged in. Instead, we should show them, and if you're not logged in, we should redirect to the login page.

@evanp

This comment has been minimized.

Show comment Hide comment
@evanp

evanp Mar 27, 2013

Contributor

See also #399 for what to do after you're logged in.

Contributor

evanp commented Mar 27, 2013

See also #399 for what to do after you're logged in.

@marjoleink

This comment has been minimized.

Show comment Hide comment
@marjoleink

marjoleink Mar 30, 2013

Buttons that require logged-in user should either not appear at all when user is not logged in, or be clearly disabled (visibly, and not by color alone; and with an attached title attribute).

Buttons that require logged-in user should either not appear at all when user is not logged in, or be clearly disabled (visibly, and not by color alone; and with an attached title attribute).

@evanp

This comment has been minimized.

Show comment Hide comment
@evanp

evanp Aug 29, 2013

Contributor

So, I'm pretty sure this works now.

Contributor

evanp commented Aug 29, 2013

So, I'm pretty sure this works now.

@evanp evanp closed this Aug 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment