Skip to content

Max firefly redesign pulling PRs #604 #605 to be merged to max-ff-redesign branch#609

Merged
arugupta1992 merged 3 commits intomax-ff-redesignfrom
max-feedback
Oct 24, 2025
Merged

Max firefly redesign pulling PRs #604 #605 to be merged to max-ff-redesign branch#609
arugupta1992 merged 3 commits intomax-ff-redesignfrom
max-feedback

Conversation

@aishwaryamathuria
Copy link
Collaborator

Pulling these 2 PRs to be pushed to ff redesign branch for max firefly testing
#604
#605

604

Addressing design feedback on prompt bar

Added shadow to prompt bar.
Increased corner radius on image/video (verb) drop down
Adjusted padding and gap to reduce the height of the prompt bar to better match Figma.
Added authoring ability to hide video models if needed. Note, this might not be needed but was added just in case for a quick authoring adjustment.
Note on authoring feature:
Adding hide-video-models as a variant to the Unity, Showcase or Hero blocks will hide video models.

Resolves: MWPW-NUMBER

Test URLs:

Before: https://main--cc--adobecom.aem.page/drafts/rclayton/firefly/prompt-max?unitylibs=stage&martech=off
After: https://main--cc--adobecom.aem.page/drafts/rclayton/firefly/prompt-max?unitylibs=max-prompt-design-feedback&martech=off
MAX Test URLs:

Before: https://max-ff-redesign--cc--adobecom.aem.page/drafts/jingle/firefly-max?milolibs=max25-theme&unitylibs=stage&martech=off
After: https://max-ff-redesign--cc--adobecom.aem.page/drafts/jingle/firefly-max?milolibs=max25-theme&unitylibs=max-prompt-design-feedback&martech=off

605

Fixes focus trap and tab focus issues reported in Slack.

Resolves: MWPW-182688

Test URLs:

@aem-code-sync
Copy link

aem-code-sync bot commented Oct 24, 2025

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

@aishwaryamathuria aishwaryamathuria requested review from Ruchika4, arugupta1992, rgclayton and shasmish and removed request for Ruchika4 October 24, 2025 12:41
@arugupta1992 arugupta1992 merged commit 012833f into max-ff-redesign Oct 24, 2025
4 of 6 checks passed
rgclayton pushed a commit that referenced this pull request Oct 24, 2025
…esign branch (#609)

Pulling these 2 PRs to be pushed to ff redesign branch for max firefly
testing
#604
#605

# 604
Addressing design feedback on prompt bar

Added shadow to prompt bar.
Increased corner radius on image/video (verb) drop down
Adjusted padding and gap to reduce the height of the prompt bar to
better match Figma.
Added authoring ability to hide video models if needed. Note, this might
not be needed but was added just in case for a quick authoring
adjustment.
Note on authoring feature:
Adding hide-video-models as a variant to the Unity, Showcase or Hero
blocks will hide video models.

Resolves: [MWPW-NUMBER](https://jira.corp.adobe.com/browse/MWPW-NUMBER)

Test URLs:

Before:
https://main--cc--adobecom.aem.page/drafts/rclayton/firefly/prompt-max?unitylibs=stage&martech=off
After:
https://main--cc--adobecom.aem.page/drafts/rclayton/firefly/prompt-max?unitylibs=max-prompt-design-feedback&martech=off
MAX Test URLs:

Before:
https://max-ff-redesign--cc--adobecom.aem.page/drafts/jingle/firefly-max?milolibs=max25-theme&unitylibs=stage&martech=off
After:
https://max-ff-redesign--cc--adobecom.aem.page/drafts/jingle/firefly-max?milolibs=max25-theme&unitylibs=max-prompt-design-feedback&martech=off

# 605 
Fixes focus trap and tab focus issues [reported in
Slack](https://adobe-mwp.slack.com/archives/C09MVV1NBE0/p1761257532527809).

Resolves: [MWPW-182688](https://jira.corp.adobe.com/browse/MWPW-182688)


**Test URLs:**
- Before: https://main--unity--adobecom.aem.page/?martech=off
- After: https://<branch>--unity--adobecom.aem.page/?martech=off

---------

Co-authored-by: Ryan Clayton <rclayton@adobe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants