GNU bug report logs - #43000
[Web pages] Proposed changes to tour/index.html

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 43000 in the body.
You can then email your comments to 43000 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Thérèse Godefroy <godef.th <at> free.fr>
To: bug-gnu-emacs <at> gnu.org
Subject: [Web pages] Proposed changes to tour/index.html
Date: Sun, 23 Aug 2020 21:24:15 +0200

[Message part 1 (text/plain, inline)]

Hello,

We are planning to migrate all templated pages on gnu.org to HTML5 in
order to improve accessibility. But before doing this, deprecated tags
such as <tt> and <acronym> need to be removed. Emacs has 2 templated
pages that contain <tt>: manual/emacs.html (maybe not in  use anymore),
and tour/index.html. I fixed the latter, but also had some fun restyling
it.

Summary of the changes:

* Replace <tt> with <kbd> or <code>: <kbd> for keyboard entries, and
<code> for the rest.

* Replace tables that are only used for layout with floating divs or
display:table + media queries.

* Modernize the "real" tables.

* Use the Emacs stylesheet, header and footer instead of the www.gnu.org
boilerplace.

The diff is rather big, but at least you can examine the page locally
because it doesn't need anything outside the emacs directory.

All the best,
Thérèse

PS. I sent a first draft to Nicolas Petton a few weeks ago, but there
were quite a few issues.

[emacs.diff.gz (application/gzip, attachment)]

Message #8 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Thérèse Godefroy <godef.th <at> free.fr>, 
 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Sun, 23 Aug 2020 22:13:54 -0400

Hi Thérèse,

Thérèse Godefroy <godef.th <at> free.fr> writes:

> We are planning to migrate all templated pages on gnu.org to HTML5 in
> order to improve accessibility. But before doing this, deprecated tags
> such as <tt> and <acronym> need to be removed. Emacs has 2 templated
> pages that contain <tt>: manual/emacs.html (maybe not in  use anymore),
> and tour/index.html. I fixed the latter, but also had some fun restyling
> it.
>
> Summary of the changes:
>
> * Replace <tt> with <kbd> or <code>: <kbd> for keyboard entries, and
> <code> for the rest.
>
> * Replace tables that are only used for layout with floating divs or
> display:table + media queries.
>
> * Modernize the "real" tables.
>
> * Use the Emacs stylesheet, header and footer instead of the www.gnu.org
> boilerplace.
>
> The diff is rather big, but at least you can examine the page locally
> because it doesn't need anything outside the emacs directory.

Thanks for working on this.

I applied the patch and looked at the result in Firefox.  I couldn't
find anything that looked wrong during a cursory visual inspection.

(And the Guided Tour in particular looks much better.)

Best regards,
Stefan Kangas

Message #11 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Thérèse Godefroy <godef.th <at> free.fr>
To: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Mon, 24 Aug 2020 14:32:03 +0200

[Message part 1 (text/plain, inline)]

Hello Stephan,

Le 24/08/2020 à 04:13, Stefan Kangas a écrit :
[...)

> I applied the patch and looked at the result in Firefox.  I couldn't
> find anything that looked wrong during a cursory visual inspection.
> 
> (And the Guided Tour in particular looks much better.)
> 
> Best regards,
> Stefan Kangas
> 

Thanks for looking at the proposed layout. I made some minor adjustments
to the dl lists and tables. The idea is to place all command definitions
on a white backgrond (or any color you choose), whether they are in a
list or a table. The attached patch should be applied on top of the
first one.

Best,
Thérèse

[emacs.1.diff (text/x-patch, attachment)]

Message #14 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Thérèse Godefroy <godef.th <at> free.fr>, 
 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Wed, 26 Aug 2020 14:58:44 -0700

Thérèse Godefroy <godef.th <at> free.fr> writes:

> Thanks for looking at the proposed layout. I made some minor adjustments
> to the dl lists and tables. The idea is to place all command definitions
> on a white backgrond (or any color you choose), whether they are in a
> list or a table. The attached patch should be applied on top of the
> first one.

Looks even better.  The proposed colors LGTM, and should be easy enough
to change for anyone who feels like tweaking it later.

[Nit. Regarding the "Tour" in #1, it says "(Click on any of these
pictures for a larger view.)" but the first picture is missing a link.]

> -<table class="commands">
> +<table class="commands modes">
>    <tr><td><kbd>M-x&nbsp;java-mode</kbd></td><td>Mode for editing Java files</td></tr>
>    <tr><td><kbd>M-x&nbsp;python-mode</kbd></td><td>Mode for editing Python files</td></tr>
>    <tr><td><kbd>M-x&nbsp;text-mode</kbd></td><td>Mode for editing text files</td></tr>
> -  <tr><td><kbd>M-x&nbsp;fundamental-mode</kbd></td><td>Mode with no specializations at all</td></tr>
> +  <tr><td><kbd>M-x fundamental-mode</kbd></td><td>Mode with no specializations at all</td></tr>
>  </table>

