Closed Bug 654680 Opened 8 years ago Closed 8 years ago

Remove useless section on Cookie notification in the Status bar.

Categories

(SeaMonkey :: Help Documentation, defect)

defect
Not set

Tracking

(seamonkey2.4 fixed)

RESOLVED FIXED
seamonkey2.4
Tracking Status
seamonkey2.4 --- fixed

People

(Reporter: philip.chee, Assigned: ewong)

References

(Depends on 1 open bug)

Details

Attachments

(1 file, 2 obsolete files)

+++ This bug was initially created as a clone of Bug #249744 +++

> In Contents | Customizing Mozilla | Toolbars | Status Bar, there should be an
> image of the cookie notification icon, but instead the alternate text is shown.
> The URI for the image should be
> "chrome://communicator/skin/cookie/taskbar-cookie.gif", and not
> "chrome://cookie/content/taskbar-cookie.gif".

Neither exist in SeaMonkey 2.0+. In fact the cookie notification doesn't exist either.

> neil@parkwaycc.co.uk      2011-05-03 14:27:56 PDT
> 
> (In reply to comment #64)
> > In fact I don't think that the statusbar cookie panel still exists. That help
> > section should just be deleted.
> 
> Probably went away when p3p did, or something...
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #539162 - Flags: review?(jh)
Comment on attachment 539162 [details] [diff] [review]
Removed Cookie notification section from help.

Review of attachment 539162 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the nits addressed.

::: suite/locales/en-US/chrome/common/help/suite-toc.rdf
@@ -865,1 @@
>  

Please remove the empty line, too.

::: suite/locales/en-US/chrome/common/help/using_priv_help.xhtml
@@ -184,5 @@
>      <li><a href="#cookies">Privacy &amp; Security Preferences -
>        Cookies</a></li>
>      <li><a href="#stored_cookies">Stored Cookies</a></li>
>      <li><a href="#cookie_sites">Cookie Websites</a></li>
> -    <li><a href="#cookie_notification">Cookie Notification</a></li>

There's another reference to this in help-index1.rdf, which also needs to be removed.
Attachment #539162 - Flags: review?(jh) → review+
Attachment #539162 - Attachment is obsolete: true
Attachment #539483 - Flags: review?(jh)
Comment on attachment 539483 [details] [diff] [review]
Removed Cookie Notification from Help (v2)

This time you forgot the using_priv_help.xhtml change. r=me with that.
Attachment #539483 - Flags: review?(jh) → review+
Attachment #539483 - Attachment is obsolete: true
Attachment #539519 - Flags: review?(jh)
Comment on attachment 539519 [details] [diff] [review]
Removed Cookie Notification from Help (v3) [Checkin: comment 7]

BTW: "r=me with X" means that if you provide a patch with X, *you* can set r+ and be done with it. Anyone can see who originally set r+ by going through the obsolete attachments. :-)
Attachment #539519 - Flags: review?(jh) → review+
Keywords: checkin-needed
Comment on attachment 539519 [details] [diff] [review]
Removed Cookie Notification from Help (v3) [Checkin: comment 7]

http://hg.mozilla.org/comm-central/rev/405de702239b

Feel free to request aurora/beta approval, both of which are currently closed.
Attachment #539519 - Attachment description: Removed Cookie Notification from Help (v3) → Removed Cookie Notification from Help (v3) [Checkin: comment 7]
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.4
Version: SeaMonkey 2.0 Branch → Trunk
You need to log in before you can comment on or make changes to this bug.