Closed
Bug 1346411
Opened 8 years ago
Closed 8 years ago
Update context menu for new Activity Stream Panel
Categories
(Firefox for iOS :: Home screen, enhancement, P1)
Tracking
()
RESOLVED
FIXED
Iteration:
1.19
| Tracking | Status | |
|---|---|---|
| fxios | 8.0+ | --- |
People
(Reporter: farhan, Assigned: farhan)
References
Details
(Whiteboard: [mobileAS])
Attachments
(1 file)
Migrating AS into a UICollectionView has broken the Context menu. Make sure the context menu works correctly.
A few other things have also changed that should be fixed with this PR.
There are now UIImageViewExtensions to set Favicons that provide correct background colors. So no need to pass the UIImage and UIColor from the HighlightCell/TopSiteCell to the context menu.
|
||
Updated•8 years ago
|
Assignee: nobody → jdarcangelo
Status: NEW → ASSIGNED
|
|
||
Comment 1•8 years ago
|
||
Farhan, I took a look at this and wasn't able to determine what the issue was with the context menu. Could you elaborate?
Assignee: jdarcangelo → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(fpatel)
|
|
||
Updated•8 years ago
|
|
Assignee | |
Comment 3•8 years ago
|
||
This is a preemptive bug for the changes that will occur when we migrate to UICollectionView. So you wont notice this on master. Yet!
Flags: needinfo?(fpatel)
|
|
Assignee | |
Updated•8 years ago
|
|
|
Assignee | |
Comment 4•8 years ago
|
||
Attachment #8853743 -
Flags: review?(bmunar)
|
||
Updated•8 years ago
|
Attachment #8853743 -
Flags: review?(bmunar) → review+
|
|
Assignee | |
Updated•8 years ago
|
Assignee: nobody → fpatel
Iteration: --- → 1.19
Priority: P2 → P1
|
|
Assignee | |
Comment 5•8 years ago
|
||
master https://github.com/mozilla-mobile/firefox-ios/commit/dda2e60cec612dd2e2e7ec1458ab4fba81fd3516
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•