Closed
Bug 337807
Opened 18 years ago
Closed 18 years ago
http://www.mozilla.org/credits/ is not valid HTML
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cameron, Assigned: cameron)
References
()
Details
Attachments
(2 files, 1 obsolete file)
8.66 KB,
patch
|
spam
:
review+
|
Details | Diff | Splinter Review |
1.17 KB,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
As noted at http://www.mozilla.org/contribute/writing/guidelines#validation - all mozilla pages should validate as HTML 4.01 Strict.
Moved styles inside <style> tag, enclosed paragraphs in <p> ... </p>, made all html tags lowercase, put the <a name> tags around the titles, encoded ampersands in the cvslog url, and added a trailing </html> tag. Gee that was hard work *wipes brow* Clearly I'm worthy of being included in this list myself... not :)
Attachment #221888 -
Flags: review?
Comment 2•18 years ago
|
||
--> mozilla.org :: www.mozilla.org
Component: *.mozilla.org → www.mozilla.org
Product: Websites → mozilla.org
Version: unspecified → other
Updated•18 years ago
|
QA Contact: other-mozilla-org → www-mozilla-org
Comment 3•18 years ago
|
||
Credits is gerv's domain.
Attachment #221888 -
Flags: review? → review-
Encoded the space in the subject for the email link. Also adjusted it so there's no lines beginnign with " First one r- is from RSZ.
Attachment #221888 -
Attachment is obsolete: true
Attachment #221889 -
Flags: review?(bmo)
Comment 5•18 years ago
|
||
Comment on attachment 221889 [details] [diff] [review] Take 2 Good job, thanks.
Attachment #221889 -
Flags: review?(bmo) → review+
RSZ checked this in :)
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment 7•18 years ago
|
||
It is generally not done to check into a page which is clearly owned by someone, before they have said it's OK. To see whether a page is clearly owned, look at the CVS log, e.g.: http://bonsai-www.mozilla.org/cvslog.cgi?file=mozilla-org/html/credits/index.html&rev=&root=/cvsroot/ No harm done in this case, apart from the fact that the CVS log now looks a little uglier, but something to bear in mind for the future. Gerv
Comment 8•18 years ago
|
||
This fixes a couple of issues that were introduced in rev 1.265 including adding a </p>, a </body>, and changing & signs to &.
Attachment #224648 -
Flags: review?(gerv)
Comment 9•18 years ago
|
||
Comment on attachment 224648 [details] [diff] [review] Fix Validation issues r=gerv. Gerv
Attachment #224648 -
Flags: review?(gerv) → review+
Comment 10•18 years ago
|
||
Checked in. Checking in index.html; /cvsroot/mozilla-org/html/credits/index.html,v <-- index.html new revision: 1.269; previous revision: 1.268 done Gerv
Status: REOPENED → RESOLVED
Closed: 18 years ago → 18 years ago
Resolution: --- → FIXED
Comment 12•18 years ago
|
||
The initial checkin in this bug was not an exact checkin of the attached patch. Specifically, the heading for the "E" section was removed in the checked-in version, while it is still present in the attached patch. Can the patch author (cameron) and the checker-in (bmo@rsz.jp) figure out how this happened, and make sure it doesn't happen again? :-) I have fixed the "E" problem with a subsequent checkin: Checking in index.html; /cvsroot/mozilla-org/html/credits/index.html,v <-- index.html new revision: 1.270; previous revision: 1.269 done Thanks, Gerv
Updated•16 years ago
|
Product: mozilla.org → Websites
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•