Add LED patterns to LED documentation #2774
Conversation
Include screenshots/screen recordings of all examples Examples mostly use the same discontinuous gradient for consistency and to make reversed and scrolling effects obvious
Should be 0.25 Hz, not 4 Hz
Move progress mask example to the mask example
|
this should remove the ignore from inspector.json |
Frustratingly, the :poster: attribute won't use transformed asset paths
|
Setting the width to 900 on the videos makes mobile scroll. Looks like the images get scaled down |
|
If you put in a static png with the same name as the video files, I think that will fix the pdf build |
As in, |
|
|
yes. That will work for image files, but I'm not sure if sphinx will automatically fall back from a video file to an image file |
Already covered in the buffer view section
This won't work, looks like ..only would probably be best. |
|
On chrome and edge, the videos don't show a play. Maybe because they are not tall enough? Works ok on Firefox
|
|
I'm not sure, both firefox and chrome showed the controls overlaid on the videos in my testing. Annoyingly, the controls obscure the videos, and there doesn't seem to be any options for moving the (eg to be below the video). Maybe we could have some custom JS to make the controls appear on hover? https://stackoverflow.com/q/40284816 |
|
The video controls are there. The slider is the progress slider. If you click on the very very top few pixels of the video and press space, it will start to play and the controls will fade away. Since this is looping, I would rather put in gifs for the website and png for the pdf. |
That was my preference too, but Daltz says it's an accessibility concern to have autoplaying gifs |
|
I guess the fix then is making the videos a lot taller so the controls render properly |
We might be able to do something like this, rather then make new videos: https://stackoverflow.com/a/65116261 |
Videos are paused by default This avoids the issue of the controls overlapping and obscuring the animations
Render gifs everywhere else (PDFs should render just the first frame)
|
A bit of feedback from our beta testing: LEDPattern examples sometimes refer to calling a function using the Units library, like Updating the example to include the Units reference before |
|
this has conflicts now since some of the css was merged in the a-stop PR |
|
\inspector fix all |
Inspector ReportUp To DateOutdated - Automatically FixedInvalid - Manual Intervention Needed |
|
There's a conflict with inspector.json |
There was a problem hiding this comment.
It's not obvious that these are videos. Mousing over them gives the same hand that the static images give, which in a static image opens the static image. I think if you add :muted: and :autoplay: they'll autoplay, and then clicking on them will stop them (see the DS A-Stop). I think that would be preferred.
Both firefox and chrome seem to render the poster different sizes from each other, and also different sizes from the video (so that when you click the video to start it, it resizes). I think it's probably ok to not use a poster.
|
\inspector fix all |
Update video extension to support :playsinline: Remove posters, and set to play automatically (can be disabled by clicking)
Resolves #2773