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
Unknown Object (File)
Wed, Apr 17, 3:05 PM
Unknown Object (File)
Wed, Apr 17, 5:36 AM
Unknown Object (File)
Sat, Apr 13, 4:00 AM
Unknown Object (File)
Sat, Apr 13, 3:59 AM
Unknown Object (File)
Sat, Apr 13, 3:59 AM
Unknown Object (File)
Sat, Apr 13, 3:59 AM
Unknown Object (File)
Sat, Apr 13, 3:59 AM
Unknown Object (File)
Sat, Apr 13, 3:54 AM
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.