289538 - clean up snippets (URIs, CTRL- instead of CTRL+, dropping 1.0 perhaps)

Status: RESOLVED FIXED

(www.mozilla.org :: General, defect)

Description

[Anne (:annevk)]

I think they should just state 'Firefox'. Also, we want to point to
 <http://www.mozilla.org/products/firefox/central>
... I guess, instead of:
 <http://www.mozilla.org/products/firefox/central.html>

Comment 1

[Daniel C]

Why point to central ?

To me that's confusing, as all web files should have extensions. Those without I
see as directories, but as you can see, central/ is not a directory:
http://www.mozilla.org/products/firefox/central/

Point to the file http://www.mozilla.org/products/firefox/central.html or if you
want to make it perhaps simpler, point to
http://www.mozilla.org/products/firefox/central/ and make a central/ directory
with the page as index.html in that directory.

Comment 2

[Anne (:annevk)]

Because that is how content negotiation works. We also use it for /support, etc.
and it makes the URI look clean.

Comment 3

[Anne (:annevk)]

Attachment: addresses updated URIs and last few CTRL- instead of CTRL+

This patch updates a lot of URIs to remove either their extension or updates
them to the URI they are now redirecting to. After this I can make a patch
removing the 1.0 from "Firefox 1.0". Although if desired I could combine them
in a single patch.

Comment 4

[Daniel C]

Well I personally don't agree with it. My resolution would be to change
httpd.conf  and disable the automatic completion of file extensions. Then to
keep the URI 'clean' turn the relevant areas into directories with index files
in them. That way, a user could still type a URI ending in /central and the same
file would be served. There would be no confusion then between files and
directories. Of course, the links should then be changd to central/ etc. because
they're directories (without the slash slows down the fetch as it just redirects
to /)

This would make it easier to have files and directories with the same name under
the same parent e.g. firefox/foobar.xhtml and firefox/foobar/

Comment 5

[Axel Hecht]

fantasi may care, and vote.

All others likely dropped the idea of URL schemes over tools by now. At least I
did.

ast should link to
http://www.mozilla-europe.org/ast/products/firefox/start/central.html, with html.
Check other centrals, too, please.

There is a smaller set of locales that have 

http://www.mozilla-europe.org/de/products/firefox/tabs/
http://www.mozilla-europe.org/de/products/firefox/search/
http://www.mozilla-europe.org/de/products/firefox/live-bookmarks/

So, check for those as well.

Comment 6

[Anne (:annevk)]

(In reply to comment #5)
> ast should link to
> http://www.mozilla-europe.org/ast/products/firefox/start/central.html

It does. I only removed the extension for mozilla.org.


> http://www.mozilla-europe.org/de/products/firefox/tabs/
> http://www.mozilla-europe.org/de/products/firefox/search/
> http://www.mozilla-europe.org/de/products/firefox/live-bookmarks/
> 
> So, check for those as well.

Again, I only checked mozilla.org. Or do you mean that they are currently
linking to mozilla.org but should not?

Comment 7

[Daniel C]

http://www.mozilla-europe.org/en/products/firefox/tabs/
http://www.mozilla-europe.org/en/products/firefox/search/
http://www.mozilla-europe.org/en/products/firefox/live-bookmarks/

This is how mozilla.org should do it IMO.

http://www.mozilla-europe.org/en/products/firefox/start/central.html - You could
even turn this into
http://www.mozilla-europe.org/en/products/firefox/start/central/ (index.html)

Comment 8

[Anne (:annevk)]

DJC, could you please stop adding useless comments. File a bug of your own if
you don't agree with the way mozilla.org works. This is not the right place.

Comment 9

[Anne (:annevk)]

Attachment: does the same plus it fixes the tr locale

Comment 10

[Daniel C]

This bug fixes some important things (CTRL- and 1.0 for example.) I agree with
these, but this bug also changes URIs which already worked, into shorter but
worse URIs. I don't agree with this, surely this is the place to express that? I
mentioned mozilla.org's server configuration because that would be another way
to do what you want with the URIs, but a better way. I used mozilla-europe as as
example of a good way to do this, after seeing it in Axel's links.

As for 'we do it with /support' I'm not sure what you mean.
http://www.mozilla.org./support/ is a directory, not a file. Can you explain to
me what you mean please? http://www.mozilla.org/products/firefox/support/ is
also a directory.

I may file another bug for the entire documentation and collection of webpages
to use better URIs if I find that directories/ are being linked to without the
trailing slash, or files are being linked to without the file extension. I
apologise if this is again a useless comment, although I don't really think it is.

DJC

Comment 11

[Anne (:annevk)]

Example:
 <http://www.mozilla.org/support/firefox/tips>

No please don't comment anymore unless you have something to say specifically
about this bug. I'm sure reviewers of the patch will take your comments into
consideration.

Comment 12

[:Gavin Sharp [email: [e-mail]]]

DJC,
You might want to take a look at http://www.w3.org/Provider/Style/URI.html ,
specifically the "What to leave out" section of "Designing URIs".

Comment 13

[fantasai]

Comment on attachment 180049 [details] [diff] [review]
does the same plus it fixes the tr locale

Assuming you're sure about the mozilla.org -> mozilla-europe changes for tr. (I
don't know what they're supposed to be.)

Comment 14

[Anne (:annevk)]

Fix checked in. The change to the tr locale was made per discussion with pike
and after reviewing the pages on Mozilla Europe and see what similar locales did.

Comment 15

[Anne (:annevk)]

Actually, I'm going to mark this fixed. If other people feel more changes have
to made to the snippets, file follow-up bugs and CC me.