Closed
Bug 390158
Opened 17 years ago
Closed 15 years ago
Hook up new satchel pref(s) to the preferences window (and clear data?)
Categories
(SeaMonkey :: Preferences, defect)
SeaMonkey
Preferences
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0
People
(Reporter: standard8, Assigned: iannbugzilla)
References
(Blocks 1 open bug)
Details
(Keywords: fixed-seamonkey2.0)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
iannbugzilla
:
review+
iannbugzilla
:
superreview+
kairo
:
approval-seamonkey2.0+
|
Details | Diff | Splinter Review |
Following the switch over to satchel, we need to hook up browser.formfill.enable in the preferences somewhere.
We probably also want to hook up an option for clearing the stored data, like FF has.
Updated•17 years ago
|
Depends on: prefpane_migration
Comment 1•16 years ago
|
||
(Filter "spam" on 'prefs-nobody-20080612'.)
Assignee: prefs → nobody
QA Contact: prefs
Comment 2•16 years ago
|
||
Prior to the switch to satchel, there was UI that enabled a user to edit
his profile's form fill-in data. Now, as far as I know, there is no such
UI. The amount of stored data grows and grows, and there is no way
(known to me) to remove items from that set of stored data. It has been
this way for almost a year now. (I'd be happy to be wrong about this.)
IMO, SM2 definitely needs to give the user control over his stored form data.
Flags: blocking-seamonkey2?
Comment 3•16 years ago
|
||
Temporary workaround: Install SQLiteManager mod for SeaMonkey <http://xsidebar.mozdev.org/modifiedmisc.html#sqlitemanagerhttp://xsidebar.mozdev.org/modifiedmisc.html#sqlitemanager> and edit the formhistory.sqlite file.
Comment 4•16 years ago
|
||
You can however clear one form autocomplete item from the list at a time by highlighting it and pressing Shift+Delete.
Comment 5•16 years ago
|
||
(In reply to comment #4)
> You can however clear one form autocomplete item from the list at a time by
> highlighting it and pressing Shift+Delete.
Neil, Perhaps you meant that in some context other than the form in which
the fields are being filled in?
The following steps (which you seem to be suggesting) certainly do NOT
cause the selected entry to be deleted.
1. Go to a form page
2. Start to type in a value into a field
3. See a drop-down list of choices
4. Select one of those choices
5. Press Shift-Delete.
For example, in this (or any) bugzilla bug page, go to the bug number
field (by the "find" button at the top of the page) and type a single
character, either "1" or "2". A list of previously entered numbers appears
in a drop down list. Select one from the list. As soon as you highlight
it, the value is placed into the text box and the drop-down list disappears.
Now, press shift+Delete, or if you like, highlight the text in the text box
and press shift+Delete. This has no effect on the saved values for that field.
Comment 6•16 years ago
|
||
I'm pretty certain that, in comment 3, Phillip meant to enter this URL:
http://xsidebar.mozdev.org/modifiedmisc.html#sqlitemanager
Comment 7•16 years ago
|
||
What Neil probably meant is go to the form field, press the down arrow (or type something, so that dropdown appears), press the down/up arrows until the entry you want is highlighted. You can press delete or shift+delete there, and the entry is gone from both the dropdown and the satchel database.
Of course, with the "Clear Private Data" function, you also delete the whole form history database if you want.
Comment 8•16 years ago
|
||
I tried the SQLite manager mod extension. Found I had almost 6000 entries
saved. I was able to delete over half of them, although it was a bit tedious
because you cannot delete more than about 80 a time (UI limitation), and you
must re-sort and re-scroll the table after each deletion. Still, this is
infinitely better than nothing! Phil, is there a bug DB for that extension?
By examining the table, I found that every time I submit a form, all the
fields, including hidden fields and fields whose values are always set
programmatically by JavaScript (not user entered) are stored in the table.
This table is a high value target for attackers, since it tends to contain
all info (sensitive or not) ever entered into any form.
Comment 9•16 years ago
|
||
> Phil, is there a bug DB for that extension?
It's somewhere here http://code.google.com/p/sqlite-manager/
I haven't been keeping up with the latest versions of that extension, so your problem might have been fixed in the latest official releases. Certainly the author has been hyperactive in fixing bugs and adding features.
Comment 10•16 years ago
|
||
In fact the latest version are compatible with SeaMonkey 2.0a!
Comment 12•16 years ago
|
||
browser.formfill.expire_days is another satchel pref that should be available ny itself in UI.
Updated•16 years ago
|
Assignee | ||
Comment 14•15 years ago
|
||
This patch:
* Adds a default setting for browser.formfill.expire_days.
* Adds exposes the preferences for browser.formfill.enable/expire_days on the history pane.
* Updates the help page to reflect the change in the history pane.
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #397638 -
Flags: superreview?(neil)
Attachment #397638 -
Flags: review?(mnyromyr)
Comment 15•15 years ago
|
||
Comment on attachment 397638 [details] [diff] [review]
Add formfill prefs to history pane patch v0.1
>+ <textbox id="formfillDay"
>+ type="number"
>+ size="4"
>+ preference="browser.formfill.expire_days"/>
Nit: needs aria-labelledby (sp?)
Oh wait, the whole pane needs it. Never mind.
Attachment #397638 -
Flags: superreview?(neil) → superreview+
Updated•15 years ago
|
Attachment #397638 -
Flags: review?(mnyromyr) → review+
Comment 16•15 years ago
|
||
Comment on attachment 397638 [details] [diff] [review]
Add formfill prefs to history pane patch v0.1
>diff --git a/suite/common/pref/pref-history.xul b/suite/common/pref/pref-history.xul
While you are touching this dialog, would you agree that the "<separator class="thin" />" in line 93 is just nonsense? If so, please remove it. ;-)
>diff --git a/suite/locales/en-US/chrome/common/help/cs_nav_prefs_navigator.xhtml b/suite/locales/en-US/chrome/common/help/cs_nav_prefs_navigator.xhtml
>+ <li><strong>Enable form and search history</strong>: Select this to let
>+ &brandShortName; to keep a history of the forms you fill in and the
>+ searches you do.</li>
1 2 2 much? ;-)
One thing I find odd here is that browsing history and location bar history do have a "Clear" button, but forms history has not. This looks rather inconsequent and illogical and should be worth an extra bug. ;-)
Assignee | ||
Comment 17•15 years ago
|
||
Requesting a= for low risk patch
Attachment #397638 -
Attachment is obsolete: true
Attachment #400177 -
Flags: superreview+
Attachment #400177 -
Flags: review+
Attachment #400177 -
Flags: approval-seamonkey2.0?
Updated•15 years ago
|
Attachment #400177 -
Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
Assignee | ||
Comment 18•15 years ago
|
||
Comment on attachment 400177 [details] [diff] [review]
Add formfill prefs to history pane patch v0.1a [Checkin: Comment 18]
http://hg.mozilla.org/comm-central/rev/e010e16d7983
Attachment #400177 -
Attachment description: Add formfill prefs to history pane patch v0.1a → Add formfill prefs to history pane patch v0.1a [Checkin: Comment 18]
Keywords: fixed-seamonkey2.0
Comment 19•15 years ago
|
||
Ian, is there anything left to do here or can we resolve this bug?
Assignee | ||
Comment 20•15 years ago
|
||
(In reply to comment #19)
> Ian, is there anything left to do here or can we resolve this bug?
Just to remind myself that there was a bug to be logged on having a consistency for clear button in the history prefs pane - see bug 516511
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•