GNU bug report logs - #49310
website: Redesign video pages

Previous Next

Full log

View this message in rfc822 format

From: Luis Felipe <luis.felipe.la <at> protonmail.com>
To: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
Cc: 49310 <at> debbugs.gnu.org
Subject: [bug#49310] website: Redesign video pages
Date: Sat, 03 Jul 2021 17:01:28 +0000

[Message part 1 (text/plain, inline)]

Hi Florian,

On Saturday, July 3rd, 2021 at 10:08 AM, pelzflorian (Florian Pelz) <pelzflorian <at> pelzflorian.de> wrote:

> Thank you! Much nicer than before, both in looks and code-wise and
>
> also documentation-wise. This was much effort.

Phew, glad to hear that!


> Shall I push your commit and afterwards redirect the old video URLs in
>
> maintenance.git?

Maybe better to redirect to /en/videos/?


> I don’t know the plan for further videos and if forcing reverse
>
> chronological order will be useful someday.

Well, yeah, I'm not sure whether it will be useful either.

I think the idea is to add a selection of videos from the community, as proposed by Julien in https://issues.guix.gnu.org/43871. So the list could grow enough to justify a reverse chronological order, like in the blog.


> > New miniature and poster images for videos:
> >
> > -   website/static/media/videos/2018/seagl-everyday-use-of-guix.mini.webp
> > -   website/static/media/videos/2018/seagl-everyday-use-of-guix.webp
> >
> >     […]
>
> Yes, switching from .png to .webp seems appropriate, even if it is
>
> unsupported on old still-supported macOS versions or ReactOS or such
>
> things, because they cannot playback the .webm videos anyway, I think.

I'd say let's use webp and webm, and if we get complaints, we see how to solve any issues (at least for images; with videos is harder because of storage resources).


> > diff --git a/website/apps/media/builder.scm b/website/apps/media/builder.scm
>
> “#:use-module (srfi srfi-1)“ can be removed.

Removed in the new patch attached.


> > Modified files:
> >
> > -   website/apps/media/data.scm (videos): New list of videos plus
> >
> >     playlists.
> >
> >     (playlist): Remove in favor of the new videos list.
>
> s/(playlist)/(playlists)/

Corrected in the new patch.

Thanks for the review, Florian.

[0001-website-Redesign-video-pages.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.