Page MenuHomePhabricator

Put Spaces on Hovercards and ObjectItemLists
ClosedPublic

Authored by epriestley on Jun 5 2015, 8:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 10:43 AM
Unknown Object (File)
Fri, Dec 6, 1:17 PM
Unknown Object (File)
Fri, Dec 6, 9:40 AM
Unknown Object (File)
Wed, Dec 4, 12:58 PM
Unknown Object (File)
Sat, Nov 30, 3:49 AM
Unknown Object (File)
Mon, Nov 25, 8:46 AM
Unknown Object (File)
Sun, Nov 24, 10:51 PM
Unknown Object (File)
Sun, Nov 24, 9:43 AM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T8449: Spaces v1 Errata
Commits
Restricted Diffusion Commit
rP1e4e121956e7: Put Spaces on Hovercards and ObjectItemLists
Summary

Ref T8449.

Test Plan

Screen Shot 2015-06-05 at 1.42.31 PM.png (324×1 px, 41 KB)

Screen Shot 2015-06-05 at 1.42.37 PM.png (202×550 px, 30 KB)

Diff Detail

Repository
rP Phabricator
Branch
spaces14
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6579
Build 6601: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Put Spaces on Hovercards and ObjectItemLists.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a subscriber: chad.
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php
151

intended*

This revision is now accepted and ready to land.Jun 5 2015, 9:09 PM
This revision was automatically updated to reflect the committed changes.

Do you have more of an idea how/when these would show? I'm wondering how far we can get by not showing Spaces UI when possible. ie, only if a user is a member of multiple Spaces and the say, search for all my tasks, which might cross two spaces?

Also another random 4am jetlagged thought, assigning a header-color to a space somehow seems like a reasonable idea.

Do you have more of an idea how/when these would show?

Here's my current thinking -- I can imagine a few rules we can consider using:

  1. Don't show anything if the object is in the your current default space (whatever space new objects you create are created into).
  2. Don't show anything if the object is in the global "default" space.
  3. Don't show anything if you can only see one space.

I think (1) is probably a little too confusing. It also means that "no space shown" has different meanings at different times, assuming we let you switch which space you're creating into (I think we almost certainly will). However, it's maybe-possible that it's fine/natural depending on how the "switching" actually evolves.

I think (2) and (3) are pretty reasonable -- if we do them both it might be a little confusing, but probably fine. I think we could probably add both rules and be able to hide spaces most of the time as a result without any real issues. I'm a tiny bit worried about hiding UI in general when a user only has access to one space, but I think my fears about this being confusing are probably unfounded and/or the kind of thing that can be addressed by just documenting the behavior.

If we swap from names to icons we might want to continue always showing spaces, to avoid a ragged left edge. Let me look into that next and we can play around with it a bit and see how it feels.

Also another random 4am jetlagged thought, assigning a header-color to a space somehow seems like a reasonable idea.

I think we might not really need super-strong cues about space membership, but I'm not sure. Seems reasonable to leave on the table, at least.