Page MenuHomePhabricator
Differential D19272

Make closed/disabled results in the remarkup autocomplete more visually clear
ClosedPublic
Actions

Authored by epriestley on Mar 30 2018, 3:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 26, 2:17 PM
Unknown Object (File)
Sun, Feb 9, 5:47 AM
Unknown Object (File)
Sun, Feb 9, 5:47 AM
Unknown Object (File)
Sun, Feb 9, 5:47 AM
Unknown Object (File)
Sun, Feb 9, 2:24 AM
Unknown Object (File)
Sat, Feb 8, 2:29 AM
Unknown Object (File)
Sat, Feb 1, 8:18 PM
Unknown Object (File)
Thu, Jan 30, 3:17 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13114: Plans: 2018 Week 13 Bonus Content
Commits
rP7eaa27683e97: Make closed/disabled results in the remarkup autocomplete more visually clear
Summary

Ref T13114. See PHI522. Although it looks like results are already ordered correctly, the override rendering isn't accommodating disabled results gracefully.

Give closed results a distinctive look (grey + strikethru) so it's clear when you're autocompleting @mention... into a disabled user.

Test Plan

Screen Shot 2018-03-30 at 8.41.49 AM.png (221×362 px, 23 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 30 2018, 3:45 PM
Harbormaster completed remote builds in B19952: Diff 46131.Mar 30 2018, 3:46 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 30 2018, 3:46 PM
epriestley requested review of this revision.
Closed by commit rP7eaa27683e97: Make closed/disabled results in the remarkup autocomplete more visually clear (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
22 lines
webroot/
rsrc/
css/
core/
5 lines
js/
phuix/
5 lines

Diff 46133

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/core/remarkup.css

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXAutocomplete.js

Loading...
Phabricator · Built by Phacility