XPCOM docs are poorly formed HTML wise

VERIFIED WONTFIX

Status

()

Core
XPCOM
--
minor
VERIFIED WONTFIX
9 years ago
9 years ago

People

(Reporter: Tyler, Assigned: Tyler)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

9 years ago
The docs use old <i>, <b> tags, and are not XHTML complaint.
(Assignee)

Comment 1

9 years ago
Created attachment 351103 [details] [diff] [review]
Readme Patch v1

This simply brings the readme to compliance with the W3C XHTML standard.
Attachment #351103 - Flags: review?(shaver)
(Assignee)

Comment 2

9 years ago
Created attachment 351104 [details] [diff] [review]
Docs Patch v1

This is a separate patch for the docs, doing much the same thing as above.
Attachment #351104 - Flags: review?(shaver)

Comment 3

9 years ago
Comment on attachment 351103 [details] [diff] [review]
Readme Patch v1

Changes of this sort are not worth accepting. Please don't use XHTML unless absolutely necessary for some other reason.
Attachment #351103 - Flags: review?(shaver)

Updated

9 years ago
Attachment #351104 - Flags: review?(shaver)

Updated

9 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
(Assignee)

Comment 4

9 years ago
Well, if you do not want to update these to XHTML for some reason, then maybe the content should be brought up to date?
As configure says, 
>Apply the patch in bugzilla bug 129216</a>. (If this bug is marked as FIXED, it >probably means there is no need to apply the patch and that these docs are out of
>date)
That bug is FIXED, so this reference could at least be removed. At least this is what it seems like.

Comment 5

9 years ago
Yes, content fixes are welcome. Please don't mix them up with markup fixes, which are usually unwelcome. Although it probably makes sense to remove these files from version control and put them up on developer.mozilla.org
(Assignee)

Comment 6

9 years ago
 Bug 470575 is one moving to dmo.
(Assignee)

Updated

9 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.