Closed
Bug 1447629
Opened 8 years ago
Closed 8 years ago
The long press context menu viewport is incorrect
Categories
(Firefox for iOS :: General, defect, P2)
Tracking
()
RESOLVED
FIXED
People
(Reporter: csuciu, Assigned: garvan)
References
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
RC 11.0 (9713)
1. Go to the History panel => Recently Closed.
2. Long press on a link to open the context menu.
Result: The context menu viewport is too small.
This is reproducible in all home panels but it's more visible in "Recently Closed".
Updated•8 years ago
|
Attachment #8961038 -
Flags: review?(jdarcangelo) → review+
merged to master (I verified this on 11.2 ipad and iphones, and 10.3 iphone on simulator)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Comment 3•8 years ago
|
||
Comment 5•8 years ago
|
||
This caused a regression in Bug 1448003.
Reverted on master:
https://github.com/mozilla-mobile/firefox-ios/commit/761f7a98e66cab695f34d3ecd8cefeefa2353c20
Reverted on v11.x:
https://github.com/mozilla-mobile/firefox-ios/commit/dc727cc2bd49d164a2c4a12347c0f577c28bb990
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Updated•8 years ago
|
Priority: -- → P2
| Reporter | ||
Comment 6•8 years ago
|
||
Verified that the old behavior is back in v11.0 (9819)
Comment 7•8 years ago
|
||
Attachment #8962481 -
Flags: review?(gkeeley)
Attachment #8961038 -
Attachment is obsolete: true
Attachment #8961038 -
Flags: review+
Comment on attachment 8962481 [details] [review]
Pull Request
I requested some cleanup items, hopefully small, just R? when ready, the rest is straightforward and looks ok.
Attachment #8962481 -
Flags: review?(gkeeley) → review-
Updated•8 years ago
|
Attachment #8962481 -
Flags: review- → review?(gkeeley)
Attachment #8962481 -
Flags: review?(gkeeley) → review+
Comment 9•8 years ago
|
||
fixed
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Keywords: regression
Summary: [Regression] The long press context menu viewport is incorrect → The long press context menu viewport is incorrect
You need to log in
before you can comment on or make changes to this bug.
Description
•