Port |Bug 628198 - Implement UI for do-not-track HTTP header| to SeaMonkey

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Preferences
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: adriank)

Tracking

Trunk
seamonkey2.1b2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Bug 628198 added UI for the privacy.donottrackheader.enabled pref to Firefox. I think we should add it as well, but possibly at a different place.

Firefox has it now under Advanced/General/Browsing. The list of checkboxes there mostly relates to our Appearance/Content.

I propose that we add it somewhere under Privary & Security because that's where it fits best. Since it doesn't really fit with the subcategories I propose we add it to the main pane, e.g. as the very first checkbox, under Private Data. It's not 100% related but IMHO good enough.

The label could be the same as in Firefox: "Tell web sites I do not want to be tracked".

Since this is probably a small change I further propose we add the Help entry at the same time, which will not only save us filing an extra bug but also having to find someone to actually write a patch for that other bug and get it reviewed.

Comment 1

7 years ago
> I propose that we add it somewhere under Privary & Security because that's
> where it fits best. Since it doesn't really fit with the subcategories I
> propose we add it to the main pane, e.g. as the very first checkbox, under
> Private Data. It's not 100% related but IMHO good enough.

That's almost exactly Alex Faaborgs proposal in attachment 506496 [details] in Bug 628198.
See bug 628198 comment 16.

Comment 2

7 years ago
I already mentioned this in #seamonkey. Too bad the logs are MIA.
(Assignee)

Comment 3

7 years ago
Created attachment 509163 [details] [diff] [review]
patch v1 (based on the Firefox patch from Bug 628198) [Checked in: Comment 6]

Not sure who is the right person to ask for review here, so if it is not you Robert, let someone else please review the patch.
Assignee: nobody → akalla
Status: NEW → ASSIGNED
Attachment #509163 - Flags: review?
(Assignee)

Updated

7 years ago
Attachment #509163 - Flags: review? → review?(kairo)
(Assignee)

Updated

7 years ago
Attachment #509163 - Flags: review?(kairo) → review?(iann_bugzilla)

Comment 4

7 years ago
> +      <!-- Do Not Track -->
> +      <preference id="privacy.donottrackheader.enabled"
Reduntant comment since the strings do not track already appear in the preference name.

> +    <!-- Do Not Track -->
> +    <groupbox id="doNotTrackGroup">

> +        <checkbox id="privacyDoNotTrackPrefs"
We seem to have flex="1" on the other checkboxes.

Comment 5

7 years ago
Comment on attachment 509163 [details] [diff] [review]
patch v1 (based on the Firefox patch from Bug 628198) [Checked in: Comment 6]

The comments can stay but as Philip suggested:
> +        <checkbox id="privacyDoNotTrackPrefs"
We seem to have flex="1" on the other checkboxes.

r=me with that fixed.
Attachment #509163 - Flags: review?(iann_bugzilla) → review+

Comment 6

7 years ago
Comment on attachment 509163 [details] [diff] [review]
patch v1 (based on the Firefox patch from Bug 628198) [Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/d1458229a862
Attachment #509163 - Attachment description: patch v1 (based on the Firefox patch from Bug 628198) → patch v1 (based on the Firefox patch from Bug 628198) [Checked in: Comment 6]

Comment 7

7 years ago
Was checked in with flex="1" added.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug]

Comment 8

7 years ago
The flex is needed for the checkbox to be able to wrap should other locales need more than one line of text. <vbox align="start"> is another approach.

Updated

7 years ago
Target Milestone: --- → seamonkey2.1b2
(In reply to comment #0)
> Since this is probably a small change I further propose we add the Help entry
> at the same time, which will not only save us filing an extra bug but also
> having to find someone to actually write a patch for that other bug and get it
> reviewed.

Not done here. Was another bug filed?
No longer blocks: 467530
(Reporter)

Updated

7 years ago
Blocks: 631566
(Reporter)

Comment 10

7 years ago
(In reply to comment #9)
> Was another bug filed?

No, filed bug 631566.
(Reporter)

Updated

7 years ago
Blocks: 631639
Depends on: 628198
You need to log in before you can comment on or make changes to this bug.