Page MenuHomePhabricator
Differential D16725

Don't show participants in Conpherence left open on mobile
ClosedPublic
Actions

Authored by chad on Oct 19 2016, 4:51 AM.
Tags
None
Referenced Files
F15483580: D16725.id40278.diff
Wed, Apr 9, 1:13 PM
F15481934: D16725.id40276.diff
Wed, Apr 9, 12:58 AM
F15478979: D16725.id.diff
Tue, Apr 8, 4:31 AM
F15477447: D16725.diff
Mon, Apr 7, 2:45 PM
F15459566: D16725.id.diff
Mon, Mar 31, 2:13 PM
F15458338: D16725.id40276.diff
Sun, Mar 30, 11:06 PM
F15428842: D16725.id40278.diff
Sun, Mar 23, 10:45 PM
F15418798: D16725.diff
Fri, Mar 21, 1:33 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T11764: Participant list stuck on top of chat history on mobile device
Commits
rP89f0015ae6ec: Don't show participants in Conpherence left open on mobile
Summary

Fixes T11764. Moves rendering of the column to client-side, which can skip if it detects we're on mobile.

Test Plan

Open column on desktop, switch to mobile, don't see column. Toggle column on mobile on and off. Switch back to desktop.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40276.Oct 19 2016, 4:51 AM
chad retitled this revision from to Don't show participants in Conpherence left open on mobile.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T11764: Participant list stuck on top of chat history on mobile device.
Herald added a subscriber: Korvin. · View Herald TranscriptOct 19 2016, 4:51 AM
epriestley accepted this revision.Oct 19 2016, 3:14 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 19 2016, 3:14 PM
Closed by commit rP89f0015ae6ec: Don't show participants in Conpherence left open on mobile (authored by chad, committed by chad). · Explain WhyOct 19 2016, 3:29 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
52 lines
src/
applications/
conpherence/
view/
23 lines
webroot/
rsrc/
js/
application/
conpherence/
34 lines
22 lines

Diff 40278

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/conpherence/view/ConpherenceLayoutView.php

Loading...
View Options

webroot/rsrc/js/application/conpherence/behavior-menu.js

Loading...
View Options

webroot/rsrc/js/application/conpherence/behavior-toggle-widget.js

Loading...
Phabricator · Built by Phacility