1963835 - [Feature Callout] Styling configurations to better support multiple MultiSelect tiles

Status: VERIFIED FIXED

(Firefox :: Messaging System, task, P1)

Description

[Mimi Jung]

Attachment: Mocked design - ideal state

We've added the ability to support multiple MultiSelect tiles within the same screen in 1952335. We'd like to leverage this in Feature Callout but we'll need to update the styling/ add some additional configuration. See attached mock up design and the current off-train solution.

Comment 1

[Mimi Jung]

Attachment: Current state within feature callout

Comment 2

[Meg Viar [:mviar]]

As part of this work, or in a follow-up bug, we may want to unify the methods we have for handling "tile" styles across arrays of tiles and single tiles (while maintaining backwards compatibility).

We use tile.style here in ContentTiles.jsx and tiles.style here in Multiselect.jsx (only works with single tile definitions of content.tiles).

Comment 3

[Mimi Jung]

Attachment: Bug 1963835 - Adds mini layout and style configuration to multiple MultiSelect tiles

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/0134df854eb0
https://hg.mozilla.org/integration/autoland/rev/090bf21a5e08
Adds mini layout and style configuration to multiple MultiSelect tiles r=omc-reviewers,mviar

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/73f4f03c718e
https://hg.mozilla.org/integration/autoland/rev/fe9ff4b2ae85
Revert "Bug 1963835 - Adds mini layout and style configuration to multiple MultiSelect tiles r=omc-reviewers,mviar" for causing xpcshell failures on test_PanelTestProvider.js

Comment 6

[Cristina Horotan [:chorotan]]

Backed out for causing xpcshell failures on test_PanelTestProvider.js and node failure

Backout link

Push with failures

Failure log xpcshell
Failure log node

Comment 7

[Mimi Jung]

Sorry about that, looks like I forgot to update the test_PanelTestProvider.js message count after adding the new one.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/0228ad2125de
https://hg.mozilla.org/integration/autoland/rev/939fe475e79f
Adds mini layout and style configuration to multiple MultiSelect tiles r=omc-reviewers,mviar

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/6ba59b3f7038
https://hg.mozilla.org/integration/autoland/rev/2aad04a53208
Revert "Bug 1963835 - Adds mini layout and style configuration to multiple MultiSelect tiles r=omc-reviewers,mviar" for causing node failures

Comment 10

[amarc]

Backed out for causing node failures

• Backout link
• Push with failures
• Failure Log

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/1381e1c79253
https://hg.mozilla.org/integration/autoland/rev/2036cf433cf9
Adds mini layout and style configuration to multiple MultiSelect tiles r=omc-reviewers,mviar

Comment 12

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/2036cf433cf9

Comment 13

[Simona Rosu [:srosu], Ecosystem QA]

I have verified this issue using the latest Firefox Nightly 142.0a1 (Build ID: 20250720210720) build on Windows 10 x64, macOS 15.5, and Ubuntu 24.04.1.

• After following the steps outlined in the test plan, I confirm that the content on the “TEST_MULTI_TILES_SURVEY” message is left-aligned and visually more refined compared to the “Current state within feature callout” image attached.
• The message contains the following elements:
  • A fox icon and a bold “Test title” positioned at the top of the message.
  • A dismiss “x” button displayed in the upper-right corner.
  • A bold description text “Answer two questions!” placed below the title.
  • “Question 1” in bold, followed by 4 checkboxes.
  • “Question 2” in bold, followed by 2 checkboxes.
  • A blue “Submit” button that is active even when no checkboxes are selected.