Closed Bug 323842 Opened 15 years ago Closed 15 years ago
Add "Allow for session" option in cookie acceptance sheet
Instead of the full monty of a disclosure triangle, etc. being added to the cookie acceptance sheet, we should simply have an "Allow for session" button, as requested by bug 173521 comment 7 and bug 173521 comment 9 (as well as multiple requests elsewhere). This adds only a modicum of complexity, and as has been pointed out elsewhere, will not even be visible to novice users unless "Ask before accepting" is checked in the privacy prefs. Patch is already done and will be coming along shortly.
Accepting. Note about the patch: there were a *lot* of minor whitespace changes that do nothing as far as code goes. They're in the patch but if a reviewer wants them out, I can re-diff. cl
Status: NEW → ASSIGNED
Minor error: "Session" should be capitalised. I can upload a new patch, or once this is reviewed, the check-in person can do it. cl
For non-coders who are interested in what this looks like. cl
Comment on attachment 208938 [details] [diff] [review] fixes broken sort, rearranges popup menu items to sort properly Errrr...ignore that patch. It's for the related bug 302865 ;) cl
Comment on attachment 208836 [details] [diff] [review] Fixes capitalisation, cleans up logic per Stuart's comments on IRC ...and this isn't obsolete, either. cl
Comment on attachment 208836 [details] [diff] [review] Fixes capitalisation, cleans up logic per Stuart's comments on IRC r=me. Works as advertized. This shouldn't be checked in without the patch for bug 302865, or the policy-editing table will be wrong. Also needs "Allow for Session" added to the Localizable.strings when commited.
Also fixes a couple style issues earlier in the file. cl
Comment on attachment 210056 [details] [diff] [review] Updated for bitrotting, logic clean-up No tabs please.
Attachment #210056 - Flags: superreview?(sfraser_bugs) → superreview-
Aaaaargh! Bloody hell. Sorry for the bugspam. cl
Target Milestone: Camino1.0 → Camino1.1
Comment on attachment 210064 [details] [diff] [review] really tab-less version of previous previous patch Some slacker took too long reviewing this, and it bitrotted. Please re-spin, and I promise to be a bit more prompt with that version.
I coulda sworn I uploaded this two days ago, but apparently not. cl
Comment on attachment 213542 [details] [diff] [review] updated for bitrot r=me with the conditions that: - bug 302865 lands first/at the same time - the commiter makes the strings addition
Checked in on the trunk and 1.8 branch. (Someone better begin keeping track of stuff for the 1.1 release notes.)
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Verified on 1.8 branch and trunk
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.