2067224 - [customizable-menu] Add a MenuGridContainer and MenuGridItem composables to compose-menu

Status: RESOLVED FIXED

(Firefox for Android :: Menu, task)

Description

[Petru-Mugurel Lingurar [:petru]]

Based on the recent investigation most menu items share a similar structure and UI.
These will be used instead of MenuNavigation and LibraryMenuGroup from Fenix to allow displaying a grid of menu items.

The only difference between the two grid UI approaches is the background of the container / cells.
The AC composable should support 3 variations:

• surface container background - for the navigation items
• surfaceContainerHigh container background - when the menu overflows the screen and the grid section becomes sticky
• surfaceBright items background + dividers - for the library items

The surfaceContainerHigh container colour should be set automatically.
The choice between the surface container background and surfaceBright items background + dividers should be of the integrators -> the menu section should allow passing this as a parameter, maybe as a useDividers boolean property.

Note: The menu item should support an optional long-press interaction - needed by navigation.

Comment 1

[Iorga Gabriel]

Attachment: Bug 2067224 - [customizable-menu] Add a MenuGridContainer and MenuGridItem composables to compose-menu

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/90060b79d462
https://hg.mozilla.org/integration/autoland/rev/151e0ac9a1ae
[customizable-menu] Add a MenuGridContainer and MenuGridItem composables to compose-menu r=android-reviewers,petru

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/151e0ac9a1ae