Page MenuHomePhabricator

Add Pinboard Items to Timeline
ClosedPublic

Authored by chad on May 18 2017, 5:25 PM.
Tags
None
Referenced Files
F12846382: D17950.id43177.diff
Fri, Mar 29, 12:57 AM
Unknown Object (File)
Feb 3 2024, 3:52 PM
Unknown Object (File)
Jan 9 2024, 10:17 AM
Unknown Object (File)
Jan 6 2024, 8:44 PM
Unknown Object (File)
Jan 3 2024, 12:05 PM
Unknown Object (File)
Dec 28 2023, 11:37 PM
Unknown Object (File)
Dec 27 2023, 11:06 AM
Unknown Object (File)
Dec 27 2023, 11:06 AM
Subscribers

Details

Summary

This allows adding of pinboard items to a timeline. I'm hoping we can get this in for Maniphest (Pholio, Cover Image) and Macro (because, Macro), but unsure how to scalably do this. Anyways, here's the front end.

Test Plan

Make some fake timeline items in UIExamples, test mobile, tablet, and desktop breakpoints.

image.png (1×2 px, 349 KB)

Diff Detail

Repository
rP Phabricator
Branch
render-image-timeline (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17087
Build 22852: Run Core Tests
Build 22851: arc lint + arc unit

Event Timeline

Looks promising to me. I think we can make this work without too much hardship.

This revision is now accepted and ready to land.May 18 2017, 5:32 PM

Yeah I started to force something into transactions but realized it'd bite me in the end if it wasn't scalable. At least pinboard for image display should work in most/all cases and case be tweaked as needed (lightbox).

This revision was automatically updated to reflect the committed changes.