HomePhabricator

Sort "closed" results (like disabled users) to the bottom of typeaheads, but…

Tags
None
Referenced Files
F4903845: Screen Shot 2017-04-14 at 12.11.39 PM.png
Apr 14 2017, 8:01 PM
F4903848: Screen Shot 2017-04-14 at 12.12.00 PM.png
Apr 14 2017, 8:01 PM
F4903849: Screen Shot 2017-04-14 at 12.17.01 PM.png
Apr 14 2017, 8:01 PM
F4903843: Screen Shot 2017-04-14 at 12.10.49 PM.png
Apr 14 2017, 8:01 PM
Subscribers
None

Description

Sort "closed" results (like disabled users) to the bottom of typeaheads, but don't hide them completely

Summary:
Fixes T12538. Instead of hiding "closed" results unless only closed results match, show closed results but sort them to the bottom.

This fixes the actual issue in T12538, and I think this is probably the correct/best behavior for the global search.

It also makes all other typeaheads use this behavior. They currently have a "bug" where enabled user abcd makes it impossible to select disabled user abc. This manifests in some real cases, where enabled function members(abc) makes it impossible to disabled user abc in some function tokenizers.

If ths feels worse, we could go back to filtering in the simpler cases and introduce a rule like "show closed results if only closed results would be shown OR if query is an exact match for the disabled result", but that gets dicier because "exact match" is a fuzzy concept.

(There are a lot of other minor bad behaviors that this doesn't try to fix.)

Test Plan:
Enabled project "instabug" no longer prevents bot user "instabug" from being shown:

Screen Shot 2017-04-14 at 12.10.49 PM.png (218×394 px, 16 KB)

Disabled user "mmaven" is sorted below enabled user "mmclewis", in defiance of the otherwise alphabetical order. There's no visual cue that this user is disabled because of T6906.

Screen Shot 2017-04-14 at 12.11.39 PM.png (242×399 px, 22 KB)

Same as above, but this source renders "disabled" in a more obvious way:

Screen Shot 2017-04-14 at 12.12.00 PM.png (182×451 px, 20 KB)

Function selecting members of active project members(instabug) no longer prevents selection of bot user instabug:

Screen Shot 2017-04-14 at 12.17.01 PM.png (172×446 px, 23 KB)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12538

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