Closed Bug 224569 Opened 21 years ago Closed 21 years ago

Cookie Manager and Exceptions window should remember column size

Categories

(Firefox :: Menus, enhancement)

Product:
Firefox
Component:
Menus
Platform:
x86
All
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: steffen.wilberg, Assigned: steffen.wilberg)

Details

Attachments

(2 files)

patch
3.15 KB, patch
p_ch
: review+
Details | Diff | Splinter Review
seamonkey fix
3.01 KB, patch
neil
: review+
dwitte
: superreview+
Details | Diff | Splinter Review 
The Cookie Manager and the Cookie Expeptions window should remember their column
sizes.

I know there are suggestions for a complete redesign of these windows, but they
don't seem to be ready yet.
Patch coming up.
Attached patch patch — — Splinter Review 
I also moved the editor comment to a new line, starting with a #, so that the
preprocessor removes it.
In CookieViewer.xul, the default width of the domain column should be somewhat
larger; I've seen pretty long domain names like www68.americanexpress.com. I
suggest 4:5 instead of 6:14.
Comment on attachment 134702 [details] [diff] [review]
patch

Pierre, can you have a look at my little patch please? Pretty simple. We did
the same in the Password Manager by the way, bug 217147. This should really be
consistent.
Attachment #134702 - Flags: review?(p_ch)
checked in, thanks!
I tweaked a bit more the flex values.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Attachment #134702 - Flags: review?(p_ch) → review+
so, this is applicable to seamonkey too, right?
Sure. It should be
mozilla/extensions/wallet/cookieviewer/resources/content/CookieViewer.xul.
Just add persist="width" to all the treecols. Change the flex as well. I can
provide a patch if you want. But please help yourself. :)
verified fixed 2003-11-09
Status: RESOLVED → VERIFIED
reopening for seamonkey fix.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Attached patch seamonkey fix — — Splinter Review 
adds persist to cookieviewer/cookiesites tabs.
Attachment #135248 - Flags: superreview?(alecf)
Attachment #135248 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 135248 [details] [diff] [review]
seamonkey fix

sr=alecf
Attachment #135248 - Flags: superreview?(alecf) → superreview+
Attachment #135248 - Flags: superreview?(alecf)
Attachment #135248 - Flags: superreview+
Attachment #135248 - Flags: review?(neil.parkwaycc.co.uk)
Attachment #135248 - Flags: review+
Comment on attachment 135248 [details] [diff] [review]
seamonkey fix

re-setting sr= flag.
Attachment #135248 - Flags: superreview?(alecf) → superreview+
fixed for seamonkey.
Status: REOPENED → RESOLVED
Closed: 21 years ago → 21 years ago
Resolution: --- → FIXED
verified.
Status: RESOLVED → VERIFIED
QA Contact: bugzilla → menus
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: