339655 - Missing space between "License" and "document" in about:

Status: RESOLVED DUPLICATE of bug 328563

(Firefox :: General, defect)

Description

[dolphinling]

http://lxr.mozilla.org/mozilla/source/toolkit/content/about.xhtml#97

Line 97 reads 
  <a href="about:license">Mozilla Public License &license.and; Netscape Public License</a>&license.part3;

There needs to be a space after the </a>

Comment 1

[dolphinling]

Attachment: patch

Patch. Untested. :) Hope I guessed right about who to request review from.

Apparently the word "document" was added since the 1.8 branch, so this patch would only apply to trunk. Another fix would be to edit the dtd with the entities to have a space in that string.

Comment 2

[dolphinling]

Since I may not have been clear before:
The about: page, when viewed, reads "... Netscape Public Licensedocument. All Rights Reserved." License is hyperlinked and document is not, but there still needs to be a space between them.

Comment 3

[dolphinling]

Part of comment 1 is wrong: this applies to the 1.8 branch as well as the trunk (but not the 1.8.0 branch).

Comment 4

[Steffen Wilberg]

In bug 328563 comment 1, Gerv said:
> I didn't know about the existence of this (about:) page. Really, all licensing
> information (including those sentences and the NPL reference) needs to be
> removed, and replaced with a single link to about:licence.
That's what bug 328563 is about (if not removing about: altogether). Feel free to take that bug.

And I only do reviews in Firefox Help Documentation. [e-mail] would be appropriate.

Comment 5

[dolphinling]

Comment on attachment 223752 [details] [diff] [review]
patch

I won't take bug 328563 because I'd have no clue what to do with it. If people want this to just be part of that, fine.

Comment 6

[Steffen Wilberg]

The linked document (about:license) doesn't take you to a "Mozilla Public License and Netscape Public License document". That document contains all the relevant licenses, but the NPL isn't one of them anymore.

Comment 7

[dolphinling]

Page was redone (in bug 328563), this bug now irrelevant. 

*** This bug has been marked as a duplicate of 328563 ***