Closed
Bug 1103321
Opened 10 years ago
Closed 10 years ago
Left mouse click / middle mouse click "go" button does not work in searchBar after mouse over doropdown
Categories
(Firefox :: Search, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: alice0775, Unassigned)
References
Details
(Keywords: regression)
[Tracking Requested - why for this release]:
Steps To Reproduce:
1. type any word
2. move mouse on dropdown once
3. Left mouse click / middle mouse click "go" button
Actual Result:
The dropdown disappears.
However, Search is not executed.
Expected Results:
Execute search with word.
Reporter | ||
Updated•10 years ago
|
Severity: normal → blocker
Reporter | ||
Updated•10 years ago
|
Keywords: useless-UI
Reporter | ||
Updated•10 years ago
|
Keywords: regression
Comment 1•10 years ago
|
||
On Mac I can't reproduce if I tap the go icon with the trackpad, but if I actually click I can reproduce. On linux I can reproduce both with tapping and clicking.
My guess is that when the panel gets hidden the mousedown event gets consumed, which prevents the click event from reaching the button.
Comment 2•10 years ago
|
||
We're not going to be able to fix this in 34. I have marked 35+ for tracking.
I have also dropped severity and the useless-ui keyword.
Severity: blocker → normal
status-firefox33:
--- → unaffected
status-firefox35:
--- → affected
status-firefox36:
--- → affected
tracking-firefox35:
--- → +
tracking-firefox36:
--- → +
Keywords: useless-UI
Comment 3•10 years ago
|
||
We're not going to get much traction here without a regression window - wontfix for 35 as we're too late in the cycle. Leaving tracking for 36 in case the regression range is found and a fix provided.
status-firefox37:
--- → affected
Keywords: regressionwindow-wanted
Comment 4•10 years ago
|
||
I think this regressed with bug 1088660 and was covered by a previous version of the fix in bug 1103455 covered this, but we simplified the patch to reduce risk for 35. This may get fixed as a side effect of bug 1114707.
Blocks: fx34-searchui
Comment 5•10 years ago
|
||
I can reproduce this bug on 34, but on my trunk build I see a different behavior: the click on the go button loads the search in the current tab, but then the search panel reopens. Dave, do you know what's going on here?
Flags: needinfo?(dtownsend)
Comment 6•10 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #5)
> I can reproduce this bug on 34, but on my trunk build I see a different
> behavior: the click on the go button loads the search in the current tab,
> but then the search panel reopens. Dave, do you know what's going on here?
I'm guessing that this was fixed by bug 1102050
Flags: needinfo?(dtownsend)
Comment 7•10 years ago
|
||
So the panel reopening would be a new separate bug? (possibly caused by bug 1106101?)
Comment 8•10 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #7)
> So the panel reopening would be a new separate bug? (possibly caused by bug
> 1106101?)
Ah yes, pretty similar to bug 1118135. Though probably we just shouldn't be opening the popup at all when clicking on the go icon so probably worth a new bug.
Comment 9•10 years ago
|
||
(In reply to Dave Townsend [:mossop] from comment #8)
> (In reply to Florian Quèze [:florian] [:flo] from comment #7)
> > So the panel reopening would be a new separate bug? (possibly caused by bug
> > 1106101?)
>
> Ah yes, pretty similar to bug 1118135. Though probably we just shouldn't be
> opening the popup at all when clicking on the go icon so probably worth a
> new bug.
I filed bug 1119450 for the go button opening the popup. This bug was fixed by bug 1102050
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Updated•10 years ago
|
Flags: qe-verify+
Comment 10•10 years ago
|
||
Verified as fixed using:
FF 36.0b2
Build Id:20150120155007
OS: Win 7 x64, Mac Os X 10.9.5, Ubuntu 12.04 x32
Comment 11•10 years ago
|
||
Verified as fixed using Firefox 37 beta 1 under Ubuntu 12.04 LTS 32-bit, Windows 7 64-bit and Mac OS X 10.9.5.
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
Keywords: regressionwindow-wanted
You need to log in
before you can comment on or make changes to this bug.
Description
•