Page MenuHomePhabricator
Differential D12916

Mobile day view should display event list, not day table
ClosedPublic
Actions

Authored by lpriestley on May 18 2015, 10:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:05 AM
Unknown Object (File)
Sat, Apr 20, 4:27 AM
Unknown Object (File)
Sat, Apr 20, 4:27 AM
Unknown Object (File)
Sat, Apr 20, 4:26 AM
Unknown Object (File)
Sat, Apr 20, 4:26 AM
Unknown Object (File)
Sat, Apr 20, 4:19 AM
Unknown Object (File)
Sat, Apr 20, 3:40 AM
Unknown Object (File)
Thu, Apr 18, 11:03 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8178: Calendar day view should show day view above the event preview sidebar on mobile.
Commits
Restricted Diffusion Commit
rPec759ef46f17: Mobile day view should display event list, not day table
Summary

Closes T8178, Mobile day view should display event list, not day table

Test Plan

Open day view on a device, observe that day table is hidden.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 31072.May 18 2015, 10:39 PM
lpriestley retitled this revision from to Mobile day view should display event list, not day table.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added reviewers: epriestley, chad.
lpriestley added a task: T8178: Calendar day view should show day view above the event preview sidebar on mobile..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 18 2015, 10:39 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 31073.May 18 2015, 10:41 PM
lpriestley edited edge metadata.

debug cleanup

lpriestley updated this revision to Diff 31075.May 18 2015, 10:42 PM

one more

epriestley accepted this revision.May 18 2015, 11:01 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 18 2015, 11:01 PM
Closed by commit rPec759ef46f17: Mobile day view should display event list, not day table (authored by lpriestley, committed by lpriestley). · Explain WhyMay 18 2015, 11:04 PM
This revision was automatically updated to reflect the committed changes.
chad edited edge metadata.May 18 2015, 11:14 PM

.device restricts to tablet and mobile. Maybe day view is fine on tablet?

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
view/
phui/
calendar/
2 lines
webroot/
rsrc/
css/
phui/
calendar/
8 lines

Diff 31076

View Options

resources/celerity/map.php

Loading...
View Options

src/view/phui/calendar/PHUICalendarDayView.php

Loading...
View Options

webroot/rsrc/css/phui/calendar/phui-calendar-day.css

Loading...
Phabricator · Built by Phacility