510630 - toolbar scrollable when awesome bar is displayed

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Geoff Gustafson]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.13) Gecko/2009080317 Fedora/3.0.13-1.fc10 Firefox/3.0.13
Build Identifier: 

If you go to the top of a page, and click on the urlbar to bring up the awesome bar list, you can then scroll the toolbar by clicking and dragging on its background. This reveals the top of the web page.


Reproducible: Always



Expected Results:  
Toolbar will be fixed in place while the awesome bar is visible.

Comment 1

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch v0.1

This patch lock the urlbar into position: fixed mode when a dialog is displayed.

Comment 2

[Stuart Parmenter]

I don't particularly understand this patch.  I don't think we should be flagging our toolbars as dialogs -- can you explain what this patch is supposed to do?

Comment 3

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #2)
> I don't particularly understand this patch.  I don't think we should be
> flagging our toolbars as dialogs -- can you explain what this patch is supposed
> to do?

I have not added a flag, I reused a previously existing one for displaying the little top right corner icon to close a dialog, for example when a dialog is opened, e.g the awesome bar, or the bookmark popup (opened via the yellow star), the toolbar is flagged as 'dialog'.
(So, I move the flag one level in the DOM hierarchy to fit my needs.)

What I mean by fixing the position is: if we are in 'dialog' mode, we want the urlbar to be: 
 - fully visible (correct the half showing bug - (pan up && Ctrl+L) )
 - unmovable (correct this bug for all dialogs not only the awesome bar dialog)

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

hmm, I do still notice an issue with this patch:

* If you display the awesomebar, the toolbar is _not_ panned when dragging on the toolbar background. However, the underlying surface is panned.

If you attempt to drag the toolbar, then press "esc" to close the awesomebar, you'll notice that the surface was panned.

Not sure if we want to try and fix it now, or as a followup bug.

It would be nice to _not_ pan the toolbar when displaying the awesomebar in beta3

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #4)
> hmm, I do still notice an issue with this patch:
> 
> * If you display the awesomebar, the toolbar is _not_ panned when dragging on
> the toolbar background. However, the underlying surface is panned.
> 
> If you attempt to drag the toolbar, then press "esc" to close the awesomebar,
> you'll notice that the surface was panned.

my bad, I've not think of that...

> 
> Not sure if we want to try and fix it now, or as a followup bug.
> 
> It would be nice to _not_ pan the toolbar when displaying the awesomebar in
> beta3

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

fixed by bug 510998

Comment 7

[Aakash Desai [:aakashd]]

verified FIXED On builds:

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.2b1pre) Gecko/20091002
Fennec/1.0b4

and

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/200910002
Fennec/1.0b4

Comment 8

[Dão Gottwald [:dao]]

bugspam