Closed Bug 561064 Opened 10 years ago Closed 10 years ago
Larry menu doesn't work correctly when finger is dragged from the Larry menu to awesome bar url field
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:126.96.36.199) Gecko/2009060309 Ubuntu/8.04 (hardy) Firefox/3.0.11 Build Identifier: Mozilla/5.0 (X11; U;Linux armv71; en-US; rv:188.8.131.52pre) Gecko/20100420 Namroka/3.6.5pre Fennec/1.1a2pre When finger is dragged from Larry menu to awesome bar url field, the focus moves to url field and menu disappears. Button on the awesome bar for Larry menu stays toggled like the menu would be visible still. If you click that button it will come untoggled and only after second click you get the Larry menu to be visible again. Reproducible: Always Steps to Reproduce: 1. Open Larry menu 2. Drag your finger from the menu to awesome bar url field 3. Release your finger. Actual Results: Button for Larry bar stays toggled even though menu is not visible anymore. Expected Results: Button for Larry bar should follow state changes correctly. Most likely this bug caused because it is not expected that focus would change to url field when dragged, so that would be the real bug behind this.
Thanks for the full of details bug report, it helps me figured what's wrong quickly! > Most likely this bug caused because it is not expected that focus would change > to url field when dragged, so that would be the real bug behind this. To be exact, the "site menu" and many dialogs in Fennec are supposed to dismiss when you do something else. With that in mind we were checking if, during a "mousedown" the target is different than the "dialog" (and a few others stuff) and if it is the case just dismiss the dialog. But in your testcase you're doing the mousedown on an authorized target and the mouseup on another non-authorized one. patch is coming. Thanks again.
Status: UNCONFIRMED → NEW
Ever confirmed: true
This patch add a check for the event target during mouseup.
Assignee: nobody → 21
Attachment #440766 - Flags: review?(mark.finkle)
Attachment #440766 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Cool! verified FIXED on build: Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:184.108.40.206pre) Gecko/20100423 Namoroka/3.6.5pre Fennec/1.1a2pre
Status: RESOLVED → VERIFIED
Component: Linux/Maemo → General
OS: Linux → Linux (embedded)
QA Contact: maemo-linux → general
Assignee: mozaakash → 21
Flags: in-litmus? → in-litmus?(mozaakash)
litmus testcase https://litmus.mozilla.org/show_test.cgi?id=12932 created to regression test this bug.
Flags: in-litmus?(mozaakash) → in-litmus+
You need to log in before you can comment on or make changes to this bug.