Closed Bug 218878 Opened 21 years ago Closed 21 years ago

Cleanup of help RDF

Categories

(SeaMonkey :: Help Documentation, defect, P2)

Product:
SeaMonkey
Component:
Help Documentation
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.7alpha

People

(Reporter: rjkeller, Assigned: rjkeller)

References

Details

Attachments

(2 files)

Patch - TOC cleanup
212.85 KB, patch
alecf
: review+
Details | Diff | Splinter Review
Patch - Index cleanup
155.69 KB, patch
alecf
: review+
Details | Diff | Splinter Review
Help RDF is dirty and needs to be cleaned.
Target Milestone: --- → mozilla1.6alpha
Accepting
Status: NEW → ASSIGNED
HUGE toc cleanup. shaved about 20K off of it!
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup jag, could you review this? This doesn't need sr=. HUGE cleanup of the help TOC.
Attachment #131283 - Flags: review?(jag)
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup neil, can you review this?
Attachment #131283 - Flags: review?(jag) → review?(neil.parkwaycc.co.uk)
Just to note: All of the <rdf:Description about="urn:root"> are moved to the top of the file, if you're wondering where they went.
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup jag, can you get to this? Neil says he doesn't know enough RDF to do this.
Attachment #131283 - Flags: review?(neil.parkwaycc.co.uk) → review?(jag)
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup timeless, I'm going to ask if you could review this. The reason is that you complaining about the help RDF was the reason why I filed this and fixed it.
Attachment #131283 - Flags: review?(jag) → review?(timeless)
Severity: normal → trivial
Priority: -- → P2
*** Bug 219924 has been marked as a duplicate of this bug. ***
timeless, can you review this?
QA Contact: tpreston → stolenclover
timeless, can you review?
Stephen, can you review this? I'm running out of reviewers to ask ;) I know that it's a challenging patch.
Attachment #131283 - Flags: review?(timeless) → review?(technutz)
The list of super-reviewers is http://www.mozilla.org/hacking/reviewers.html#the-super-reviewers If any of those folks are no longer doing super-reviews (at all, as opposed to taking a vacation or being swamped) please let staff@mozilla.org know. You can use the "requests" link in the bugzilla footer to avoid reviewers who are totally swamped with requests (e.g. sspitzer).
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup Clearing my review request. Timeless said he'd get to this in a bit.
Attachment #131283 - Flags: review?(technutz) → review?
Attachment #131283 - Flags: review? → review?(alecf)
Comment on attachment 131283 [details] [diff] [review] Patch - TOC cleanup rs=alecf
Attachment #131283 - Flags: review?(alecf) → review+
Checked in! What a cleanup! We lost over 2,500 lines of RDF!
Blocks: 222178
Depends on: 95770
Blocks: 67376
Target Milestone: mozilla1.6alpha → mozilla1.7alpha
This patch cleans up the index and fixes bug 222178.
Attachment #137816 - Flags: review?(alecf)
Comment on attachment 137816 [details] [diff] [review] Patch - Index cleanup r=alecf
Attachment #137816 - Flags: review?(alecf) → review+
OK, my cleanup plan has been completely checked in! 4,415 lines of useless RDF removed! phew.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Typo fixed. Thanks for telling me about it!
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: