[Piglit] [PATCH 0/4] summary/html: html fixes

Ilia Mirkin imirkin at alum.mit.edu
Mon Jan 21 20:44:05 UTC 2019


Series is

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

As an aside, it appears that we're using xhtml? That's pretty much
dead nowadays. The recommendation tends to be to have <!DOCTYPE html>
on the first line, and that's it.

Cheers,

  -ilia

On Mon, Jan 21, 2019 at 11:01 AM Erik Faye-Lund
<erik.faye-lund at collabora.com> wrote:
>
> After a recent update, it seems GNOME Web / Epiphany has gotten stricter
> about the HTML it accepts. This patch series fixes up the issues I
> noticed.
>
> Erik Faye-Lund (4):
>   summary/html: close the right html-tag
>   summary/html: open pre-tag instead of closing it
>   summary/html: ensure xml-line is the very first line
>   summary/html: escape html
>
>  templates/feature.mako     | 3 ++-
>  templates/index.mako       | 7 ++++---
>  templates/test_result.mako | 4 ++--
>  3 files changed, 8 insertions(+), 6 deletions(-)
>
> --
> 2.20.1
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list