Page MenuHomePhabricator
Differential D12969

Possible fix for month view day deadzone
ClosedPublic
Actions

Authored by lpriestley on May 21 2015, 6:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 3, 7:59 PM
Unknown Object (File)
Thu, Feb 27, 7:47 PM
Unknown Object (File)
Thu, Feb 20, 10:25 PM
Unknown Object (File)
Thu, Feb 20, 1:42 AM
Unknown Object (File)
Fri, Feb 14, 9:00 PM
Unknown Object (File)
Tue, Feb 11, 9:14 PM
Unknown Object (File)
Mon, Feb 10, 3:32 AM
Unknown Object (File)
Sun, Feb 9, 2:33 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8193: On month days with a long list of events, there's a deadzone in the month cells where clicking won't open day view.
Commits
Restricted Diffusion Commit
rPd3268aecc226: Possible fix for month view day deadzone
Summary

Ref T8193, Possible fix for month view day deadzone

Test Plan

Open Calendar month view in Chrome, Firefox, or Safari. Verify that days with many events still link to the day views of those days.

Diff Detail

Repository
rP Phabricator
Branch
calendarmonthdaylink
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6190
Build 6211: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 31244.May 21 2015, 6:06 PM
lpriestley retitled this revision from to Possible fix for month view day deadzone.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8193: On month days with a long list of events, there's a deadzone in the month cells where clicking won't open day view..
Herald added reviewers: chad, Blessed Reviewers. · View Herald TranscriptMay 21 2015, 6:06 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 31245.May 21 2015, 6:07 PM
lpriestley edited edge metadata.

map

lpriestley updated this revision to Diff 31254.May 21 2015, 10:57 PM

Specificity

chad edited edge metadata.May 21 2015, 11:07 PM

itsawesome

lpriestley updated this revision to Diff 31257.May 21 2015, 11:56 PM
lpriestley edited edge metadata.

surrendering to php

lpriestley updated this revision to Diff 31258.May 22 2015, 12:02 AM

indents

lpriestley updated this revision to Diff 31259.May 22 2015, 12:03 AM

map

lpriestley added a comment.May 22 2015, 12:04 AM

Very disappointed this isn't a css-only fix, but don't see a way around it.

epriestley accepted this revision.May 22 2015, 12:04 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/view/phui/calendar/PHUICalendarMonthView.php
308 ↗(On Diff #31259)

I think $count and $max_height are always passed, so you could remove the defaults.

309 ↗(On Diff #31259)

Not technically used, since it's always assigned a little later on.

This revision is now accepted and ready to land.May 22 2015, 12:04 AM
lpriestley updated this revision to Diff 31260.May 22 2015, 12:08 AM
lpriestley marked 2 inline comments as done.
lpriestley edited edge metadata.

technicalities

Closed by commit rPd3268aecc226: Possible fix for month view day deadzone (authored by lpriestley, committed by lpriestley). · Explain WhyMay 22 2015, 12:11 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
webroot/
rsrc/
css/
phui/
calendar/
11 lines

Diff 31244

View Options

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

Loading...
Phabricator · Built by Phacility