Page MenuHomePhabricator

Normalize icon color on user/project lists.
ClosedPublic

Authored by chad on Feb 5 2016, 8:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 10:24 AM
Unknown Object (File)
Thu, Apr 11, 7:47 AM
Unknown Object (File)
Fri, Apr 5, 7:55 AM
Unknown Object (File)
Thu, Mar 28, 9:45 PM
Unknown Object (File)
Mar 18 2024, 4:37 AM
Unknown Object (File)
Mar 6 2024, 4:51 AM
Unknown Object (File)
Feb 27 2024, 11:54 PM
Unknown Object (File)
Feb 20 2024, 2:26 AM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPe2668a5fd04b: Normalize icon color on user/project lists.
Summary

Minor, just fall back to the grey icon in all cases (too much color for me).

Test Plan

Review a Project and a Profile

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Normalize icon color on user/project lists..
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Feb 5 2016, 8:29 PM

There's now no actual indication of the tag color of a project on the list view. Maybe bars?

I'm not sure it's helpful on these views. In most cases we show a subsection of all available meta-data. Like, we'll show the name, icon, and color, but not the image. On workboards this is great. In list views though I don't find the color very useful (I'm mostly just looking at the name).

That is, it's a little odd that I can query by color but can't actually see which results are which color:

Screen Shot 2016-02-05 at 12.32.43 PM.png (296×484 px, 27 KB)

I'm fine with getting rid of it but we should probably remove query-by-color and projects.colors if we do. I only added the config like two weeks ago so users probably aren't too attached to it yet.

Let me limit this to just (people profiles) in the meantime.

chad edited edge metadata.
  • scope to just profiles
This revision was automatically updated to reflect the committed changes.