HomePhabricator

Improve some Spaces behaviors

Description

Improve some Spaces behaviors

Summary:
Ref T8449. Try out some more subtle behaviors:

  • Make the "Space" control part of the policy control, so the UI shows "Visible To: [Space][Policy]". I think this helps make the role of spaces more clear. It also makes them easier to implement.
  • Don't show the default space in headers: instead, show nothing.
  • If the user has access to only one space, pretend spaces don't exist (no edit controls, no header stuff).

This might be confusing, but I think most of the time it will all align fairly well with user expectation.

Test Plan:

  • Viewed a list of pastes (saw Space with non-default space, no space with default space, no space with user in only one space).
  • Viewed a paste (saw Space with non-default space, saw no space with default space, saw no space with user in only one space).
  • Edited spaces on objects (control as privileged user, no control as locked user).
  • Created a new paste in a space (got space select as privileged user, no select as locked user).

Reviewers: chad, btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T8449

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Jun 10 2015, 10:52 PM
Reviewer
btrahan
Differential Revision
D13229: Improve some Spaces behaviors
Parents
rP8a10cfbc9873: Fix missing recurrence end date control
Branches
Unknown
Tags
Unknown
Tasks
T8449: Spaces v1 Errata