Skip to content

[MWPW-193582] - Optimize global navigation sticky detection and animation layers#5897

Open
DKos95 wants to merge 2 commits intostagefrom
perf-global-navigation-optimizations
Open

[MWPW-193582] - Optimize global navigation sticky detection and animation layers#5897
DKos95 wants to merge 2 commits intostagefrom
perf-global-navigation-optimizations

Conversation

@DKos95
Copy link
Copy Markdown
Contributor

@DKos95 DKos95 commented May 5, 2026

Replaces the getBoundingClientRect() check in the scroll handler for local nav sticky state with an IntersectionObserver at threshold: 1, removing the layout-triggering rect read from the scroll hot path entirely. The scroll listener is now also marked { passive: true }. Also adds will-change: translate, opacity to the nav link slide/fade animations so the browser promotes those elements to the compositor upfront.

Resolves: MWPW-193582

PSI Test URLs:
Before: https://main--milo--adobecom.aem.page/?martech=off
After: https://perf-global-navigation-optimizations--milo--adobecom.aem.page/?martech=off

Test URLs:
Before: https://load-c2-styles--upp--adobecom.aem.live/homepage/drafts/ramuntea/redesign-demo?milolibs=site-redesign-foundation
After: https://load-c2-styles--upp--adobecom.aem.live/homepage/drafts/ramuntea/redesign-demo?milolibs=perf-global-navigation-optimizations

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@DKos95 DKos95 requested a review from a team May 5, 2026 13:37
@DKos95 DKos95 requested a review from a team as a code owner May 5, 2026 13:37
@aem-code-sync
Copy link
Copy Markdown
Contributor

aem-code-sync Bot commented May 5, 2026

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

  • Re-sync branch
Commits

Copy link
Copy Markdown

@SilviuLCF SilviuLCF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bandana147
Copy link
Copy Markdown
Contributor

@DKos95 This is not C2 block, C1 gnav this has lot of combination with sticky banner and promos as well. This will need thorough testing of the C1 gnav

cc: @spadmasa

@DKos95 DKos95 changed the base branch from site-redesign-foundation to stage May 6, 2026 08:59
@DKos95
Copy link
Copy Markdown
Contributor Author

DKos95 commented May 6, 2026

@DKos95 This is not C2 block, C1 gnav this has lot of combination with sticky banner and promos as well. This will need thorough testing of the C1 gnav

cc: @spadmasa

My mistake, it should still be a nice improvement though for c1, wether your team has the bandwidth to test this thoroughly is another thing.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

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.

6 participants