388509 - URL without a scheme in the Mail Start Page prefs UI doesn't work

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Build ID: Today's trunk Win32 builds (this is really cross-platform, though).

Summary: URL without a scheme in the Mail Start Page won't load

Steps to Reproduce:

1. In the "Thunderbird Start Page" section, enter a path/hostname without a scheme, such as www.coleypharma.com or coleypharma.com
2. On Windows, click OK (on Mac, just dismiss the dialog)
3. Click "Go | Start Page"

Expected Results:

The URL without a scheme, entered in step #1 loads

Actual Results:

Look like we do a reflow (wiping out Thunderbird's header pane!), but we don't load that page.  Nothing is thrown in the Error Console, either.

David and I chatted on IRC, and we both seemed to think either validation--including but not limited to disabling the OK button (at least on Windows) until we have a scheme we like--is the way to go.

Comment 1

[David :Bienvenu]

I've changed the summary a little to reflect the fact that we really just want to prevent this in the prefs UI.

Comment 2

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Firefox allows one to use e.g. www.mozilla.org as homepage. 
Though currently, no start page is working for me... bug 388441.

Comment 3

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Attachment: proposed fix

Use urifixup to allow start pages without scheme etc. 

For offline we have the disk cache nowadays...

Comment 4

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Though for testing, beware of bug 480606.

Comment 5

[Phil Ringnalda (:philor)]

Comment on attachment 364568 [details] [diff] [review]
proposed fix

We don't have the disk cache on, say, first run after a version change. If you want to do that, let's do it in a separate bug.

Comment 6

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Well, for those special cases where we are offline and don't have the page in cache the "Offline Mode" error page is shown. It has a bit browser specific text, but other than that, isn't that an improvement over a totally blank area?

Comment 7

[Phil Ringnalda (:philor)]

Yep, you might be able to make that case (probably to Bryan, who'll be on your side anyway since he thinks it should never under any circumstances be blank) in the bug on doing it. Or you might wind up getting stuck with doing mail/locales/en_US/chrome/overrides/netError.dtd ;)

Comment 8

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Comment on attachment 364568 [details] [diff] [review]
proposed fix

So, any reason not to do this now when we have our own error page?

Comment 9

[Bryan Clark (DevTools PM) [:clarkbw]]

Comment on attachment 364568 [details] [diff] [review]
proposed fix

looks good

Comment 10

[Phil Ringnalda (:philor)]

Comment on attachment 364568 [details] [diff] [review]
proposed fix

Sure.

Comment 11

[Magnus Melin [:mkmelin] (away, back Aug 4)]

changeset:   2463:4f81a119cef8
http://hg.mozilla.org/comm-central/rev/4f81a119cef8

->FIXED