Page MenuHomePhabricator

Remove 'full-display' setting from Conpherence, spruce up search results
ClosedPublic

Authored by chad on Oct 15 2016, 7:35 PM.
Tags
None
Referenced Files
F14050889: D16710.id40239.diff
Thu, Nov 14, 9:12 PM
F14050507: D16710.diff
Thu, Nov 14, 6:56 PM
F14039794: D16710.id40239.diff
Mon, Nov 11, 6:21 AM
F14039217: D16710.diff
Mon, Nov 11, 4:31 AM
F14023564: D16710.diff
Thu, Nov 7, 2:46 AM
F14022436: D16710.id40236.diff
Wed, Nov 6, 5:21 PM
F14022435: D16710.id40237.diff
Wed, Nov 6, 5:21 PM
F14022434: D16710.id40238.diff
Wed, Nov 6, 5:21 PM
Subscribers

Details

Summary

This removes 'full-display', 'minimal-display' from Conpherence, which I recall was because we had 2 UIs for column and regular chat. I'm also tossing in slightly nicer search results, with a link to the actual message and the full date shown for context.

Test Plan

Post a message in mobile, tablet, full conpherence, and in durable column. Clean up UI in durable column. Do a search in Full UI, click on result date, get taken to the message... usually. My test data is a little wonky, but I think this works most of the time.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Remove 'full-display' setting from Conpherence transactions.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
  • minor css background fix
  • make search show full dates
  • js link to room result, hopefully scroll you there
chad retitled this revision from Remove 'full-display' setting from Conpherence transactions to Remove 'full-display' setting from Conpherence,, spruce up search results.Oct 16 2016, 3:30 AM
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad retitled this revision from Remove 'full-display' setting from Conpherence,, spruce up search results to Remove 'full-display' setting from Conpherence, spruce up search results.Oct 17 2016, 7:40 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 17 2016, 10:38 PM
This revision was automatically updated to reflect the committed changes.