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

Tapani Pälli tapani.palli at intel.com
Tue Jan 22 06:00:14 UTC 2019



On 1/21/19 10:44 PM, Ilia Mirkin wrote:
> 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.

Was about to say the same ..

<!DOCTYPE html>

and then set charset in the <head> part:

<meta charset="utf-8">


> 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
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
> 


More information about the Piglit mailing list