Add FXIOS-10460 [Homepage] Context Menu Navigation

cyndichin commented 2 days ago

:scroll: Tickets

:bulb: Description

Add navigation foundation to show the context menu / photon action sheet.

Handle long press action and dispatch a navigation action for the context menu
Added new method to coordinator to showContextMenu this will create the ContextMenuState which will be use to populate the viewModel used for the PhotonActionSheet view
Created new ContextMenuState which will hold action logic.

Next PRs (after PTO) will include specific actions for pocket and top sites. Tests will be added for the context menu state, UI tests will be added at a later date to test the navigation.

:pencil: Checklist

You have to check all boxes before merging

[x] Filled in the above information (tickets numbers and description of your work)
[x] Updated the PR name to follow our PR naming guidelines
[ ] Wrote unit tests and/or ensured the tests suite is passing
[ ] When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
[ ] If needed, I updated documentation / comments for complex code and public methods
[ ] If needed, added a backport comment (example @Mergifyio backport release/v120)

Screenshots

https://github.com/user-attachments/assets/7696d708-5e8d-422c-9fd4-fd33d0ef76f1

cyndichin commented 2 days ago

@tusharC95 , @DanielDervishi , @Cramsden for visibility

mobiletest-ci-bot commented 1 day ago

	Messages
:book:	Project coverage: 32.32%
:book:	Edited 9 files
:book:	Created 1 files

Client.app: Coverage: 31.12

File	Coverage
BrowserViewController.swift	4.74%	⚠️
BrowserCoordinator.swift	69.37%	✅
BrowserNavigationHandler.swift	76.92%	✅
ContextMenuState.swift	0.0%	⚠️
BrowserViewControllerState.swift	47.7%	⚠️
NavigationBrowserAction.swift	100.0%	✅
HomepageViewController.swift	35.73%	⚠️
BrowserNavigationType.swift	100.0%	✅

Generated by :no_entry_sign: Danger Swift against 8a92232c760cfcd7c024997729ef1e5817dc7ccc

cyndichin commented 1 day ago

If no changes needed, feel free to approve + merge this PR. Otherwise, will address any changes when I return.

mozilla-mobile / firefox-ios