Closed
Bug 221260
Opened 22 years ago
Closed 22 years ago
p3p isn't in the glossary
Categories
(Documentation Graveyard :: Help Viewer, enhancement)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: rjkeller)
Details
Attachments
(1 file, 2 obsolete files)
|
3.57 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
this is not an endorsement of p3p, however if we're going to have the silly
thing in some flavor of our product then we should probably mention it in the
glossary. (it can say that "people don't think it will help" if you want to
appease me personally, but that's not a requirement for resolving this bug.)
ok, this is inaccurate.
SSL is listed twice:
SSL
and
Secure Sockets Layer (SSL)
P3P is only listed under "Platform for Privacy Preferences (P3P)"
| Assignee | ||
Comment 2•22 years ago
|
||
We should have it under P3P too, though.
Confirming bug.
-> Doc: User
Status: UNCONFIRMED → NEW
Component: Help → User
Ever confirmed: true
Product: Browser → Documentation
Version: Trunk → unspecified
| Assignee | ||
Comment 3•22 years ago
|
||
| Assignee | ||
Updated•22 years ago
|
Attachment #132673 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 4•22 years ago
|
||
Comment on attachment 132673 [details] [diff] [review]
Patch
Shouldn't the p3p entry say <b id="p3p">P3P</b> see ...
Nit: neaten up the line wrapping.
Attachment #132673 -
Flags: review?(neil.parkwaycc.co.uk) → review-
| Assignee | ||
Comment 5•22 years ago
|
||
bah. I think I'm losing my ability to type. This is my third patch today that I
goofed up the text. New patch coming ;(
| Assignee | ||
Comment 6•22 years ago
|
||
sorry about that, Neil.
Attachment #132673 -
Attachment is obsolete: true
| Assignee | ||
Updated•22 years ago
|
Attachment #132680 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 7•22 years ago
|
||
Comment on attachment 132680 [details] [diff] [review]
Patch with Neils comments
OK, r=me if a) you put a full stop after the P3P b) use a capital S for see c)
use a capital P for preferences d) put (P3P) after preferences. If you want to
know why, take a look at the SSL entry.
Attachment #132680 -
Flags: review?(neil.parkwaycc.co.uk) → review+
Comment 8•22 years ago
|
||
Comment on attachment 132680 [details] [diff] [review]
Patch with Neils comments
Wait a sec, there a should also be a </a> before the .
SSL turned out to be a bad example. It appears to link to itself :-(
Attachment #132680 -
Flags: review+ → review-
| Assignee | ||
Comment 9•22 years ago
|
||
Attachment #132680 -
Attachment is obsolete: true
| Assignee | ||
Updated•22 years ago
|
Attachment #132704 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 10•22 years ago
|
||
Comment on attachment 132704 [details] [diff] [review]
Patch 3
Looks OK now.
Attachment #132704 -
Flags: review?(neil.parkwaycc.co.uk) → review+
| Assignee | ||
Comment 11•22 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 12•22 years ago
|
||
The glossary should link to using_priv_help.html#privacy_levels_window Keller,
can you fix this?
| Reporter | ||
Comment 13•22 years ago
|
||
vrfy fixed 2003102804
daniel: you can file a new bug for that, i'm satisfied :).
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•