Skip to content

[MWPW-182688] MAX - Addressing tab focus issues.#605

Closed
rgclayton wants to merge 2 commits intostagefrom
max-prompt-tab-focus
Closed

[MWPW-182688] MAX - Addressing tab focus issues.#605
rgclayton wants to merge 2 commits intostagefrom
max-prompt-tab-focus

Conversation

@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 self-requested a review October 24, 2025 11:41
@shasmish shasmish requested a review from arugupta1992 October 24, 2025 11:43
arugupta1992 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>
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>
sanjayms01 pushed a commit that referenced this pull request Oct 24, 2025
max-ff-redesign PR's to stage
* #608
* #605
* #604
* #610

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

**Test URLs:**
- Before: https://main--unity--adobecom.aem.page/?martech=off
- After: https://<branch>--unity--adobecom.aem.page/?martech=off
@rgclayton
Copy link
Contributor Author

Closing. These were merged into max-ff-redesign and that branch was merged to stage.

@rgclayton rgclayton closed this Oct 24, 2025
sanjayms01 pushed a commit that referenced this pull request Oct 24, 2025
max-ff-redesign PR's to stage
* #608
* #605
* #604
* #610

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

**Test URLs:**
- Before: https://main--unity--adobecom.aem.page/?martech=off
- After: https://<branch>--unity--adobecom.aem.page/?martech=off
sanjayms01 pushed a commit that referenced this pull request Oct 24, 2025
max-ff-redesign PR's to stage
* #608
* #605
* #604
* #610

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

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