[2/2] WebUI: Fix XHTML conformity in WebUI header file

Submitted by Adrian Schmutzler on June 9, 2017, 10:59 p.m.

Details

Message ID 1497049192-38505-1-git-send-email-freifunk@adrianschmutzler.de
State Accepted
Headers show

Commit Message

Adrian Schmutzler June 9, 2017, 10:59 p.m.
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
index 3fb28c5..8df0ed0 100755
--- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
+++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
@@ -34,15 +34,15 @@  fi
 	<meta http-equiv="content-type" content="text/html; charset=utf-8" />
 	<title><%= ${HOSTNAME} %></title>
 	<link href="/style.css" rel="stylesheet" type="text/css" media="screen" />
-       <meta http-equiv="cache-control" content="no-cache">
-       <meta http-equiv="pragma" content="no-cache">
-       <meta http-equiv="expires" content="0">
+       <meta http-equiv="cache-control" content="no-cache" />
+       <meta http-equiv="pragma" content="no-cache" />
+       <meta http-equiv="expires" content="0" />
 </head>
 <body>
 	<div id="header" style="position: relative;">
 		<h1><%= ${HOSTNAME} %></h1>
 		<h2>Freifunk Franken Router</h2>
-		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" />
+		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" alt="" />
 	</div>
 	<div style="height: 30px; background-color: #ffb400;"></div>
 	<div id="menu">

Comments

Christian Dresel July 2, 2017, 2:53 p.m.
Hallo

macht soweit Sinn aber es gibt noch viele weitere stellen wo man das
fixen sollte, z.b.:

https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff-web/files/www/public/cgi-bin/status.html
20-22

Weitere hab ich jetzt nicht auf die schnell gefunden bin aber auch nicht
alle Files durch, müsste man mal genauer durchgucken.

Reviewed-by: Christian Dresel <fff@chrisi01.de>

mfg

Christian

On 10.06.2017 00:59, Adrian Schmutzler wrote:
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> index 3fb28c5..8df0ed0 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> @@ -34,15 +34,15 @@ fi
>  	<meta http-equiv="content-type" content="text/html; charset=utf-8" />
>  	<title><%= ${HOSTNAME} %></title>
>  	<link href="/style.css" rel="stylesheet" type="text/css" media="screen" />
> -       <meta http-equiv="cache-control" content="no-cache">
> -       <meta http-equiv="pragma" content="no-cache">
> -       <meta http-equiv="expires" content="0">
> +       <meta http-equiv="cache-control" content="no-cache" />
> +       <meta http-equiv="pragma" content="no-cache" />
> +       <meta http-equiv="expires" content="0" />
>  </head>
>  <body>
>  	<div id="header" style="position: relative;">
>  		<h1><%= ${HOSTNAME} %></h1>
>  		<h2>Freifunk Franken Router</h2>
> -		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" />
> +		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" alt="" />
>  	</div>
>  	<div style="height: 30px; background-color: #ffb400;"></div>
>  	<div id="menu">
>
Adrian Schmutzler July 2, 2017, 2:55 p.m.
Gibt noch einige Stellen, aber die sind nicht alle so einfach zu fixen.

Da der header überall drinsteht und einfach zu fixen war, hab ich erstmal
nur den gemacht.

-----Original Message-----
From: Christian Dresel [mailto:fff@chrisi01.de] 
Sent: Sonntag, 2. Juli 2017 16:53
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>;
franken-dev@freifunk.net
Subject: Re: [PATCH 2/2] WebUI: Fix XHTML conformity in WebUI header file

Hallo

macht soweit Sinn aber es gibt noch viele weitere stellen wo man das fixen
sollte, z.b.:

https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff
-web/files/www/public/cgi-bin/status.html
20-22

Weitere hab ich jetzt nicht auf die schnell gefunden bin aber auch nicht
alle Files durch, müsste man mal genauer durchgucken.

mfg

Christian

