Closed
Bug 304309
Opened 19 years ago
Closed 16 years ago
convert to satchel from wallet
Categories
(SeaMonkey :: UI Design, enhancement)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wsmwk, Assigned: neil)
References
(Blocks 2 open bugs)
Details
Attachments
(14 files, 5 obsolete files)
satchel would be a nice improvement to forthcoming SM 1.1. (backport satchel from Firefox?)
Comment 1•18 years ago
|
||
*** Bug 356773 has been marked as a duplicate of this bug. ***
Comment 2•18 years ago
|
||
*** Bug 356773 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 3•17 years ago
|
||
1.5 will have satchel, I guess. I don't know whether we'll have dropped the xpfe autocomplete widget by then, but it only seems to need a few tweaks to work with satchel, which expects an nsIAutoCompletePopup; four of the functions don't exist and would need to be copied from the toolkit widget, while selectBy would need to be tweaked to work with the new signature and selectedIndex would need to be tweaked to use -1 as the sentinel instead of null.
Assignee | ||
Comment 4•17 years ago
|
||
Attachment #255487 -
Flags: review?(jag)
Comment 5•17 years ago
|
||
It seems like to pick up satchel we also need to pick up password manager, just trying to pick up satchel there appears to be internal dependencies for nsFormFillController.cpp on toolkit's pm: nsPasswordManager::AutoCompleteSearch nsFormFillController.o: In function `nsSingleSignonPrompt': /mozilla/smtest/mozilla/toolkit/components/satchel/src/../../passwordmgr/base/nsSingleSignonPrompt.h:64: undefined reference to `vtable for nsSingleSignonPrompt' The second could probably be fixed, but I don't think the first can. The question would therefore be can we pick up toolkit's pm with xpfe's autocomplete?
Comment 6•17 years ago
|
||
I've just done another build - xpfe autocomplete, satchel + toolkit password manager, no wallet. Password Manager seems to work ok, though I haven't checked multiple logins to the same web page. Satchel doesn't work as the autocompletepopup doesn't implement nsIAutoCompletePopup - as Neil commented in comment 3. Other than that, we *may* be able to take both satchel + pm for xpfe trunk builds as well as sr ones.
Comment 7•17 years ago
|
||
Something I've just noticed that would function differently with toolkit's password manager is even when you've saved the password, it will still prompt for a password again - this could be a bit annoying to the user in mail/address book, but perhaps we'd just have to fix that ;-)
Comment 8•17 years ago
|
||
Comment on attachment 255487 [details] [diff] [review] Fix selectBy I assume you'll be updating urlbarBindings.xml too. I would replace the ?: in |var reverse| with && || (idiom, and it took me a while to verify that the ?: did the same thing).
Attachment #255487 -
Flags: superreview+
Attachment #255487 -
Flags: review?(jag)
Attachment #255487 -
Flags: review+
Assignee | ||
Comment 9•17 years ago
|
||
Although I was wondering whether to move the dir/amount code into getNextIndex. Unrelated bug: when you down arrow from the search highlight it wraps to the first result but when you up arrow from the first result the highlight cancels.
Attachment #255487 -
Attachment is obsolete: true
Attachment #256477 -
Flags: superreview?(jag)
Comment 10•17 years ago
|
||
Comment on attachment 256477 [details] [diff] [review] Fixes urlbar too (d'oh) + var amount = aPage ? this.parentNode.pageCount : 1; ? this.parentNode.pageCount - 1 : 1, no?
Attachment #256477 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 11•17 years ago
|
||
I also stomped on the bug which makes down arrow from the search engine skip to selecting the first autocomplete result to match the up arrow behaviour.
Attachment #256477 -
Attachment is obsolete: true
Attachment #257013 -
Flags: superreview?(jag)
Assignee | ||
Updated•17 years ago
|
Attachment #257013 -
Flags: review?(ajschult)
Comment 12•17 years ago
|
||
Comment on attachment 257013 [details] [diff] [review] Moved aReverse and aPage into getNextIndex r=ajschult
Attachment #257013 -
Flags: review?(ajschult) → review+
Comment 13•17 years ago
|
||
OK, I have the patch from in here applied and did some modifications that should go most of the way to get pwdmgr+satchel in. Now I'm running into an exception - we're missing an autocomplete interface, it seems. This is what I get: Fehler: [Exception... "Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]" nsresult: "0x80004002 (NS_NOINTERFACE)" location: "JS frame :: chrome://global/content/bindings/browser.xml :: attachFormFill :: line 379" data: no] Quelldatei: chrome://global/content/bindings/browser.xml Zeile: 379 The missing interface is nsIAutoCompletePopup, see http://mxr.mozilla.org/mozilla/source/toolkit/content/widgets/browser.xml#379
Comment 14•17 years ago
|
||
http://ctemp.kairo.at/doc/hacking/bug304309-wip-v1.diff is a WIP patch that gets me to the point where I get the exception in comment #13 BTW, while digging into that, I saw that Thunderbird seems to still build wallet as well - that might explain why I see no password saving in mailnews (checkbox missing on password entering dialogs, no prefilled passwords used). With the patch, passwords in browser are prefilled if you have only one name/pwd saved for that site (if you once triggered the "save passwasswords?" dialog for any site), if you have multiple names/passwords, you need to enter the name and on tabbing out of the name field, the password gets prefilled. No dropdown is shown on either username or form prefilling (probably connected with that exception).
Comment 15•17 years ago
|
||
Thunderbird has bug 239131 about this, btw - just as a reference.
Comment 16•17 years ago
|
||
(In reply to comment 14) > With the patch, passwords in browser are prefilled if you have only one > name/pwd saved for that site (if you once triggered the "save passwasswords?" > dialog for any site), if you have multiple names/passwords, you need to enter > the name and on tabbing out of the name field, the password gets prefilled. No > dropdown is shown on either username or form prefilling (probably connected > with that exception). No that's standard Firefox behaviour.
Comment 17•17 years ago
|
||
KaiRo see: Firefox Bug 376668 (Improve discoverability of PW autofill with multiple accounts)
Updated•17 years ago
|
Attachment #257013 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 18•17 years ago
|
||
Since the selected index has to be an integer, this is basically an s/null/-1/ exercise. However, there are a couple of enhancements: 1. Rather than manually invalidating and firing select events, I use the tree selection to do it all for me. This saves a bunch of code, and will eventually let me s/menuactive/selected/ in the CSS, thus eliminating the menuactive atom. 2. getRowAt already returns -1 which we were originally converting to null ;-)
Attachment #266935 -
Flags: superreview?(jag)
Attachment #266935 -
Flags: review?(ajschult)
Comment 19•17 years ago
|
||
Comment on attachment 266935 [details] [diff] [review] Fix selectedIndex There's some tabs in the selection getter and setter in autocomplete.xml, and you left a dump() in the textbox property getter. sr=jag with those fixed.
Attachment #266935 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 20•17 years ago
|
||
(In reply to comment #19) >There's some tabs in the selection getter and setter in autocomplete.xml Oops. Wrong .vimrc :-( Fixed locally.
Comment 21•17 years ago
|
||
Comment on attachment 266935 [details] [diff] [review] Fix selectedIndex >Index: xpfe/components/autocomplete/resources/content/autocomplete.xml > setTree: function(aTree) > { The now-defunct mBoxObject is still referenced in this method. With that fixed, r=ajschult
Attachment #266935 -
Flags: review?(ajschult) → review+
Updated•17 years ago
|
Assignee: jag → neil
Assignee | ||
Comment 22•17 years ago
|
||
(In reply to comment #21) >(From update of attachment 266935 [details] [diff] [review]) >>Index: xpfe/components/autocomplete/resources/content/autocomplete.xml >> setTree: function(aTree) >> { >The now-defunct mBoxObject is still referenced in this method. Oops... what happened was that originally I tried to remove mAtomSelected too but I changed my mind and decided to do that in a separate patch, however I restored too much code by mistake ;-)
Assignee | ||
Comment 23•17 years ago
|
||
I discovered that the autocomplete highlighting doesn't work in Mac Suiterunner, and by extension in Mac Thunderbird. The previous patch to autocomplete.xml accidentally fixed that by making the ::-moz-tree-XXX(selected) styles work so we can remove the old menuactive support and switch to selected throughout.
Attachment #267342 -
Flags: superreview?(jag)
Attachment #267342 -
Flags: review?(mano)
Comment 24•17 years ago
|
||
(In reply to comment #23) > I discovered that the autocomplete highlighting doesn't work in Mac > Suiterunner, and by extension in Mac Thunderbird. The previous patch to > autocomplete.xml accidentally fixed that by making the > ::-moz-tree-XXX(selected) styles work so we can remove the old menuactive > support and switch to selected throughout. Neil, as I mentioned on IRC yesterday it is working fine for my trunk debug build. But there is one small nit. The last entry within the autocomplete list is cut-off at the bottom and all entries are too narrow to each other. Dunno how that was before on Mac because I have it at least for 3 month now. Would this patch also be possible for Thunderbird 2.0.0.x without big modifications? For the 1.8 branch the autocomplete is also broken for Mac.
Status: NEW → ASSIGNED
Comment 25•17 years ago
|
||
This is my current WIP version of the patch for the real switch from wallet to pwdmgr/satchel (parts of the patch are by Standard8). Note that if the implements I noted in autocomplete.xml is actually set on this line, SeaMonkey crashes when you focus a HTML textbox.
Comment 26•17 years ago
|
||
Comment on attachment 267342 [details] [diff] [review] Fix Mac themeing sr=jag
Attachment #267342 -
Flags: superreview?(jag) → superreview+
Comment 27•17 years ago
|
||
Note to self or whoever will finish up the real switch: bug 380931 has moved most LoginManager prefs to all.js, so we don't need those in browser-prefs.js any more.
Depends on: 380931
Comment 28•17 years ago
|
||
Comment on attachment 267342 [details] [diff] [review] Fix Mac themeing r=mano
Attachment #267342 -
Flags: review?(mano) → review+
Comment 29•17 years ago
|
||
Bug 375881 dependency was accidentally removed by me. Fixing it. Sorry for the problem.
Blocks: 375881
Assignee | ||
Comment 30•17 years ago
|
||
Comment on attachment 267401 [details] [diff] [review] WIP patch for the real wallet -> pwdmgr/satchel switch, v2 >+ifdef MOZ_XUL >+DIRS += \ >+ passwordmgr \ >+ satchel \ >+ $(NULL) >+ > ifndef MOZ_SUITE > # XXX Suite doesn't want these just yet >-ifdef MOZ_XUL > DIRS += \ > autocomplete \ Actually we will need all of autocomplete. (Currently we just export public.) What happened to the "Log Out" menuitem?
Comment 31•17 years ago
|
||
(In reply to comment #30) > Actually we will need all of autocomplete. OK, will include all of toolkit autocomplete in further WIP patches. > What happened to the "Log Out" menuitem? I don't know where/how to call it in toolkit. Firefox apparently doesn't have it. If it's really that useful, we should work on adding it in a followup bug.
Assignee | ||
Comment 32•17 years ago
|
||
From nsIAutoCompletePopup.idl readonly attribute AString overrideValue;
Attachment #268722 -
Flags: superreview?(jag)
Attachment #268722 -
Flags: review?(ajschult)
Updated•17 years ago
|
Attachment #268722 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 33•17 years ago
|
||
A subsequent patch will remove the other references to __AUTOCOMPLETE_BOX__
Attachment #268783 -
Flags: superreview?(jag)
Attachment #268783 -
Flags: review?(jag)
Updated•17 years ago
|
Attachment #268783 -
Flags: superreview?(jag)
Attachment #268783 -
Flags: superreview+
Attachment #268783 -
Flags: review?(jag)
Attachment #268783 -
Flags: review+
Assignee | ||
Comment 34•17 years ago
|
||
Toolkit expects the funky treecolAutoCompleteXXX names which I guess are less likely to collide with other elements (we have to use ids because of the way trees work). I also rewrote the way the comment column is shown, as toolkit seems to want to be able to set it dynamically (although I see no callers yet).
Attachment #269137 -
Flags: superreview?(jag)
Attachment #269137 -
Flags: review?(ajschult)
Comment 35•17 years ago
|
||
Comment on attachment 268722 [details] [diff] [review] Change getOverrideValue() to overrideValue >Index: suite/browser/urlbarBindings.xml >- return this.mSearchBox.getOverrideValue(); >+ return this.mSearchBox.overrideValue(); no ()
Attachment #268722 -
Flags: review?(ajschult) → review+
Comment 36•17 years ago
|
||
Comment on attachment 269137 [details] [diff] [review] Change column names >Index: xpfe/components/autocomplete/resources/content/autocomplete.xml >- this.tree.__AUTOCOMPLETE_BOX__ = this.textbox; >- this.treebody.__AUTOCOMPLETE_BOX__ = this.textbox; __AUTOCOMPLETE_BOX__ is still referenced in autocomplete-tree and autocomplete-treebody. Did you intend to also remove those?
Assignee | ||
Comment 37•17 years ago
|
||
Oops, the __AUTOCOMPLETE_BOX__ and selectedIndex changes were supposed to be part of the next patch, which reworks the way selectedIndex works ;-)
Attachment #269137 -
Attachment is obsolete: true
Attachment #269188 -
Flags: superreview?(jag)
Attachment #269188 -
Flags: review?(ajschult)
Attachment #269137 -
Flags: superreview?(jag)
Attachment #269137 -
Flags: review?(ajschult)
Assignee | ||
Comment 38•17 years ago
|
||
* Moves the selectedIndex handling from the "view" to the resultsPopup * Added a method to the "view" to handle a result click (toolkit uses this) * Makes the "view" a property on the resultsPopup as well as the tree * Removed the remaining __AUTOCOMPLETE_BOX__ variables * Removed unused bindings Note: This patch applies on top of attachment 269188 [details] [diff] [review].
Attachment #269298 -
Flags: superreview?(jag)
Attachment #269298 -
Flags: review?(ajschult)
Updated•17 years ago
|
Attachment #269188 -
Flags: review?(ajschult) → review+
Comment 39•17 years ago
|
||
Comment on attachment 269298 [details] [diff] [review] Rework the selectedIndex handling >Index: toolkit/content/xul.css toolkit's autcomplete.xml still references the bits you're removing >Index: xpfe/components/autocomplete/resources/content/autocomplete.xml >+ this.tree.view = this.textbox.view; this line was added in the previous patch
Assignee | ||
Comment 40•17 years ago
|
||
(In reply to comment #39) >(From update of attachment 269298 [details] [diff] [review]) >>Index: toolkit/content/xul.css >toolkit's autcomplete.xml still references the bits you're removing Sorry for not using a bigger -u but that's inside a suite/tb #ifdef >>Index: xpfe/components/autocomplete/resources/content/autocomplete.xml >>+ this.tree.view = this.textbox.view; >this line was added in the previous patch Then that patch was incorrect, I think. Sorry.
Updated•17 years ago
|
Attachment #269188 -
Flags: superreview?(jag) → superreview+
Comment 41•17 years ago
|
||
Comment on attachment 269298 [details] [diff] [review] Rework the selectedIndex handling Looks ok to me. Remember to remove the this.tree.view = this.textbox.view line from the other patch. sr=jag
Attachment #269298 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 42•17 years ago
|
||
Like attachment 269298 [details] [diff] [review] but with extra null-testing required because of a bug currently marked as security-sensitive.
Attachment #269298 -
Attachment is obsolete: true
Attachment #271893 -
Flags: review?(ajschult)
Attachment #269298 -
Flags: review?(ajschult)
Comment 43•17 years ago
|
||
Comment on attachment 271893 [details] [diff] [review] Rework the selectedIndex handling >Index: xpfe/components/autocomplete/resources/content/autocomplete.xml >- return this.selectedIndex = this.getNextIndex(aReverse, aPage, this.selectedIndex, this.textbox.view.rowCount - 1); >+ return this.selectedIndex = this.getNextIndex(aReverse, aPage, this.selectedIndex, this.view.rowCount - 1); This same change in urlbarBindings.xml, right? Or (with the other changes to autocomplete.xml here) you could change that to this.tree.view.rowCount, which seems toolkit-compatible.
Attachment #271893 -
Flags: review?(ajschult) → review+
Assignee | ||
Comment 44•17 years ago
|
||
Attachment #272193 -
Flags: superreview?(jag)
Attachment #272193 -
Flags: review?(ajschult)
Assignee | ||
Comment 45•17 years ago
|
||
This patch depends on attachment 272193 [details] [diff] [review] of course.
Attachment #272194 -
Flags: review?(kairo)
Comment 46•17 years ago
|
||
Comment on attachment 272193 [details] [diff] [review] Actually implement nsIAutoCompletePopup >- <xul:panel type="autocomplete" ignorekeys="true" anonid="popup" class="autocomplete-result-popup" hidden="true" xbl:inherits="for=id,nomatch"/> >+ <xul:panel type="autocomplete" anonid="popup" class="autocomplete-result-popup" xbl:inherits="for=id,nomatch"/> Hmm. removing ignorekeys doesn't seem to break anything, but AFAICT, it's not related to toolkit migration. Is ignorekeys just obsolete (it seems only used by toolkit/xpfe autocomplete widgets)?
Comment 47•17 years ago
|
||
Comment on attachment 272193 [details] [diff] [review] Actually implement nsIAutoCompletePopup >+ this.width = aWidth; This needs to be setAttribute or style.width. r=me with that fixed assuming you have an answer to comment 46.
Attachment #272193 -
Flags: review?(ajschult) → review+
Comment 48•17 years ago
|
||
(In reply to comment #45) > Created an attachment (id=272194) [details] > Actually use the popup > This patch depends on attachment 272193 [details] [diff] [review] of course. I'm not 100% sure (having not tried building it) but won't this patch also need to incorporate some changes to the packaging files as we're not building libxul yet? I'd also half expect to see wallet being disabled as you're enabling password manager and satchel - or is there still a bit more work to get those actually in use?
Comment 49•17 years ago
|
||
Comment on attachment 272193 [details] [diff] [review] Actually implement nsIAutoCompletePopup sr=jag
Attachment #272193 -
Flags: superreview?(jag) → superreview+
Assignee | ||
Comment 50•17 years ago
|
||
(In reply to comment #46) >Is ignorekeys just obsolete I believe it is now, but only for Enn's new <xul:panel> element. (In reply to comment #47) >(From update of attachment 272193 [details] [diff] [review]) >>+ this.width = aWidth; >This needs to be setAttribute or style.width. No it doesn't, nsXULElement::SetWidth calls SetAttr internally. See NS_IMPL_XUL_STRING_ATTR(Width, width) (In reply to comment #48) >I'm not 100% sure (having not tried building it) but won't this patch also need >to incorporate some changes to the packaging files as we're not building libxul >yet? >I'd also half expect to see wallet being disabled as you're enabling password >manager and satchel - or is there still a bit more work to get those actually >in use? Probably yes to both, but I don't know enough about either aspect. Attachment 267401 [details] [diff] also seems to contain some other changes Kairo felt necessary.
Comment 51•17 years ago
|
||
For removing wallet, we also need to replace password manager. I hope even the new login manager is a drop-in replacement for browser functionality, but mailnews needs some work, see bug 239131. Password manager is still not nice if you have multiple passwords saved for a page, but we're on par with Firefox there - and bug 376668 targets to improve that. I still need to test the real switch patch a bit more, perhaps there's a way to use the satchel part for the moment and wait for bug 239131 with the password part. The good thing is that the patches here (I think the part of building toolkit autocomplete might also be needed) make the urlbar autocomplete work with places history (latest patch from bug 382187 + setting MOZ_PLACES=1).
Comment 52•17 years ago
|
||
(In reply to comment #51) > Password manager is still not nice if you have multiple passwords saved for a > page, but we're on par with Firefox there - and bug 376668 targets to improve > that. We also have bug 382437 that means the mailnews dialogs don't display correctly. > I still need to test the real switch patch a bit more, perhaps there's a way to > use the satchel part for the moment and wait for bug 239131 with the password > part. I'm fairly sure we could rip out the form fill part of wallet from its extension and keep the passowrd manager for now, especially as we're the only ones using that part of wallet. I'll try and have a look in the next couple of days if you want.
Comment 53•17 years ago
|
||
This is what I think we need to do for disabling wallet's form fill. I've not tested it yet (my build is still going), but its hopefully a good first draft. We need to be slightly careful wrt camino - I've found that they appear to use wallet.properties (I'm not sure how it gets there, but it seems to), and I think from that they may use some other wallet code as well.
Comment 54•17 years ago
|
||
(In reply to comment #53) > We need to be slightly careful wrt camino Argh, yes, Camino-like-Firefox is using wallet - not sure what for, though (if only passwords or also form prefilling). If they depend on the form prefilling stuff, you should just make it #ifdef MOZ_CAMINO or whatever they are using. You cannot remove the MOZ_XUL_APP ifdefs, as they are still xpfe-based as well :(
Comment 55•17 years ago
|
||
Comment on attachment 272194 [details] [diff] [review] Actually use the popup >Index: suite/browser/navigator.xul >=================================================================== >+ <panel type="autocomplete" class="autocomplete-result-popup" id="PopupAutoComplete"/> The idea to set that class here is nice, but I think I'd like better if we add panel[type="autocomplete"] as an additional selector to that rule in xul.css, because that gives us toolkit compat, which some extensions might expect. r=me if you add that additional change: Index: suite/browser/browser-prefs.js =================================================================== pref("signon.rememberSignons", true); pref("signon.expireMasterPassword", false); +pref("browser.formfill.enable", true); -pref("wallet.captureForms", true); +pref("wallet.captureForms", false); pref("wallet.enabled", true); With this, wallet form prefilling should be disabled and satchel's activated. I think we can get this in right now and defer the password manager changes and the real removal to when it's ready (I hope we convinced dolske enough to do the LoginManager changes we need for mailnews).
Attachment #272194 -
Flags: review?(kairo) → review+
Assignee | ||
Comment 56•17 years ago
|
||
Enn because I'm removing some CSS popup workarounds that might not be relevant.
Attachment #274290 -
Flags: superreview?(jag)
Attachment #274290 -
Flags: review?(gavin.sharp)
Attachment #274290 -
Flags: review?(enndeakin)
Comment 57•17 years ago
|
||
Comment on attachment 272194 [details] [diff] [review] Actually use the popup r=me on the change to toolkit/components/Makefile.in .
Attachment #272194 -
Flags: review+
Comment 58•17 years ago
|
||
Comment on attachment 274290 [details] [diff] [review] Convert autocomplete-result-popup to type="panel" and disablehistory="false" to enablehistory="true" >+.autocomplete-history-popup { >+ -moz-binding: url("chrome://global/content/bindings/autocomplete.xml#autocomplete-history-popup"); >+} The class doesn't seem to be used in toolkit. I don't mind this being here, but I notice that there are references in the theme to it which we could remove, or remove in a follow up bug. > .autocomplete-history-dropmarker { >- -moz-binding: url("chrome://global/content/autocomplete.xml#history-dropmarker"); >+ display: none; >+} >+ >+.autocomplete-history-dropmarker { >+ display: -moz-box; >+ -moz-binding: url("chrome://global/content/bindings/autocomplete.xml#history-dropmarker"); Two uses of 'display' here, but neither should be needed as dropmarker is -moz-box already. I think that removing the popupset hackery should be ok too as popups don't lay out when they are closed any more.
Attachment #274290 -
Flags: review?(enndeakin) → review+
Assignee | ||
Comment 59•17 years ago
|
||
Comment on attachment 274290 [details] [diff] [review] Convert autocomplete-result-popup to type="panel" and disablehistory="false" to enablehistory="true" KaiRo pointed out that I built toolkit and xpfe in the wrong order so it looked like this worked when it doesn't.
Attachment #274290 -
Flags: superreview?(jag)
Attachment #274290 -
Flags: review?(gavin.sharp)
Attachment #274290 -
Flags: review-
Assignee | ||
Comment 60•17 years ago
|
||
Including panel[type="autocomplete"], preferences and packaging.
Attachment #272194 -
Attachment is obsolete: true
Comment 61•17 years ago
|
||
OK, with this in and satchel activated and working (yay!) I think we should split off all remaining work into followups and close this as FIXED. I'll file followups for password manager and for the RTL stuff I had in my WIP patch once I'm really awake again. Neil, I'll leave it to you to mark this bug resolved and file any followup autocomplete issues :)
Comment 62•17 years ago
|
||
Will the disappearing of the search option on the URL bar be addressed by some of these follow-ups above?
Assignee | ||
Comment 63•17 years ago
|
||
Attachment #274382 -
Flags: review?(kairo)
Updated•17 years ago
|
Attachment #274382 -
Flags: review?(kairo) → review+
Assignee | ||
Comment 64•17 years ago
|
||
After a closer examination of the code satchel only calls passwordmgr without null-checking if passwordmgr has already called satchel.
Attachment #274994 -
Flags: review?
Assignee | ||
Comment 65•17 years ago
|
||
Comment on attachment 274994 [details] [diff] [review] satchel doesn't actually need passwordmgr After a closer examination of the code satchel only calls passwordmgr without null-checking if passwordmgr has already called satchel.
Attachment #274994 -
Flags: review? → review?(ted.mielczarek)
Comment 66•17 years ago
|
||
Comment on attachment 274994 [details] [diff] [review] satchel doesn't actually need passwordmgr Is there a bug number you could put in that comment for the work to fix that XXX?
Attachment #274994 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 67•17 years ago
|
||
Comment on attachment 274994 [details] [diff] [review] satchel doesn't actually need passwordmgr Playing safe because I can't figure out how the rules apply to changes in toolkit Makefiles that only affect SeaMonkey. (In reply to comment #66) >Is there a bug number you could put in that comment for the work to fix that XXX? That would be bug 239131.
Attachment #274994 -
Flags: approval1.9?
Comment 68•17 years ago
|
||
Comment on attachment 274994 [details] [diff] [review] satchel doesn't actually need passwordmgr You don't need approval for toolkit patches at this point (let alone toolkit patches that only affect SeaMonkey), see http://groups.google.com/group/mozilla.dev.planning/browse_thread/thread/f569a23ce266a451/265344ae5291dee0 .
Attachment #274994 -
Flags: approval1.9?
Comment 71•16 years ago
|
||
Is there a reason why this bug is still open? IIRC, satchel has been working fine in SeaMonkey trunk for some time now...
Assignee | ||
Comment 72•16 years ago
|
||
I guess not...
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 73•15 years ago
|
||
Hi! Just got Seamonkey 2RC1 to try it out, and noticed form manager's gone. With the previous Form Manager, I was able to quickly fill out standard fields like Name, Email, Address etc. in long forms by just doing tools > form manager > fill form data (or smth to that effect). Is this feature now gone?
Assignee | ||
Comment 74•15 years ago
|
||
The feature to fill in whole forms at once has gone, yes. Without the aid of an add-on the best you can do is to double-click each field in turn and pick the (preferred) value from the list each time. While it would be possible to write an add-on that would do this for all input fields automatically I don't actually know whether such an add-on exists.
Comment 75•15 years ago
|
||
Thought so. And the old manager is totally gone, I suppose. The new one is nice, sometimes, but it's sad when a new version of software loses some of the nice features the previous version had...
Comment 76•15 years ago
|
||
I believe Roboform is an application/add-on most of the people are using now to store passwords and form data. See http://www.roboform.com/.
Comment 77•14 years ago
|
||
Lastpass is also a common alternative.
Comment 78•6 years ago
|
||
Pushed by frgrahl@gmx.net: https://hg.mozilla.org/comm-central/rev/47841d72f585 Continuation of synchronisation of xpfe autocomplete with toolkit interfaces b=304309 r=ajschult rs=mscott https://hg.mozilla.org/comm-central/rev/d6542f70f7d5 Continuation of synchronisation of xpfe autocomplete with toolkit interfaces b=304309 r=ajschult sr=jag https://hg.mozilla.org/comm-central/rev/6cd0c87d55b4 Switch xpfe autocomplete widget from -moz-tree-*(menuactive) to -moz-tree-*(selected) to match toolkit b=304309 r=Mano sr=jag https://hg.mozilla.org/comm-central/rev/f9f2ffc5968f Continuation of synchronisation of xpfe autocomplete with toolkit interfaces (change getOverrideValue() to attribute overrideValue) b=304309 r=ajschult sr=jag https://hg.mozilla.org/comm-central/rev/e5dcb3170fc6 Change column names to match what toolkit wants b=304309 r=ajschult sr=jag https://hg.mozilla.org/comm-central/rev/a2477a8e0a9e Rework xpfe autocomplete selectedIndex handling b=304309 r=ajschult sr=jag https://hg.mozilla.org/comm-central/rev/8d1b9edd87ee Make xpfe autocomplete implement nsIAutoCompletePopup b=304309 r=ajschult sr=jag https://hg.mozilla.org/comm-central/rev/e40d54276af6 Start using satchel in suite b=304309 r=KaiRo (toolkit change is ifndef Firefox)
You need to log in
before you can comment on or make changes to this bug.
Description
•