Should the "&nbsp;" really be removed here?

I think the proposed changes should be installed.  Thanks again.

Best regards,
Stefan Kangas

Message #17 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Thérèse Godefroy <godef.th <at> free.fr>
To: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Thu, 27 Aug 2020 14:26:22 +0200

[Message part 1 (text/plain, inline)]

Le 26/08/2020 à 23:58, Stefan Kangas a écrit :
[...]
> [Nit. Regarding the "Tour" in #1, it says "(Click on any of these
> pictures for a larger view.)" but the first picture is missing a link.]

Thanks for catching this sneaky bug! It only affects wide screens (above
55em). What happens is that the big numbers are outside the normal flow,
and cover part  of the image. There are 2 ways to fix this:

* Push the numbers to the bottom layer with "z-index: -1".

* Give them enough margin to keep clear of the screenshots (9em works fine).

I chose the first option because it only takes one definition.

>> -<table class="commands">
>> +<table class="commands modes">
>>    <tr><td><kbd>M-x&nbsp;java-mode</kbd></td><td>Mode for editing Java files</td></tr>
>>    <tr><td><kbd>M-x&nbsp;python-mode</kbd></td><td>Mode for editing Python files</td></tr>
>>    <tr><td><kbd>M-x&nbsp;text-mode</kbd></td><td>Mode for editing text files</td></tr>
>> -  <tr><td><kbd>M-x&nbsp;fundamental-mode</kbd></td><td>Mode with no specializations at all</td></tr>
>> +  <tr><td><kbd>M-x fundamental-mode</kbd></td><td>Mode with no specializations at all</td></tr>
>>  </table>
> 
> Should the "&nbsp;" really be removed here?

In fact, it doesn't make any difference because the line wraps at the
second dash anyway. I wanted another wrapping point for very narrow
screens, but finally decided to linearize the tables below 35em.

The latest changes are in emacs.2.diff (attached).


I think it would be just as well to use dl lists for all the command
definitions. If the definitions are very short, they can be on the same
line as the command. We can even line them up vertically, table-like.


> I think the proposed changes should be installed.  Thanks again.

Would you like me to install them?

Best,
Thérèse

[emacs.2.diff (text/x-patch, attachment)]

Message #20 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Thérèse Godefroy <godef.th <at> free.fr>
Cc: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Tue, 1 Sep 2020 07:36:30 -0700

Thérèse Godefroy <godef.th <at> free.fr> writes:

>> I think the proposed changes should be installed.  Thanks again.
>
> Would you like me to install them?

Yes, please.

Message #25 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Thérèse Godefroy <godef.th <at> free.fr>
To: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Tue, 1 Sep 2020 20:43:43 +0200

Le 01/09/2020 à 16:36, Stefan Kangas a écrit :
> Thérèse Godefroy <godef.th <at> free.fr> writes:
> 
>>> I think the proposed changes should be installed.  Thanks again.
>>
>> Would you like me to install them?
> 
> Yes, please.
> 
Done.

Message #28 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Thérèse Godefroy <godef.th <at> free.fr>
Cc: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Sun, 6 Sep 2020 03:47:05 +0000

Hi Thérèse,

Thérèse Godefroy <godef.th <at> free.fr> writes:

> Done.

Thanks for doing that; it looks much better.

I'm seeing some oddities when navigating using the top row menu now:

1. I go to https://www.gnu.org/software/emacs/
2. I click Download in the top row menu
3. I click Home

I now end up at: https://www.gnu.org/savannah-checkouts/gnu/emacs/emacs.html

I would expect to end up back at https://www.gnu.org/software/emacs/

Could this be related to your recent changes?

Best regards,
Stefan Kangas

Message #31 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Therese Godefroy <thg <at> gnu.org>
To: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Sun, 6 Sep 2020 17:48:39 +0200

Le 06/09/2020 à 05:47, Stefan Kangas a écrit :
[...]
> I'm seeing some oddities when navigating using the top row menu now:
> 
> 1. I go to https://www.gnu.org/software/emacs/
> 2. I click Download in the top row menu
> 3. I click Home
> 
> I now end up at: https://www.gnu.org/savannah-checkouts/gnu/emacs/emacs.html
> 
> I would expect to end up back at https://www.gnu.org/software/emacs/
> 
> Could this be related to your recent changes?
> 
> Best regards,
> Stefan Kangas
> 

Hello Stefan,

