Page MenuHomePhabricator
Differential D16643

Add 'Persist Chat' option in Conpherence notification menu
ClosedPublic
Actions

Authored by chad on Oct 1 2016, 5:27 PM.
Tags
None
Referenced Files
F15447605: D16643.diff
Fri, Mar 28, 12:41 AM
F15446414: D16643.id40070.diff
Thu, Mar 27, 7:00 PM
F15446413: D16643.id.diff
Thu, Mar 27, 7:00 PM
F15429891: D16643.diff
Mon, Mar 24, 4:45 AM
F15412353: D16643.diff
Wed, Mar 19, 12:37 PM
F15411965: D16643.id.diff
Wed, Mar 19, 10:33 AM
F15302837: D16643.id40069.diff
Mar 5 2025, 10:43 PM
Unknown Object (File)
Feb 25 2025, 7:22 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP2f2126ecce08: Add 'Persist Chat' option in Conpherence notification menu
Summary

This exposes the chat window to a larger audience beside people who accidentaly hit \.

Test Plan

Lots of clicks and reloads.

pasted_file (114×422 px, 10 KB)

Diff Detail

Repository
rP Phabricator
Branch
persist-chat (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13960
Build 18084: Run Core Tests
Build 18083: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 40067.Oct 1 2016, 5:27 PM
chad retitled this revision from to Add 'Persist Chat' option in Conpherence notification menu.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptOct 1 2016, 5:27 PM
chad edited the test plan for this revision. (Show Details)Oct 1 2016, 5:28 PM
epriestley accepted this revision.Oct 1 2016, 6:18 PM
epriestley edited edge metadata.

You're going to handle all the Quicksand bugs that get reported, right?

(This checkbox has the wrong state if you press \ too?)

This revision is now accepted and ready to land.Oct 1 2016, 6:18 PM
chad added a comment.Oct 1 2016, 6:21 PM

maybe remove the hotkey?

chad added a comment.Oct 1 2016, 6:25 PM

(removing hotkey)

epriestley added a comment.Oct 1 2016, 6:28 PM

Seems reasonable.

epriestley added a comment.Oct 1 2016, 6:28 PM

oh you didn't actually update it to

m8b need to ⌘+S

chad updated this revision to Diff 40069.Oct 1 2016, 6:31 PM
chad edited edge metadata.
  • remove hotkey
Closed by commit rP2f2126ecce08: Add 'Persist Chat' option in Conpherence notification menu (authored by chad, committed by chad). · Explain WhyOct 1 2016, 6:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
32 lines
src/
applications/
conpherence/
controller/
42 lines
webroot/
rsrc/
css/
application/
conpherence/
17 lines
js/
application/
conpherence/
7 lines

Diff 40069

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/conpherence/controller/ConpherenceNotificationPanelController.php

Loading...
View Options

webroot/rsrc/css/application/conpherence/notification.css

Loading...
View Options

webroot/rsrc/js/application/conpherence/behavior-durable-column.js

Loading...
Phabricator · Built by Phacility