In the Managing Cookies section, the screenshot is for the Exceptions dialog when it should be for the Stored Cookies dialog. The Exceptions dialog image should be moved to underneath the "The Site List" header. I don't think we can get a good screenshot of the Stored Cookies dialog because to do so would require displaying a private cookie (although if a mozilla.org site does this, it might be doable - likely not in time for 1.0, tho, unless someone wants to expend the effort). In the "The Site List" section, the button to access the Exceptions dialog is referred to as "Exceptions...", when it's really the "Exceptions" button. It's probably a good idea to also document the UI in the Exceptions dialog for adding sites to blocked/allowed lists. This should be relatively quick and painless. Because it's a l10n change, I'm nominating it for blocking-aviary1.0PR status in a sec. I should be able to post a patch sometime tonight.
Now or never Jeff :). We're getting close to 1.0 and unless we have a error in the docs, this won't block 1.0RC1.
Created attachment 155644 [details] [diff] [review] Patch - also contains stuff from patch for bug 249231 Everything's fixed in the patch. It turns out the image I said was wrong should have been switched with the screenshot for the section below it (which currently isn't on the site, so it wasn't displaying). Exceptions... is now Exceptions. I also added a quick blurb about adding sites through the UI. It's modeled on the deletion section for consistency. Now that there's a patch (I forgot about it after posting, hence my delay in creating a patch), I'll renominate it for blocking1.0PR in a sec.
Comment on attachment 155644 [details] [diff] [review] Patch - also contains stuff from patch for bug 249231 Heh...I do want a review, don't I? Mustn't forget that important step...
Attachment #155644 - Flags: review?(rlk)
Comment on attachment 155644 [details] [diff] [review] Patch - also contains stuff from patch for bug 249231 Looks good! firstname.lastname@example.org
Attachment #155644 - Flags: review?(rlk) → review+
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox1.0beta
You need to log in before you can comment on or make changes to this bug.