Closed owendodd closed 7 years ago
@owendodd @eessex: Gonna get a checklist going again, I found it helpful last time for EOY! Feel free to use/not use.
Analytics
Page
Live State: Player & Streaming
Preview State
Meta Section
Lower Priority
QA
Kana
Eve
Nice To Have
@owendodd Do we want to add audio controls and the View in Cardboard option somewhere too?
@kanaabe good call. def makes sense to add. I'll adjust the player UI to accommodate.
@owendodd if possible, I'd like to put the View in Cardboard option somewhere at the top of the player:
The long explanation: there are two parts to the player -- the controls overlay (which is hosted on Force), and the VR canvas iframe (which is hosted in an S3 bucket). The controls have our scrubber, play/pause button etc. However, the View in Cardboard button has to be placed on the VR canvas side. Reasoning about whether we should open space on the bottom for this button (am i on desktop or mobile?) is going to get messy and I'd rather we just place it somewhere that the controls definitely won't overlap with.
It's also starting to get a little crowded at the bottom on mobile with the button, scrub, vricon, audio, and cardboard all there. Lmk if that works for ya!
@kanaabe How does this look? Desktop would be the same sans the cardboard icon.
👏 Sweet thank you!
What do you think about simplifying the volume to just Mute/Unmute? I usually just use the volume control on my phone/keyboard if I need to adjust. Facebook and NYT just does mute/unmute.
For more technical convincing, we have to hook into the VR View to adjust the video volume, and I think with a slider, it becomes heavier for the VR View to listen on slider-type events vs a simple button. It's definitely possible -- I'm just trying to keep the number of events going into the VR View to a minimum.
@kanaabe yeah that's fine with me!
Almost makes me wonder if we could just not include it at all. But its probably good to have something. What do you think?
I'll put it in for now and we can take it out if it feels too bulky on smaller phones. Not difficult to add the toggle at all.
@kanaabe @eessex
Here is the 360 guide modal. Sketch file here. I created this article draft that it could pull from (wasn't sure if this should live as an Ed. article or somewhere else).
Slight updates to the player UI:
Not necessary for launch but wondering if we can make the info button scroll down the page rather than jump. @eessex
Big priorities I know about:
[ ] File isn't rendering in Cardboard view for some mobile browsers, file too big?
[x] Facebook share image needs to be updated
Secondary priorities:
[x] Parsely analytics should be added
[x] Sailthru metadata should be added
Two small updates when we've taken a breath:
[x] Update YouTube language to the following: For the most immersive 3D experience, watch in Google Cardboard via the YouTube app.
[x] Markdown links don't seem to be rendering in the About the Series copy
Landing page for the video series we are producing for the venice biennale.
Some high level notes: • Each video needs it own url, and should be loaded dynamically. • Hitting the base url should redirect to the first video • Each video exists as a carousel that has two states live and preview.
• Live videos have a video file, a cover image, associated artist(s), a title and a description • Preview videos have a cover image, and a title • All UBS logos should link to them
Video Player • Default state is a full screen image BG • Play button also shows time (not totally necessary) • Play button and side arrow hover states should both be solid white fill with black arrow / text • On hitting play title, nav and image should fade out, while video + pause button in bottom left should fade out and video should fade in / autoplay. • once video finishes it should fade to black and show 'Read more' (scrolls down page) 'Next video' (navigates right) + Share
Lower Priority: • Venice Coverage footer rail (other editorial coverage) • Text link feature • Artist follow compontent • Open in Youtube app button
Sketch file here: https://cl.ly/2p392c24251l