Page MenuHomePhabricator

Improve handling of multiple edit forms when logged out
ClosedPublic

Authored by epriestley on Dec 17 2015, 2:55 PM.
Tags
None
Referenced Files
F14059289: D14806.diff
Sun, Nov 17, 6:51 PM
F14050899: D14806.id35800.diff
Thu, Nov 14, 9:21 PM
F14047922: D14806.diff
Thu, Nov 14, 5:49 AM
F14036183: D14806.diff
Sun, Nov 10, 8:56 AM
F14029055: D14806.id35800.diff
Fri, Nov 8, 7:06 PM
F14019795: D14806.diff
Tue, Nov 5, 11:39 PM
F14012025: D14806.id35800.diff
Fri, Nov 1, 6:57 AM
F14008767: D14806.diff
Wed, Oct 30, 6:37 AM
Subscribers
None

Details

Summary

Ref T10004. Currently, when a logged-out user visits an application like Maniphest, we show them a disabled "Create Task" button with no dropdown menu.

This is technically correct in some sense because none of the items in the menu will work, but we can be more helpful and show the items, just in a disabled state:

Screen Shot 2015-12-17 at 6.44.56 AM.png (215×302 px, 20 KB)

When the user clicks these, they'll be pushed through the login flow and (after D14804) end up on the same page they were on when they selected the item. From here, they can proceed normally.

I changed "...to continue." to "...to take this action." to hopefully be a little more clear. In particular, we do not continue the action after you log in: you end up back on the same page you started on. For example, if you clicked "Create New Bug" from the list view, you end up back on the list view and need to click "Create New Bug" again. If you clicked "Edit Task" from some task detail page, you end up on the task detail page and have to click "Edit Task" again.

I think this behavior is always very good. I think it is often the best possible behavior: for actions like "Edit Blocking Tasks" and "Merge Duplicates In", the alternatives I can see are:

  • Send user back to task page (best?)
  • Send user to standalone page with weird dialog on it and no context (underlying problem behavior all of this is tackling, clearly not good)
  • Send user back to task page, but with dialog open (very complicated, seems kind of confusing/undesirable?)

For actions like "Create New Bug" or "Edit Task", we have slightly better options:

  • Send user back to task page (very good?)
  • Send user to edit/create page (slightly better?)

However, we have no way to tell if a Workflow "makes sense" to complete in a standalone way. That is, we can't automatically determine which workflows are like "Edit Task" and which workflows are like "Merge Duplicates In".

Even within an action, this distinction is not straightforward. For example, "Create Task" can standalone from the Maniphest list view, but should not from a Workboard. "Edit Task" can standalone from the task detail page, but should not from an "Edit" pencil action on a list or a workboard.

Since the simpler behavior is easy, very good in all cases, often the best behavior, and never (I think?) confusing or misleading, I don't plan to puruse the "bring you back to the page, with the dialog open" behavior at any point. I'm theoretically open to discussion here if you REALLY want the dialogs to pop open magically but I think it's probably a lot of work.

Test Plan

As a logged out user, clicked "Create Task". Got a dropdown showing the options available to me if I log in. Clicked one, logged in, ended up in a reasonable place (the task list page where I'd started).

Diff Detail

Repository
rP Phabricator
Branch
ae3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9612
Build 11504: Run Core Tests
Build 11503: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Improve handling of multiple edit forms when logged out.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 17 2015, 4:21 PM
This revision was automatically updated to reflect the committed changes.