zaurus.html: a mistake, again

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

zaurus.html: a mistake, again

Maxime Villard
Hi,
rapha@ committed changes to hppa.html, landisk.html and zaurus.html. For
zaurus.html, he mistakenly removed "<br clear=all>" - which causes the image
to overlap - and forgot to remove "<a href="images/zaurus-c3000.jpg">".


--- zaurus.html 2013-08-05 12:51:21.357942758 +0200
+++ zaurus.html 2013-08-05 13:03:47.293045735 +0200
@@ -17,7 +17,6 @@
 <h2><font color="#e00000">OpenBSD/zaurus</font></h2>
 
 <hr>
-<a href="images/zaurus-c3000.jpg">
 <img src="images/zaurus-c3000-small.jpg" width="400" height="300" align="right" alt="zaurus sl-c3000"></a>
 
 OpenBSD/zaurus is an effort to make OpenBSD run on several PDAs of the
@@ -43,6 +42,7 @@
   <li><a href="#install">Getting and installing</a><br>
   <li><a href="#projects">Projects & bugs left to fix</a><br>
 </ul>
+<br clear=all>
 <hr>
 
 <a name="history"></a>


You know, guys, I think you should be a little bit more *attentive* when
editing the html pages.

Reply | Threaded
Open this post in threaded view
|

Re: zaurus.html: a mistake, again

Stuart Henderson-6
CC'ing to rapha@, it is usually best to copy the relevant developer if there's
a problem with a commit rather than just hope they come across it on a mailing list.


On 2013/08/05 13:06, Maxime Villard wrote:

> Hi,
> rapha@ committed changes to hppa.html, landisk.html and zaurus.html. For
> zaurus.html, he mistakenly removed "<br clear=all>" - which causes the image
> to overlap - and forgot to remove "<a href="images/zaurus-c3000.jpg">".
>
>
> --- zaurus.html 2013-08-05 12:51:21.357942758 +0200
> +++ zaurus.html 2013-08-05 13:03:47.293045735 +0200
> @@ -17,7 +17,6 @@
>  <h2><font color="#e00000">OpenBSD/zaurus</font></h2>
>  
>  <hr>
> -<a href="images/zaurus-c3000.jpg">
>  <img src="images/zaurus-c3000-small.jpg" width="400" height="300" align="right" alt="zaurus sl-c3000"></a>
>  
>  OpenBSD/zaurus is an effort to make OpenBSD run on several PDAs of the
> @@ -43,6 +42,7 @@
>    <li><a href="#install">Getting and installing</a><br>
>    <li><a href="#projects">Projects & bugs left to fix</a><br>
>  </ul>
> +<br clear=all>
>  <hr>
>  
>  <a name="history"></a>
>
>
> You know, guys, I think you should be a little bit more *attentive* when
> editing the html pages.

Reply | Threaded
Open this post in threaded view
|

Re: zaurus.html: a mistake, again

Miod Vallat
In reply to this post by Maxime Villard
> You know, guys, I think you should be a little bit more *attentive* when
> editing the html pages.

Yes your diff removes a <a> while keeping a </a>.
Pot, kettle, black.

Miod

Reply | Threaded
Open this post in threaded view
|

Re: zaurus.html: a mistake, again

Maxime Villard
Le 05/08/2013 13:41, Miod Vallat a écrit :
>> You know, guys, I think you should be a little bit more *attentive* when
>> editing the html pages.
>
> Yes your diff removes a <a> while keeping a </a>.
> Pot, kettle, black.
>
> Miod
>
>

So, he also forgot to remove that </a>.

Reply | Threaded
Open this post in threaded view
|

Re: zaurus.html: a mistake, again

Miod Vallat
> So, he also forgot to remove that </a>.

No; the <a> ... </a> is intentional, and there is no need to remove it.

Miod

Reply | Threaded
Open this post in threaded view
|

Re: zaurus.html: a mistake, again

Maxime Villard
>> So, he also forgot to remove that </a>.
>
> No; the <a> ... </a> is intentional, and there is no need to remove it.
>
> Miod
>
>

Yes I know, but since he removed these <a> ... </a> in hppa.html and socppc.html
in the same commit, he must have forgotten to remove those in zaurus.html.

Anyway, if you don't see it fit, just forget it.