Closed Bug 168930 Opened 23 years ago Closed 22 years ago

Cookie accept/decline alert buttons are in the wrong order

Categories

(Core :: Networking: Cookies, defect, P4)

PowerPC
Mac System 9.x
defect

Tracking

()

VERIFIED FIXED
mozilla1.3beta

People

(Reporter: mpt, Assigned: skasinathan)

References

()

Details

(Keywords: dataloss, regression)

Attachments

(3 files)

Build: 2002091503, Mac OS 9.2.2 To reproduce: 1. Set the pref to be asked whenever a new cookie is set (`Edit' > `Preferences...' > `Privacy & Security' > `Cookies' > `Ask me before storing a cookie'. 2. Go to a Web page which tries to set a new cookie. 3. Look at the alert. What you should see: * The button for accepting the cookie should be in the corner of the alert. What you actually see: * The button for accepting the cookie is second from the left. This is contrary to the Human Interface Guidelines (the affirmative button should be in the right corner). It is also the opposite of the previous (correct) button ordering, so that people who routinely declined cookies will now be accepting them by mistake, and vice versa.
Keywords: dataloss, regression
Status: NEW → ASSIGNED
Priority: -- → P4
Target Milestone: --- → mozilla1.3beta
-> suresh
Assignee: morse → suresh
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Attached patch patch attachedSplinter Review
1. This patch makes Cookie accept dialog to be consistent with rest of the mozilla dialogs, i.e Ok and Cancel buttons appears at the bottom. 2. Also, on mac it appears the correct way. (Cancel, OK).
Attachment #111639 - Flags: superreview?(jaggernaut)
Attachment #111639 - Flags: review?(danm)
Comment on attachment 111639 [details] [diff] [review] patch attached sr=jag
Attachment #111639 - Flags: superreview?(jaggernaut) → superreview+
this is fixed in trunk.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
this caused bug 189522... please don't tell me that the change to "ok/cancel" from "yes/no" is wanted...
Comment on attachment 111639 [details] [diff] [review] patch attached removing obso9lete review request
Attachment #111639 - Flags: review?(danm)
V/fixed. CFM RIP anyhow.
Status: RESOLVED → VERIFIED
V/fixed. CFM RIP anyhow.
QA Contact: tever → benc
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: