Closed Bug 515479 Opened 11 years ago Closed 11 years ago
remove mail accounts from old account wizard for TB
File | New | Other Accounts brings up the old account wizard, which includes items for mail account and two forms of gmail accounts (imap and pop3). I think we want to remove those items for TB to get rid of duplicate UI. If/when SM decides to use the new auto config stuff, then we could rip out a fair amount of the old wizard.
I feel somewhat responsible for making sure this happens, so I'll take it :-)
Assignee: nobody → bienvenu
Target Milestone: --- → Thunderbird 3.0rc1
David, I guess as long as we don't remove all the code for the old account wizard, this bug does not have any l10n impact, right?
no l10n impact, really, other than we'll be removing the use of a couple strings from TB but not SM. We can leave the strings in the TB .dtd or .properties files, or remove them, whatever's easier for l10n.
Removing strings also counts as l10n impact :) I would suggest that we either try to get to this before the final string freeze at the end of September or just remove the UI from TB3 and remove the strings in a followup bug once TB3 is on its own comm-1.9.1 branch.
I need to try this on the mac, and have someone try it on linux to see that it does the right thing. It seems to do the right thing on Windows. In particular, I'm worried about movemail on linux (if that shows up at all today; I'm not sure. I don't see it on the mac). I guess I need to worry a bit more about mail/installer/removed-files.in as well, to add the gmail and docmac rdf files.
In theory, movemail still shows up on Linux, and you should be able to get it to show up on OS X by just copying movemail.rdf from src/mailnews/base/ispdata/movemail.rdf to Thunderbird.app/Contents/MacOS/isp/ (which is probably less trouble than seeing whether the extension I attached in bug 379705 still works). Oh, and you should probably look at that attachment before trying to get anyone to use it, since it looks more like an hg stat than an hg (q)diff :)
(In reply to comment #6) > > Oh, and you should probably look at that attachment before trying to get anyone > to use it, since it looks more like an hg stat than an hg (q)diff :) Yes, the actual diff is left as an exercise for the reader :-)
(In reply to comment #7) > (In reply to comment #6) > > > > Oh, and you should probably look at that attachment before trying to get anyone > > to use it, since it looks more like an hg stat than an hg (q)diff :) > > Yes, the actual diff is left as an exercise for the reader :-) ah, it's more of an abstract diff then. I see some amazing code but bad commenting :-)
Heh. So I really don't understand mail/installer/removed-files.in - it seems to already have isp/gmail.rdf, which is what I would have thought I would have added to remove gmail.rdf...
Attachment #400846 - Attachment is obsolete: true
No, isp/gmail.rdf is the old one from before we had isp/en-US/gmail.rdf and isp/de-DE/googlemail.rdf. You want to remove isp/ab_CD/(gmail|googlemail).rdf, and... good luck with that!
Keep in mind that this wizard can be called with the "New Account" button in the "Mail & Newsgroups Account Settings" window. http://www.mozilla.org/projects/security/pki/psm/smime_land_files/accountSettingSecuri.gif I'm not sure how many peaople use "New > Mail Account" to create an account, but the other way is more likely what former Outlook (Express) users are going to use. Maybe the "mail accounts" option from the old wizard should stay but call the new wizard ?
Only to let you know: I *never* use File --> New --> Mail Account to create an mail account. I always use "Add account" button in the account settings dialog. By this missing knowledge I create a bugreport, thinking that "autoconfig" is missing. Bug #518558
That button will have to go i think.
You wanna remove it? I think that it is a bad idea. If you remove it from me, I need go to Google ask "how to create a mail account in Thunderbird", because I always use this button (in account manager). I never see, in the past, File --> New --> Mail account (I see it only TODAY).
Well we can hardly have two buttons there (for the different types).
I think the idea is to have a drop down button like we have on the toolbar - new mail account by default, but with an option to add an other kind of account.
I'm not sure it's worth enumerating all the possible localized gmail rdf files we should remove from 2.0 installations, since there's not much harm in leaving them there. Users with those localized builds would still see gmail as an option in the old account wizard, which isn't ideal, but not the end of the world either.
Depends on: 518558
I ended up trying to add all the files we should remove from 2.0 installs. Eventually, someone has to clean up the locales to get rid of isps.txt, but I don't think that's required for this bug. requesting sr from Neil to make sure this doesn't affect SM.
Whiteboard: [no l10n impact] → [no l10n impact][has patch for review philor, sr neil]
Attachment #403582 - Flags: superreview?(neil) → superreview+
Comment on attachment 403582 [details] [diff] [review] proposed fix I can believe that this won't affect SeaMonkey.
Note, in the File/New menu there is a separator between Mail Account and Other Accounts. It would be more logical to move the separator below Other Accounts. Also, when ripping the New Account Setup dialog, after removing Email account, and Gmail IMAP and Gmail POP (which are also no longer needed), only the Blogs&News Feeds and the Newsgroup Account remain. Better to move this two options to the File New menu, so that it becomes: File->New->Mail Account File->New->Blogs&News Feeds (Which doesn't really require a wizard) File->New->Newsgroup Account
pinging for review
Whiteboard: [no l10n impact][has patch for review philor, sr neil] → [no l10n impact][has sr, has patch for review philor]
Attachment #403582 - Flags: review?(philringnalda) → review+
fix checked in.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [no l10n impact][has sr, has patch for review philor] → [no l10n impact]
You need to log in before you can comment on or make changes to this bug.