Page MenuHomePhabricator
Differential D18350

Hide curtainview on mobile if it's empty
ClosedPublic
Actions

Authored by chad on Aug 6 2017, 3:06 AM.
Tags
None
Referenced Files
F18237369: D18350.diff
Wed, Aug 20, 9:20 PM
F18093524: D18350.id44108.diff
Aug 7 2025, 7:02 PM
F18090428: D18350.id44104.diff
Aug 6 2025, 3:46 PM
F18086985: D18350.diff
Aug 6 2025, 1:40 AM
F17938415: D18350.id.diff
Jul 31 2025, 2:10 AM
F17936291: D18350.id44108.diff
Jul 31 2025, 12:07 AM
F17864984: D18350.diff
Jul 28 2025, 2:46 AM
F17859698: D18350.id44104.diff
Jul 27 2025, 9:00 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP58db64c81f5c: Hide curtainview on mobile if it's empty
Summary

If we don't have any panels, just an action list, we want to hide the entire box on mobile since it's just an empty line.

Test Plan

Review Owners, Differential curtains on mobile, desktop.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Aug 6 2017, 3:06 AM
Herald added a subscriber: Korvin. · View Herald TranscriptAug 6 2017, 3:06 AM
Harbormaster completed remote builds in B17923: Diff 44104.Aug 6 2017, 3:07 AM
epriestley accepted this revision.Aug 6 2017, 11:44 AM
This revision is now accepted and ready to land.Aug 6 2017, 11:44 AM
Closed by commit rP58db64c81f5c: Hide curtainview on mobile if it's empty (authored by chad, committed by chad). · Explain WhyAug 6 2017, 3:07 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
view/
layout/
9 lines
webroot/
rsrc/
css/
phui/
4 lines

Diff 44108

View Options

resources/celerity/map.php

Loading...
View Options

src/view/layout/PHUICurtainView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-curtain-view.css

Loading...
Phabricator · Built by Phacility