On 10.06.2017 00:59, Adrian Schmutzler wrote:
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header 
> b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> index 3fb28c5..8df0ed0 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> @@ -34,15 +34,15 @@ fi
>  	<meta http-equiv="content-type" content="text/html; charset=utf-8"
/>
>  	<title><%= ${HOSTNAME} %></title>
>  	<link href="/style.css" rel="stylesheet" type="text/css"
media="screen" />
> -       <meta http-equiv="cache-control" content="no-cache">
> -       <meta http-equiv="pragma" content="no-cache">
> -       <meta http-equiv="expires" content="0">
> +       <meta http-equiv="cache-control" content="no-cache" />
> +       <meta http-equiv="pragma" content="no-cache" />
> +       <meta http-equiv="expires" content="0" />
>  </head>
>  <body>
>  	<div id="header" style="position: relative;">
>  		<h1><%= ${HOSTNAME} %></h1>
>  		<h2>Freifunk Franken Router</h2>
> -		<img src="/freifunk.svg" style="position: absolute; top:
1px; right: 42px;" />
> +		<img src="/freifunk.svg" style="position: absolute; top:
1px; 
> +right: 42px;" alt="" />
>  	</div>
>  	<div style="height: 30px; background-color: #ffb400;"></div>
>  	<div id="menu">
>
Christian Dresel July 2, 2017, 3:02 p.m.
hi

On 02.07.2017 16:55, mail@adrianschmutzler.de wrote:
> Gibt noch einige Stellen, aber die sind nicht alle so einfach zu fixen.
> 
> Da der header überall drinsteht und einfach zu fixen war, hab ich erstmal

für die Public Seite gibt es einen extra Header wo genau der gleiche
Fehler drinnen ist. Schau dir meinen Link genau an ;) /public/ und /ssl/

mfg

Christian

> nur den gemacht.
> 
> -----Original Message-----
> From: Christian Dresel [mailto:fff@chrisi01.de] 
> Sent: Sonntag, 2. Juli 2017 16:53
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>;
> franken-dev@freifunk.net
> Subject: Re: [PATCH 2/2] WebUI: Fix XHTML conformity in WebUI header file
> 
> Hallo
> 
> macht soweit Sinn aber es gibt noch viele weitere stellen wo man das fixen
> sollte, z.b.:
> 
> https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff
> -web/files/www/public/cgi-bin/status.html
> 20-22
> 
> Weitere hab ich jetzt nicht auf die schnell gefunden bin aber auch nicht
> alle Files durch, müsste man mal genauer durchgucken.
> 
> mfg
> 
> Christian
> 
> On 10.06.2017 00:59, Adrian Schmutzler wrote:
>> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>> ---
>>  src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header 
>> b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> index 3fb28c5..8df0ed0 100755
>> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> @@ -34,15 +34,15 @@ fi
>>  	<meta http-equiv="content-type" content="text/html; charset=utf-8"
> />
>>  	<title><%= ${HOSTNAME} %></title>
>>  	<link href="/style.css" rel="stylesheet" type="text/css"
> media="screen" />
>> -       <meta http-equiv="cache-control" content="no-cache">
>> -       <meta http-equiv="pragma" content="no-cache">
>> -       <meta http-equiv="expires" content="0">
>> +       <meta http-equiv="cache-control" content="no-cache" />
>> +       <meta http-equiv="pragma" content="no-cache" />
>> +       <meta http-equiv="expires" content="0" />
>>  </head>
>>  <body>
>>  	<div id="header" style="position: relative;">
>>  		<h1><%= ${HOSTNAME} %></h1>
>>  		<h2>Freifunk Franken Router</h2>
>> -		<img src="/freifunk.svg" style="position: absolute; top:
> 1px; right: 42px;" />
>> +		<img src="/freifunk.svg" style="position: absolute; top:
> 1px; 
>> +right: 42px;" alt="" />
>>  	</div>
>>  	<div style="height: 30px; background-color: #ffb400;"></div>
>>  	<div id="menu">
>>
> 
>
Tim Niemeyer July 2, 2017, 3:48 p.m.
Hi

auch Applied.

Tim

