Minor tweaks to cookies docs

RESOLVED FIXED in Firefox1.0beta

Status

Firefox Graveyard
Help Documentation
--
trivial
RESOLVED FIXED
14 years ago
2 years ago

People

(Reporter: Jeff Walden (gone starting June 8), Assigned: Jeff Walden (gone starting June 8))

Tracking

({fixed-aviary1.0})

1.0 Branch
Firefox1.0beta
fixed-aviary1.0

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
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.
(Assignee)

Updated

14 years ago
Flags: blocking-aviary1.0PR?

Comment 1

14 years ago
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.

Updated

14 years ago
Flags: blocking-aviary1.0PR?
(Assignee)

Comment 2

14 years ago
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.
(Assignee)

Updated

14 years ago
Flags: blocking-aviary1.0PR?
(Assignee)

Comment 3

14 years ago
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 4

14 years ago
Comment on attachment 155644 [details] [diff] [review]
Patch - also contains stuff from patch for bug 249231

Looks good! r=rlk@trfenv.com
Attachment #155644 - Flags: review?(rlk) → review+

Comment 5

14 years ago
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
(Assignee)

Updated

14 years ago
Flags: blocking-aviary1.0PR?
(Assignee)

Updated

14 years ago
Target Milestone: --- → Firefox1.0beta
(Assignee)

Updated

14 years ago
Keywords: fixed-aviary1.0
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.