Closed
Bug 224569
Opened 21 years ago
Closed 21 years ago
Cookie Manager and Exceptions window should remember column size
Categories
(Firefox :: Menus, enhancement)
Tracking
()
VERIFIED
FIXED
People
(Reporter: steffen.wilberg, Assigned: steffen.wilberg)
Details
Attachments
(2 files)
3.15 KB,
patch
|
p_ch
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•21 years ago
|
||
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.
Assignee | ||
Comment 2•21 years ago
|
||
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)
Comment 3•21 years ago
|
||
checked in, thanks!
I tweaked a bit more the flex values.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•21 years ago
|
Attachment #134702 -
Flags: review?(p_ch) → review+
Comment 4•21 years ago
|
||
so, this is applicable to seamonkey too, right?
Assignee | ||
Comment 5•21 years ago
|
||
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. :)
Comment 7•21 years ago
|
||
reopening for seamonkey fix.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 8•21 years ago
|
||
adds persist to cookieviewer/cookiesites tabs.
Updated•21 years ago
|
Attachment #135248 -
Flags: superreview?(alecf)
Attachment #135248 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 9•21 years ago
|
||
Comment on attachment 135248 [details] [diff] [review]
seamonkey fix
sr=alecf
Attachment #135248 -
Flags: superreview?(alecf) → superreview+
Updated•21 years ago
|
Attachment #135248 -
Flags: superreview?(alecf)
Attachment #135248 -
Flags: superreview+
Attachment #135248 -
Flags: review?(neil.parkwaycc.co.uk)
Attachment #135248 -
Flags: review+
Comment 10•21 years ago
|
||
Comment on attachment 135248 [details] [diff] [review]
seamonkey fix
re-setting sr= flag.
Attachment #135248 -
Flags: superreview?(alecf) → superreview+
Comment 11•21 years ago
|
||
fixed for seamonkey.
Status: REOPENED → RESOLVED
Closed: 21 years ago → 21 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
QA Contact: bugzilla → menus
You need to log in
before you can comment on or make changes to this bug.
Description
•