333623 - about:blank's favicon draggable from location bar but not from tabs

Status: RESOLVED FIXED

(Camino Graveyard :: Drag & Drop, defect)

Description

[froodian (Ian Leue)]

The favicon for about:blank is draggable (like all other pages) from the location bar, but isn't from tabs.  Either behavior would be OK, but they should definitely match.

I'm in favor of disabling dragging in the location bar (rather than enabling it in the tabs), since I can't think of a reason why anybody would want to drag and drop a blank page.

Comment 1

[Samuel Sidler (old account; do not CC)]

I'm actually in favor of the other way, enabling drag&drop on the tabs. An icon is an icon is an icon and we shouldn't treat it better just because it's not a "standard" page.

That said, how are all our other about: pages treated?

Comment 2

[froodian (Ian Leue)]

I see what you mean about keeping icons' behavior consistent regardless of the page.

All other about: pages act like regular webpages.

Comment 3

[Stuart Morgan]

(In reply to comment #2)
> All other about: pages act like regular webpages.

Because all other about: pages have content, like regular web pages.

Comment 4

[Samuel Sidler (old account; do not CC)]

(In reply to comment #3)
> Because all other about: pages have content, like regular web pages.
> 

Yes, they do. I just don't see why we should "discriminate" simply because about:blank doesn't have content. It's unexpected for the (albeit crazy) users who try to do this.

Comment 5

[Stuart Morgan]

The icon is dragable as a proxy for the location, and about blank *has* no location. It's already treated differently in that the url is displayed blank, because from the user's perspective there is nothing there. Having a proxy for nothing doesn't make sense (see for example TextEdit, which don't even display a proxy icon for an unsaved new document, for the same reason)

Comment 6

[Chris Lawson (gone)]

Attachment: stops drag of icon if about:blank is the current URL

Comment 7

[Håkan Waara]

Comment on attachment 224331 [details] [diff] [review]
stops drag of icon if about:blank is the current URL

>-  if (!urlString)
>+  // don't allow dragging of proxy icon for empty pages
>+  if ((!urlString) || [MainController isBlankURL:urlString])
>     return;

Drop the extra parenthesis for !urlString please.

Comment 8

[Chris Lawson (gone)]

Comment on attachment 224331 [details] [diff] [review]
stops drag of icon if about:blank is the current URL

sr=pink?

cl

Comment 9

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 224331 [details] [diff] [review]
stops drag of icon if about:blank is the current URL

sr=pink

Comment 10

[Nick Kreeger]

Fixed trunk and 1.8 branch.