HomePhabricator

Restore the "Log In" menubar action

Description

Restore the "Log In" menubar action

Summary:
See https://discourse.phabricator-community.org/t/activation-link-in-welcome-mail-only-works-if-new-user-isnt-semi-logged-in/740/7.

In T13024, I rewrote the main menu bar to hide potentially sensitive items (like notification and message counts and saved search filters) until users fully log in.

However, the "Log In" item got caught in this too. For clarity, rename shouldAllowPartialSessions() to shouldRequireFullSession() (since logged-out users don't have any session at all, so it would be a bit misleading to say that "Log In" "allows" a partial session). Then let "Log In" work again for logged-out users.

(In most cases, users are prompted to log in when they take an action which requires them to be logged in -- like creating or editing an object, or adding comments -- so this item doesn't really need to exist. However, it aligns better with user expectations in many cases to have it present, and some reasonable operations like "Check if I have notifications/messages" don't have an obvious thing to click otherwise.)

Test Plan: Viewed site in an incognito window, saw "Log In" button again. Browsed normally, saw normal menu.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D18818

Details

Provenance
epriestleyAuthored on Dec 5 2017, 1:42 PM
epriestleyPushed on Dec 5 2017, 8:13 PM
Reviewer
amckinley
Differential Revision
D18818: Restore the "Log In" menubar action
Parents
rPa989dd181dd1: Fix Mercurial commit history ordering
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18930
Build 25527: Run Core Tests