I don't think this has to do with recent changes. It has to do with URL
rewriting by an Apache module.

The URLs in your navigation links are relative to the current directory,
which happens to be on Savannah:
      <a href="emacs.html">Home</a>
      <a href="download.html">Download</a>

If you want them to be rewritten to the software directory, you need to
make the URLs relative to server root:
      <a href="/software/emacs/emacs.html">Home</a>
      <a href="/software/emacs/download.html">Download</a>

As a bonus, you will be able to move files from one directory to another
without breaking the links. For instance, the navlinks will be the same
in emacs.html and tour/index.html.

All the best,
Thérèse

Message #34 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Therese Godefroy <thg <at> gnu.org>, 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Sun, 6 Sep 2020 20:13:01 -0700

Therese Godefroy <thg <at> gnu.org> writes:

> I don't think this has to do with recent changes. It has to do with URL
> rewriting by an Apache module.
>
> The URLs in your navigation links are relative to the current directory,
> which happens to be on Savannah:
>       <a href="emacs.html">Home</a>
>       <a href="download.html">Download</a>
>
> If you want them to be rewritten to the software directory, you need to
> make the URLs relative to server root:
>       <a href="/software/emacs/emacs.html">Home</a>
>       <a href="/software/emacs/download.html">Download</a>
>
> As a bonus, you will be able to move files from one directory to another
> without breaking the links. For instance, the navlinks will be the same
> in emacs.html and tour/index.html.

Making the links absolute has the unfortunate side-effect that it
becomes harder to test the site locally during development, right?
I'd hope that there is some way to avoid that.

I see that there is a 302 (temporary) redirect here:

$ curl https://www.gnu.org/software/emacs/index.html
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>302 Found</title>
</head><body>
<h1>Found</h1>
<p>The document has moved <a
href="https://www.gnu.org/savannah-checkouts/gnu/emacs/emacs.html">here</a>.</p>
<hr>
<address>Apache/2.4.7 Server at www.gnu.org Port 443</address>
</body></html>

Do you know if this is expected?

Best regards,
Stefan Kangas

Message #37 received at 43000 <at> debbugs.gnu.org (full text, mbox):

From: Therese Godefroy <thg <at> gnu.org>
To: 43000 <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Mon, 7 Sep 2020 17:07:35 +0200

Hello Stefan,

Le 07/09/2020 à 05:13, Stefan Kangas a écrit :
[...]
> Making the links absolute has the unfortunate side-effect that it
> becomes harder to test the site locally during development, right?
> I'd hope that there is some way to avoid that.

The only way I know of is to mount the emacs directory at the root of a
local web server. It's pretty convenient, in fact. But of course you
won't do all the fancy things that the gnu server does.

> I see that there is a 302 (temporary) redirect here:
> 
> $ curl https://www.gnu.org/software/emacs/index.html
> <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
> <html><head>
> <title>302 Found</title>
> </head><body>
> <h1>Found</h1>
> <p>The document has moved <a
> href="https://www.gnu.org/savannah-checkouts/gnu/emacs/emacs.html">here</a>.</p>
> <hr>
> <address>Apache/2.4.7 Server at www.gnu.org Port 443</address>
> </body></html>
> 
> Do you know if this is expected?

I think this is expected, because you get it every time the index is
rewritten to another file. For instance, you get a 302 with gsrc
(index.html is rewritten to gsrc.html) but not with barcode (index.html
is a real file).

There are 2 sorts of rewrite directives:

* Some are in the Apache config. You and I can't change them (unless you
are a sysadmin). One of the things they do is rewrite
/software/emacs/emacs.html to /savannah-checkouts/gnu/emacs/emacs.html:

    RewriteCond /var/www/savannah-checkouts/gnu/$1 -d
    RewriteRule ^/software/([^/]+)(.*)$ /savannah-checkouts/gnu/$1$2

* Others are generated from the .symlinks files, which webmasters and
package maintainers can edit. Emacs has at least one:

    https://www.gnu.org/software/emacs/.symlinks

Here is the first line:

    emacs.html index.html

It is used to generate an entry in a rewrite map which is processed by a
series of directives. The last one ends with [R,L], explaining the 302.

Best,
Thérèse

Message #42 received at 43000-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Therese Godefroy <thg <at> gnu.org>, 43000-done <at> debbugs.gnu.org
Subject: Re: bug#43000: [Web pages] Proposed changes to tour/index.html
Date: Sat, 24 Oct 2020 05:37:31 -0700

Hi Therese,

Therese Godefroy <thg <at> gnu.org> writes:

> I think this is expected, because you get it every time the index is
> rewritten to another file.

OK, thank you.  I'm therefore closing this bug.

Best regards,
Stefan Kangas

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.