Support videos in article viewer

[wvffle]

The app seems promising.
The only two things that I may consider missing are #199 and ability to play videos embedded in the article.

Currently they're displayed as a play icon but after clicking it does nothing

Tested on Factorio blog: https://www.factorio.com/blog/rss

[jocmp]

thanks for raising, I'll track something for videos in #199.

[jocmp]

Basic full screen video support is available as of 2024.08.1021-dev.

[wvffle]

@jocmp I'm testing this build now with yesterday's post in the linked RSS feed.

In the default display mode, only first video is recognized as the video and can be played by clicking on the play button, although fullscreen seems to be disabled.

The rest of the videos are not recognized and only the play icon is available as a link to the video resource. These buttons seem to have different sizes.

In the full content mode none of the videos are recognized.

[jocmp]

Thanks for the details. I'll see what I can do.

[jocmp]

I played around with full screen support. It's pretty janky and I'm not happy with the solution. For instance, the scroll and video state reset when you rotate the device.

So I went back to the drawing board and compared NetNewsWire and Reeder's video players. Both readers only allow inline video.

NetNewsWire

Reeder

I was missing a few settings to play videos inline. I've added those to #229. Long story short, I'll support inline video and skip full screen videos to avoid that technical complexity.

The final result shown below. Videos won't autoplay in Full Content but they'll have all the controls to play and pause.

redux.mov

[jocmp]

Changes are live in the latest release. I also made sure that looped autoplay videos work for factorio.com. Here's what that looks like

autoplay.mp4

[wvffle]

I cannot replicate the autoplay shown in the last video with the latest github release but I am now able to play those videos inline just fine

[wvffle]

Some QOL features to consider:

• Show thumbnail of the video
• Auto pause the video if it's out of the view (especially if the video has audio track)

[PhilC813]

Twitter posts embedded in articles are rendered in full content mode by Capy, but the videos included in some of these posts don't make it. One example here.

Do you think it would be possible to have them show up? Although I fear it may require JavaScript.

[jocmp]

It might be possible - sometimes the JavaScript will work if it's embedded in an iframe. This is how YouTube videos work in the app for example.

I have an outstanding ticket to investigate embed. I added Twitter to the list

• Verify embeds work #676