Page MenuHomePhabricator
Differential D15191

Improve type and icon information in typeahead
ClosedPublic
Actions

Authored by epriestley on Feb 5 2016, 8:45 PM.
Tags
None
Referenced Files
F14399877: D15191.id36672.diff
Sun, Dec 22, 6:02 PM
Unknown Object (File)
Fri, Dec 6, 11:37 AM
Unknown Object (File)
Wed, Dec 4, 3:56 PM
Unknown Object (File)
Thu, Nov 28, 4:52 PM
Unknown Object (File)
Mon, Nov 25, 8:50 AM
Unknown Object (File)
Nov 22 2024, 5:22 AM
Unknown Object (File)
Nov 20 2024, 9:29 PM
Unknown Object (File)
Nov 16 2024, 9:26 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10289: Clarify project type in search typeahead
Commits
Restricted Diffusion Commit
rP4132ba0853e9: Improve type and icon information in typeahead
Summary

Ref T10289. This probably doesn't cover everything but should do a little bit better.

Although we should mabye just exlude milestones from this menu completely?

Test Plan

Screen Shot 2016-02-05 at 12.44.49 PM.png (232×340 px, 19 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36672.Feb 5 2016, 8:45 PM
epriestley retitled this revision from to Improve type and icon information in typeahead.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10289: Clarify project type in search typeahead.
epriestley added a comment.Feb 5 2016, 8:46 PM

Even if we drop milestones out of this menu I think this is still an improvement for other projects given that we're giving icons stronger types now.

chad edited edge metadata.Feb 5 2016, 8:47 PM

I like it! (for milestones also, I keep using search to get to Projects (v3))

chad accepted this revision.Feb 5 2016, 8:47 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 5 2016, 8:47 PM
Closed by commit rP4132ba0853e9: Improve type and icon information in typeahead (authored by epriestley, committed by epriestley). · Explain WhyFeb 5 2016, 8:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
27 lines
src/
applications/
diffusion/
typeahead/
5 lines
project/
storage/
2 lines
typeahead/
2 lines
webroot/
rsrc/
js/
core/
11 lines

Diff 36673

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/diffusion/typeahead/DiffusionSymbolDatasource.php

Loading...
View Options

src/applications/project/storage/PhabricatorProject.php

Loading...
View Options

src/applications/project/typeahead/PhabricatorProjectDatasource.php

Loading...
View Options

webroot/rsrc/js/core/behavior-search-typeahead.js

Loading...
Phabricator · Built by Phacility