Closed
Bug 850673
Opened 12 years ago
Closed 12 years ago
Defect - Double clicking Next in Find bar selects and displays grippers for next selectable instance
Categories
(Firefox for Metro Graveyard :: Browser, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 22
People
(Reporter: virgil.dicu, Assigned: jimm)
References
Details
(Whiteboard: feature=defect u=metro_firefox_user c=content_features p=1)
Attachments
(1 file)
(deleted),
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows NT 6.2; rv:22.0) Gecko/20130312 Firefox/22.0 (http://hg.mozilla.org/mozilla-central/rev/7433bc4545c9)
Associated with bug 831940 - Find previous and next instance of text in a page
1. Launch Firefox Metro.
2. Load https://www.mozilla.org/en-US/firefox/features/ (or any other page)
3. Invoke the find bar (right click settings and Find or Ctrl+F)
4. Type any word (e.g. "easy")
5. Double click the Next arrow.
Actual result: the grippers are displayed.
I'm not sure if this is intended or not for tablets, but there is no use for these in Metro Desktop. Haven't found any other references for comparison.
Note: Simply selecting any word, without the Find bar will display the grippers as well.
Updated•12 years ago
|
Blocks: metrov1triage
Comment 2•12 years ago
|
||
I think this depends on how much work is involved. If it's a relatively easy fix, we should get to this soon. If it's complicated, we can tackle it in v2. What say you engineers, is this a difficult problem?
Flags: needinfo?(asa)
Updated•12 years ago
|
Flags: needinfo?(jmathies)
Assignee | ||
Comment 3•12 years ago
|
||
Should be pretty easy to fix. When enabled the find bar should disable the SelectionHelperUI.
Flags: needinfo?(jmathies)
Updated•12 years ago
|
Priority: -- → P1
Summary: Double clicking Next in Find bar selects and displays grippers for next selectable instance → Defect - Double clicking Next in Find bar selects and displays grippers for next selectable instance
Whiteboard: feature=defect u=metro_firefox_user c=content_features p=tbd
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → jmathies
Assignee | ||
Updated•12 years ago
|
Whiteboard: feature=defect u=metro_firefox_user c=content_features p=tbd → feature=defect u=metro_firefox_user c=content_features p=2
Assignee | ||
Comment 4•12 years ago
|
||
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 730629 [details] [diff] [review]
fix
really simple patch.
Attachment #730629 -
Flags: review?(mbrubeck)
Updated•12 years ago
|
Attachment #730629 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Updated•12 years ago
|
Whiteboard: feature=defect u=metro_firefox_user c=content_features p=2 → feature=defect u=metro_firefox_user c=content_features p=1
Assignee | ||
Comment 6•12 years ago
|
||
Updated•12 years ago
|
Status: NEW → ASSIGNED
QA Contact: jbecerra
Comment 7•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
Assignee | ||
Updated•12 years ago
|
OS: Windows 8 → Windows 8 Metro
Version: 22 Branch → Trunk
Updated•12 years ago
|
Flags: needinfo?(jbecerra)
Reporter | ||
Comment 8•12 years ago
|
||
Mozilla/5.0 (Windows NT 6.2; rv:23.0) Gecko/20130404 Firefox/23.0
Verified using steps in comment 0. One can double click fast enough and Nightly will now return corresponding results without grippers.
One adjacent question, though: selecting text in a normal page will display grippers while using Metro desktop. Is this intended?
I haven't seen this called out in any story.
Status: RESOLVED → VERIFIED
Comment 9•12 years ago
|
||
I think that's a bug but will let Jim confirm
Assignee | ||
Comment 10•12 years ago
|
||
(In reply to Virgil Dicu [:virgil] [QA] from comment #8)
> One adjacent question, though: selecting text in a normal page will display
> grippers while using Metro desktop. Is this intended?
> I haven't seen this called out in any story.
It's a bug if you can reproduce it using the mouse in metro land. running the metro
interface on desktop isn't an official product which we maintain.
Comment 11•12 years ago
|
||
I've seen this in metro as well in Immersive mode but I don't have exact steps.
Updated•12 years ago
|
Flags: needinfo?(jbecerra)
Comment 12•11 years ago
|
||
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130825030201
Built from http://hg.mozilla.org/mozilla-central/rev/01576441bdc6
WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in comment0 and got expected result.
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•