1. you should use prompt service (patch) 2. you shouldn't use any alert, instead the next button shouldn't be enabled until the user really can continue.
Created attachment 95263 [details] [diff] [review] Updated as per timeless' comments
Attachment #84919 - Attachment is obsolete: true
Assignee: racham → neil
Summary: account wizard nags using window.alert → make account wizard smart enough not to need window.alert
Neil: any update here or are the patched obsolete?
Comment on attachment 126994 [details] [diff] [review] Bitrot update I'll look at this -
oh my, that's quite old. It's almost certainly bit-rotted again.
OK, it has bit-rotted. Do you have an unbit-rotted version? If not, I'll try to resurrect it.
ugh, sorry, the bit rot is a bit out of control, and the diffs are hard to read because of code moving around, so I'm going to mark this obsolete and hope Neil gives me yet an other chance :-(
Created attachment 322237 [details] [diff] [review] Bitrot update Unfortunately the patch now looks a little ugly in places, see what you think?
Attachment #322237 - Flags: superreview? → superreview?(bienvenu)
the patch looks OK - I just want to apply it and do some basic tests, since the account wizard doesn't get a lot of day to day testing in nightly builds :-)
Comment on attachment 322237 [details] [diff] [review] Bitrot update seems to work fine - I tried all four kinds of accounts (pop3, imap, gmail, and rss), and Next seem to be enabled appropriately.
Attachment #322237 - Flags: superreview?(bienvenu) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: --- → seamonkey2.0alpha
You need to log in before you can comment on or make changes to this bug.