Bug 82935

Comment 1 Giorgos Keramidas 2005-07-03 23:30:02 UTC

On 2005-07-03 19:09, Daniel Gerzo <danger@rulez.sk> wrote:
> --- article.sgml.diff begins here ---
> --- article.sgml.orig	Sun Jul  3 16:59:50 2005
> +++ article.sgml	Sun Jul  3 18:58:39 2005
> @@ -45,8 +45,11 @@
>        <ulink url="http://gerda.univie.ac.at/freebsd-laptops/">The &os;
>        Laptop Compatibility List</ulink>.</para>
>  
> -    <para>For communications with other &os; laptop users, check out
> -      the &a.mobile.name; list.</para>
> +    <para>If you want to communicate with other &os; laptop users,
> +      checkout the &a.mobile.name; list.  You can also get additional
> +      information about using Laptops on &os; at
> +      <ulink url="http://tuxmobil.org/mobile_bsd.html">
> +	http://tuxmobil.org/mobile_bsd.html</ulink>.</para>

Nice diff.  I only have a minor comment.  When the content of a <ulink>
element is just the URL itself, it's probably better to leave the
content of the element empty -- the stylesheets will take care of
duplicating the URL in the generated link, i.e.:

	<ulink url="http://example.net/"></ulink>

This doesn't duplicate the URL, making it double the effort to switch to
a different URL and twice as error-prone (i.e. update one but forget the
other).

If this looks ok to the rest of the -www people, I can commit it with
this minor change to the diff :-)

- Giorgos

On Sun, Jul 03, 2005 at 10:30:20PM +0000, Giorgos Keramidas wrote:
> The following reply was made to PR www/82935; it has been noted by GNATS.
> 
> From: Giorgos Keramidas <keramida@freebsd.org>
> To: Daniel Gerzo <danger@rulez.sk>
> Cc: FreeBSD-gnats-submit@freebsd.org
> Subject: Re: www/82935: Update and fix english in FreeBSD on Laptops article
> Date: Mon, 4 Jul 2005 01:30:02 +0300
> 
>  On 2005-07-03 19:09, Daniel Gerzo <danger@rulez.sk> wrote:
>  > --- article.sgml.diff begins here ---
>  > --- article.sgml.orig	Sun Jul  3 16:59:50 2005
>  > +++ article.sgml	Sun Jul  3 18:58:39 2005
>  > @@ -45,8 +45,11 @@
>  >        <ulink url="http://gerda.univie.ac.at/freebsd-laptops/">The &os;
>  >        Laptop Compatibility List</ulink>.</para>
>  >  
>  > -    <para>For communications with other &os; laptop users, check out
>  > -      the &a.mobile.name; list.</para>
>  > +    <para>If you want to communicate with other &os; laptop users,
>  > +      checkout the &a.mobile.name; list.  You can also get additional
>  > +      information about using Laptops on &os; at
>  > +      <ulink url="http://tuxmobil.org/mobile_bsd.html">
>  > +	http://tuxmobil.org/mobile_bsd.html</ulink>.</para>
>  
>  Nice diff.  I only have a minor comment.  When the content of a <ulink>
>  element is just the URL itself, it's probably better to leave the
>  content of the element empty -- the stylesheets will take care of
>  duplicating the URL in the generated link, i.e.:
>  
>  	<ulink url="http://example.net/"></ulink>
>  
>  This doesn't duplicate the URL, making it double the effort to switch to
>  a different URL and twice as error-prone (i.e. update one but forget the
>  other).
>  
>  If this looks ok to the rest of the -www people, I can commit it with
>  this minor change to the diff :-)


I'm happy with this, although "check out" is more correct than
"checkout".

Ceri
-- 
Only two things are infinite, the universe and human stupidity, and I'm
not sure about the former.			  -- Einstein (attrib.)

Comment 3 Giorgos Keramidas 2005-07-04 11:38:38 UTC

State Changed
From-To: open->closed

Committed.  Thanks :) 

Comment 4 Giorgos Keramidas 2005-07-04 11:38:38 UTC

Responsible Changed
From-To: freebsd-www->keramida