Am Sonntag, den 02.07.2017, 16:53 +0200 schrieb Christian Dresel:
> Hallo
> 
> macht soweit Sinn aber es gibt noch viele weitere stellen wo man das
> fixen sollte, z.b.:
> 
> https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff-web/files/www/public/cgi-bin/status.html
> 20-22
> 
> Weitere hab ich jetzt nicht auf die schnell gefunden bin aber auch nicht
> alle Files durch, müsste man mal genauer durchgucken.
> 
> Reviewed-by: Christian Dresel <fff@chrisi01.de>
> 
> mfg
> 
> Christian
> 
> On 10.06.2017 00:59, Adrian Schmutzler wrote:
> > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > ---
> >  src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> > index 3fb28c5..8df0ed0 100755
> > --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> > +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
> > @@ -34,15 +34,15 @@ fi
> >  	<meta http-equiv="content-type" content="text/html; charset=utf-8" />
> >  	<title><%= ${HOSTNAME} %></title>
> >  	<link href="/style.css" rel="stylesheet" type="text/css" media="screen" />
> > -       <meta http-equiv="cache-control" content="no-cache">
> > -       <meta http-equiv="pragma" content="no-cache">
> > -       <meta http-equiv="expires" content="0">
> > +       <meta http-equiv="cache-control" content="no-cache" />
> > +       <meta http-equiv="pragma" content="no-cache" />
> > +       <meta http-equiv="expires" content="0" />
> >  </head>
> >  <body>
> >  	<div id="header" style="position: relative;">
> >  		<h1><%= ${HOSTNAME} %></h1>
> >  		<h2>Freifunk Franken Router</h2>
> > -		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" />
> > +		<img src="/freifunk.svg" style="position: absolute; top: 1px; right: 42px;" alt="" />
> >  	</div>
> >  	<div style="height: 30px; background-color: #ffb400;"></div>
> >  	<div id="menu">
> > 
> 
> -- 
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Adrian Schmutzler July 2, 2017, 5:45 p.m.
Hab für den public Teil einen Patch gebaut. Was noch fehlt, sind die
Content-Teile unter /ssl/

Hab auch überlegt, statt XHTML einen HTML5 Header zu setzen, aber dann gehen
die ganzen Caching meta-Tags nicht mehr ...

-----Original Message-----
From: Christian Dresel [mailto:fff@chrisi01.de] 
Sent: Sonntag, 2. Juli 2017 17:02
To: mail@adrianschmutzler.de; franken-dev@freifunk.net
Subject: Re: [PATCH 2/2] WebUI: Fix XHTML conformity in WebUI header file

hi

On 02.07.2017 16:55, mail@adrianschmutzler.de wrote:
> Gibt noch einige Stellen, aber die sind nicht alle so einfach zu fixen.
> 
> Da der header überall drinsteht und einfach zu fixen war, hab ich 
> erstmal

für die Public Seite gibt es einen extra Header wo genau der gleiche Fehler
drinnen ist. Schau dir meinen Link genau an ;) /public/ und /ssl/

mfg

Christian

> nur den gemacht.
> 
> -----Original Message-----
> From: Christian Dresel [mailto:fff@chrisi01.de]
> Sent: Sonntag, 2. Juli 2017 16:53
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; 
> franken-dev@freifunk.net
> Subject: Re: [PATCH 2/2] WebUI: Fix XHTML conformity in WebUI header 
> file
> 
> Hallo
> 
> macht soweit Sinn aber es gibt noch viele weitere stellen wo man das 
> fixen sollte, z.b.:
> 
> https://github.com/FreifunkFranken/firmware/blob/master/src/packages/f
> ff/fff -web/files/www/public/cgi-bin/status.html
> 20-22
> 
> Weitere hab ich jetzt nicht auf die schnell gefunden bin aber auch 
> nicht alle Files durch, müsste man mal genauer durchgucken.
> 
> mfg
> 
> Christian
> 
> On 10.06.2017 00:59, Adrian Schmutzler wrote:
>> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>> ---
>>  src/packages/fff/fff-web/files/www/ssl/cgi-bin/header | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> index 3fb28c5..8df0ed0 100755
>> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/header
>> @@ -34,15 +34,15 @@ fi
>>  	<meta http-equiv="content-type" content="text/html; charset=utf-8"
> />
>>  	<title><%= ${HOSTNAME} %></title>
>>  	<link href="/style.css" rel="stylesheet" type="text/css"
> media="screen" />
>> -       <meta http-equiv="cache-control" content="no-cache">
>> -       <meta http-equiv="pragma" content="no-cache">
>> -       <meta http-equiv="expires" content="0">
>> +       <meta http-equiv="cache-control" content="no-cache" />
>> +       <meta http-equiv="pragma" content="no-cache" />
>> +       <meta http-equiv="expires" content="0" />
>>  </head>
>>  <body>
>>  	<div id="header" style="position: relative;">
>>  		<h1><%= ${HOSTNAME} %></h1>
>>  		<h2>Freifunk Franken Router</h2>
>> -		<img src="/freifunk.svg" style="position: absolute; top:
> 1px; right: 42px;" />
>> +		<img src="/freifunk.svg" style="position: absolute; top:
> 1px;
>> +right: 42px;" alt="" />
>>  	</div>
>>  	<div style="height: 30px; background-color: #ffb400;"></div>
>>  	<div id="menu">
>>
> 
>