Refactor MTE-3854 WIP: Menu Refactor Cherry Pick (Smoke Test only)

clarmso commented 2 days ago

:scroll: Tickets

You have to check all boxes before merging

[ ] Filled in the above information (tickets numbers and description of your work)
[ ] 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)

mobiletest-ci-bot commented 2 days ago

	Messages
:book:	Edited 10 files
:book:	Created 0 files

Generated by :no_entry_sign: Danger Swift against b582b8f90b841e28c8e028bfc4827fea31360c33

isabelrios commented 19 hours ago

@clarmso how is this work going? I just leave a question, apart from that, when do you expect this could land?

clarmso commented 14 hours ago

@clarmso how is this work going? I just leave a question, apart from that, when do you expect this could land?

This PR can land when the menu refactor is enabled. The bookmark() related question is for a full functional tests.

We will address the issue on full functional tests after this PR is landed so that the developers are not blocked.