8000 MWPW-167716 Remove await from both gnav and georouting by sharmrj · Pull Request #3680 · adobecom/milo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

MWPW-167716 Remove await from both gnav and georouting #3680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 13, 2025

Conversation

sharmrj
Copy link
Contributor
@sharmrj sharmrj commented Feb 13, 2025

This is the first second of four five PRs (the first is #3720) aimed at improving Global-Navigation performance. This first PR does not directly affect the global-navigation (or georouting), but is rather aimed at improving the load times of the sections after the first one. For more information look at the discussion #3558

The first section is assumed to be LCP and is prioritized. After the first section we load a series of features/blocks, among them georouting and global-navigation. Since we await both of these blocks, the subsequent sections cannot continue until they stop blocking the main thread.

This can be bad because the user cannot see the rest of the page, or even scroll, until the subsequent sections load. The affect of this poor UX is not something we measure but it is important nonetheless. And even though we assume LCP should be in the first section, in practice it is not always.

The solution is then to remove the awaits on the global-navigation and georouting. The effects of this change have been documented in the discussion #3558.

Resolves: MWPW-167716

Test URLs:

Copy link
Contributor
aem-code-sync bot commented Feb 13, 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

Copy link
Contributor
aem-code-sync bot commented Feb 13, 2025
Page Scores Audits Google
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

}
const header = document.querySelector('header');
if (header) {
header.classList.add('gnav-hide');
await loadBlock(header);
loadBlock(header);
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any influence on CLS if you do this? Also what happens in terms of personalization? What if that loads before the GNAV?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No there's no CLS.
AFAICT if personalization must load it will load before either gnav or georouting; either right before georouting in loadPostLCP or in checkForPageMods. In both cases it is awaited. There is some personalization that takes place inside the gnav, but simply removing the await does not affect it in any way.

@@ -1192,13 +1192,15 @@ async function loadPostLCP(config) {

const georouting = getMetadata('georouting') || config.geoRouting;
if (georouting === 'on') {
const { default: loadGeoRouting } = await import('../features/georoutingv2/georoutingv2.js');
await loadGeoRouting(config, createTag, getMetadata, loadBlock, loadStyle);
import('../features/georoutingv2/georoutingv2.js')
Copy link
Contributor

Choose a reason for hiding this comment

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

Given @vhargrave concerns in the discussion, I suppose this was brought up with the georouting PM?

Copy link
Contributor

This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR.

@sharmrj
Copy link
Contributor Author
sharmrj commented Feb 14, 2025

To give an idea of what this change accomplishes, it essentially schedules our network requests postLCP more efficiently. As a visual representation, consider the below screen shot of our page load currently.

Screenshot 2025-02-14 at 9 25 54 AM

We see that many network requests are chained, blocking further requests until they are received. Specifically there's a visible bottleneck in the middle there. That happens to be georouting and global-navigation. See the trace here:
Trace-20250214T092251.json.zip

Now with the change in this PR:

Screenshot 2025-02-14 at 9 25 19 AM trace:

Trace-20250214T092719.json.zip

We can see that the bottleneck is gone. By removing just these two awaits we'll see fewer gaps in how our code and, subsequently, our network requests are scheduled (This is what I mean by scheduling more efficiently).

@salonijain3 salonijain3 added the high-impact Any PR that may affect consumers label Feb 17, 2025
Copy link
Contributor
@vhargrave vhargrave left a comment

Choose a reason for hiding this comment

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

see comments

@@ -1192,13 +1192,15 @@ async function loadPostLCP(config) {

const georouting = getMetadata('georouting') || config.geoRouting;
if (georouting === 'on') {
const { default: loadGeoRouting } = await import('../features/georoutingv2/georoutingv2.js');
await loadGeoRouting(config, createTag, getMetadata, loadBlock, loadStyle);
import('../features/georoutingv2/georoutingv2.js')
Copy link
Contributor

Choose a reason for hiding this comment

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

If you haven't done the performance enhancement we discussed for the georouting, then please confirm that this delay was tested and discussed with the georouting PM.
Otherwise please at least do the enhancement that was discussed (kicking off the loading of the .json file here).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We've tagged them on JIRA, and we'll continue the conversation there.
I looked into doing that optimization here, but I think it would fit well as its own PR. I'd prefer to keep the PRs as granular as we can. If you prefer I can make it the very next optimization we do.

Copy link
Contributor

Choose a reason for hiding this comment

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

I understand that, but I can't approve a change to the load time of the georouting that goes live if you haven't approved it with the georouting PM or mitigated it with the change we discussed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok I've raised a separate PR to optimize georouting first and then we can merge this one. Here's the PR @vhargrave : #3720

Copy link
Contributor

Choose a reason for hiding this comment

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

@sharmrj perfect, thank you.
I've left some feedback 🙌

}
const header = document.querySelector('header');
if (header) {
header.classList.add('gnav-hide');
await loadBlock(header);
loadBlock(header);
header.classList.remove('gnav-hide');
}
loadTemplate();
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a must, but i bet you could kick the loading of the fonts.js at the beginning of the postLCP somewhere and only await it here.. That would probably also give you some great savings.

Copy link
Contributor
github-actions bot commented Mar 4, 2025

This PR has not been updated recently and will be closed in 7 days if no action is taken. Please ensure all checks are passing, https://github.com/orgs/adobecom/discussions/997 provides instructions. If the PR is ready to be merged, please mark it with the "Ready for Stage" label.

@github-actions github-actions bot added the Stale label Mar 4, 2025
@aem-code-sync aem-code-sync bot temporarily deployed to non-blocking-gnav-geourouting March 5, 2025 07:12 Inactive
@sharmrj sharmrj requested a review from vhargrave March 5, 2025 07:12
@vhargrave
Copy link
Contributor

@sharmrj nala tests are failing for some reason. Can you take a look?

@sharmrj sharmrj removed the Stale label Mar 5, 2025
@sharmrj
Copy link
Contributor Author
sharmrj commented Mar 5, 2025

@vhargrave I'm not sure what the problem was but they've passed now.

Copy link
Contributor
@vhargrave vhargrave left a comment

Choose a reason for hiding this comment

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

nice!

@sigadamvenkata
Copy link

this was tested in given branch. complete testing would be done for integrations, devices etc on stage.

@sigadamvenkata sigadamvenkata added verified PR has been E2E tested by a reviewer and removed needs-verification PR requires E2E testing by a reviewer labels Mar 13, 2025
@milo-pr-merge milo-pr-merge bot merged commit 57ded10 into stage Mar 13, 2025
22 checks passed
@milo-pr-merge milo-pr-merge bot deleted the non-blocking-gnav-geourouting branch March 13, 2025 17:38
@milo-pr-merge milo-pr-merge bot mentioned this pull request Mar 13, 2025
mokimo pushed a commit that referenced this pull request Mar 14, 2025
* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
@milo-pr-merge milo-pr-merge bot mentioned this pull request Mar 14, 2025
milo-pr-merge bot added a commit that referenced this pull request Apr 22, 2025
…ation (#3969)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3824)

MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3630)

* updated an svg inside a url() in the globalnav css to have data:image/svg+xml;utf8 inorder to prevent a console error

* changed a ; to a ,

* Removed the svg as it was unnecessary

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* MWPW-170562: saving work

* saving work

* MWPW-170562: sorted lang & country dropdowns

* MWPW-170562: unit tests

* MWPW-170562: unit tests

* MWPW-170562: unit tests

* MWPW-170562: lint

* MWPW-170562: auto country/lang

* MWPW-170562: auto country/lang

* MWPW-170562: sort

* lint

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
Co-authored-by: Sheridan Sunier <sunier@macbookpro.lan>
sheridansunier pushed a commit to sheridansunier/milo that referenced this pull request May 12, 2025
* ENB-7820: Add values for event95 (adobecom#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (adobecom#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (adobecom#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (adobecom#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (adobecom#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (adobecom#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (adobecom#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (adobecom#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
sheridansunier pushed a commit to sheridansunier/milo that referenced this pull request May 12, 2025
* ENB-7820: Add values for event95 (adobecom#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (adobecom#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (adobecom#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (adobecom#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (adobecom#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (adobecom#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (adobecom#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (adobecom#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
milo-pr-merge bot added a commit that referenced this pull request May 13, 2025
…ion (#4102)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces
C403


* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-170933: saving work

* MWPW-170933: bulk publisher support for language first

* lint

* ui changes

* tool-tip

* chris code change

* chris changes

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
sheridansunier pushed a commit to sheridansunier/milo that referenced this pull request May 13, 2025
* ENB-7820: Add values for event95 (adobecom#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (adobecom#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (adobecom#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (adobecom#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (adobecom#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (adobecom#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (adobecom#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (adobecom#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
milo-pr-merge bot added a commit that referenced this pull request May 15, 2025
* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* add pathname into config for autodetect purposes

* MWPW-172910: autodetect localization

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* lint

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
ivanvatadobe pushed a commit that referenced this pull request May 15, 2025
…ion (#4102)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-170933: saving work

* MWPW-170933: bulk publisher support for language first

* lint

* ui changes

* tool-tip

* chris code change

* chris changes

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

* weird merge issue

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
ivanvatadobe pushed a commit that referenced this pull request May 15, 2025
* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* add pathname into config for autodetect purposes

* MWPW-172910: autodetect localization

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* merge weirdness

* lint

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
milo-pr-merge bot added a commit that referenced this pull request Jun 9, 2025
* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3824)

MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3630)

* updated an svg inside a url() in the globalnav css to have data:image/svg+xml;utf8 inorder to prevent a console error

* changed a ; to a ,

* Removed the svg as it was unnecessary

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* MWPW-162398 draft

* MWPW-162398 enable mas preview

- getSettings will consider preview attribute of mas-commerce-service presence as the fact it should be using preview,
- if preview is on, then using mas' previewFragment client library to act like adobeio, but on browser,
- surface used (for stuff like placeholders) is either computed depending on wcs api key, or coming from preview parameter value,
- is url is .page, or www.stage, then preview is set by default,
- preview configuration can be overriden with masPreview parameter (url or local storage) set to off for disabling, or a value to use another surface (if api key is not corresponding to a registered surface)

* some fixes

* some other fixes

* some doc and changes

* small fixes

* Reduce gnav lana sampleRate to 0.01% (#4150)

* review comment

* Revert "Improve carousel accessibility (#4103)" from prod (#4195)

Revert "Improve carousel accessibility (#4103)"

This reverts commit 34d2016.

* some fixes

* review comments

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
skholkhojaev added a commit to skholkhojaev/milo that referenced this pull request Jun 11, 2025
* [MWPW-173100][NALA] Automation test script for Timeline block (adobecom#4301)

Update nala automation tests for timeline

* Fixing replace check for Gnav test url (adobecom#4333)

Fixing replace check for test url

* MWPW-170229: Show action scroller nav on resize (adobecom#4311)

* MWPW-170229: Show action scroller nav on resize

* MWPW-170229: Debounce scroll and resize events

* [MWPW-173161] - Refactor preflight to export checks (adobecom#4109)

* MWPW-170805 initial refactoring of seo checks.

* MWPW-170805 update the links method to cache its result

* MWPW-170805 allow for document to be passed

* update preflight

* update performance checks

* update assets checks

* don't pass url if not necessary to assets

* cache image result

* import mini-preflight in utils

* hotfix - seo title typos

* hotfixes

* hotfix

* hotfix

* Removed unnecessary comments

* fixed linter errors

* removed old performance panel unit tests

* Mwpw 170378 preflight assets unit test  (adobecom#4131)

* added unit tests for assets tab in preflight

* fixed linting

* centralized checks imports

* added unit tests

---------

Co-authored-by: Victor Hargrave <vhargrave@adobe.com>
Co-authored-by: Sino Kholkhojaev <132879006+skholkhojaev@users.noreply.github.com>

* MWPW-162398 enable mas preview  (adobecom#4148)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (adobecom#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (adobecom#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (adobecom#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (adobecom#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (adobecom#3817)

* ENB-7820: Add values for event95 (adobecom#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (adobecom#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (adobecom#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (adobecom#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (adobecom#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (adobecom#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (adobecom#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (adobecom#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-167191 Removed an unnecessary css url() that was causing a console error (adobecom#3824)

MWPW-167191 Removed an unnecessary css url() that was causing a console error (adobecom#3630)

* updated an svg inside a url() in the globalnav css to have data:image/svg+xml;utf8 inorder to prevent a console error

* changed a ; to a ,

* Removed the svg as it was unnecessary

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* MWPW-162398 draft

* MWPW-162398 enable mas preview

- getSettings will consider preview attribute of mas-commerce-service presence as the fact it should be using preview,
- if preview is on, then using mas' previewFragment client library to act like adobeio, but on browser,
- surface used (for stuff like placeholders) is either computed depending on wcs api key, or coming from preview parameter value,
- is url is .page, or www.stage, then preview is set by default,
- preview configuration can be overriden with masPreview parameter (url or local storage) set to off for disabling, or a value to use another surface (if api key is not corresponding to a registered surface)

* some fixes

* some other fixes

* some doc and changes

* small fixes

* Reduce gnav lana sampleRate to 0.01% (adobecom#4150)

* review comment

* Revert "Improve carousel accessibility (adobecom#4103)" from prod (adobecom#4195)

Revert "Improve carousel accessibility (adobecom#4103)"

This reverts commit 34d2016.

* some fixes

* review comments

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>

* MWPW-171672: Copy to clipboard for checkout CTAs (adobecom#4171)

* added copying to clipboard

* lint fix

* moved to a separate module

* used getMetadata

* added a tooltip

* fixed brackets

* added ClipboardItem

* updated tests

* replaced the icon

* updated unit test

* MWPW-172886 [MEP][MILO] Dynamic content on RFI page via referrer | MEP (adobecom#4273)

* first commit

* Enhance path normalization to include localhost and simplify previous page match logic

* Refactor checkForPreviousPageMatch to accept referrer as a parameter and add comprehensive tests

* Remove commented suggestion for error handling

* Refactor checkForPreviousPageMatch to use lastVisitedPage parameter for improved clarity

* Exposing close gnav function for clients (adobecom#4274)

* Exposing close gnav function for clients

* Lint fix

* Add a check if the gnav is in open state before calling toggle

* MWPW-173759: Unordered list styling for merch cards (adobecom#4298)

* MWPW-173759: Unordered list styling for merch cards

* style specificity fix

* [MWPW-173502] Remove hash from how-to schema (adobecom#4299)

* [MWPW-174367] Inline fragment not behaving as expected in block (adobecom#4317)

* Initial checkin. Working state.

* Move replacecell handling to utils from fragments

* Fix: Replace hardcoded string with constant for replacecell handling

* Refactor: Replace constant OVERRIDE_AUTO_INLINE with inline string in modal handling

* Unit testing.

* Unit testing.

* Removed unnecessary test statement.

---------

Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>

* MWPW-172018 Update default price for CIS umbrella regions to TM (adobecom#4323)

Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>

* MWPW-173830: Add tooltip to notification pill (adobecom#4295)

* MWPW-173830: Add tooltip to notification pill

* MWPW-173830: PR feedback and add checks

* MWPW-173528 Fix Safari mobile preview (adobecom#4231)

* MWPW-173528 Fix Safari mobile preview

Safari requires that the spoofed user agent be set on the load event and not before that.

* Show the currently selected device view

* Remove flicker when switching between devices

* Fix CodeQL Incomplete URL substring sanitization

---------

Co-authored-by: Biljana Cvijanovic <hit11757@adobe.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Ratko Zagorac <90400759+zagi25@users.noreply.github.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Victor Hargrave <vhargrave@adobe.com>
Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
Co-authored-by: Mira Fedas <30750556+mirafedas@users.noreply.github.com>
Co-authored-by: Jan Ivan Viloria <158211769+ivanvatadobe@users.noreply.github.com>
Co-authored-by: Angelo Statescu <angelostatescu.adobe@gmail.com>
Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>
Co-authored-by: Dave Linhart <132396886+AdobeLinhart@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>
Co-authored-by: Bozo Jovicic <37440641+bozojovicic@users.noreply.github.com>
Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>
sheridansunier pushed a commit to sheridansunier/milo that referenced this pull request Jun 16, 2025
* ENB-7820: Add values for event95 (adobecom#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (adobecom#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (adobecom#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (adobecom#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (adobecom#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (adobecom#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (adobecom#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (adobecom#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
milo-pr-merge bot added a commit that referenced this pull request Jun 17, 2025
…on contrast / navigation title (#4355)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3824)

MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3630)

* updated an svg inside a url() in the globalnav css to have data:image/svg+xml;utf8 inorder to prevent a console error

* changed a ; to a ,

* Removed the svg as it was unnecessary

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* Reduce gnav lana sampleRate to 0.01% (#4150)

* Revert "Improve carousel accessibility (#4103)" from prod (#4195)

Revert "Improve carousel accessibility (#4103)"

This reverts commit 34d2016.

* Revert "[MWPW-173470] - Korea free trial restrict" (main) (#4236)

Revert "[MWPW-173470] - Korea free trial restrict (#4185)"

This reverts commit 351cfe6.

* MWPW-173888 + MWPW-173919: Appending extra options for relative urls & multiple cta reopen fix (#4237)

* MWPW-173875 direct to main

* Korea free trial restrict (#4282)

* [MWPW-173016] - block korea free trial links/buttons

* [MWPW-173016] - korea text filter added

* [MWPW-173016] - update if

* [MWPW-173016] - code optimization

* [MWPW-173016] - improve import

* [MWPW-173016] - modal check, string check added

* [MWPW-173016] - strings added

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - remove whitespace

* [MWPW-173470] - optimize

* [MWPW-173470] - remove variable

* [MWPW-173016] - optimize code

* [MWPW-173016] - group logic

* [MWPW-173016] - merch solution added

* [MWPW-173016] - merch ref comment added

* [MWPW-173470] - null safe

* [MWPW-173470] - code optimization

* [MWPW-173470] - nav korea restrict coverage

* [MWPW-173470] - fix eslint console error

---------

Co-authored-by: Dusan Kosanovic <dusan.kosanovic@hitthecode.com>

* Revert "[MWPW-173717] Hyphenate large headings on mobile (#4199)"

This reverts commit 7b82821.

* Update utilities.js (#4312)

Making a change to the utilities file

Co-authored-by: Saloni Jain <6162294+salonijain3@users.noreply.github.com>

* MWPW-173772 [merch-card-collection] introduce mobile filter text

* MWPW-173772 fix for catalog

* icon & display text mgt

* fix

* more fixes

* add more unit test

* fixing autoblock fill

* MWPW-173772 bump sidenav h2 font size by 1px

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
Co-authored-by: Angelo Statescu <angelostatescu.adobe@gmail.com>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>
Co-authored-by: Narcis Radu <github@narcisradu.ro>
Co-authored-by: Saloni Jain <6162294+salonijain3@users.noreply.github.com>
Co-authored-by: Mariia Lukianets <mariia.lukianets@gmail.com>
milo-pr-merge bot added a commit that referenced this pull request Jun 23, 2025
…nt (#4417)

* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* Revert "Improve carousel accessibility (#4103)" from prod (#4195)

Revert "Improve carousel accessibility (#4103)"

This reverts commit 34d2016.

* Revert "[MWPW-173470] - Korea free trial restrict" (main) (#4236)

Revert "[MWPW-173470] - Korea free trial restrict (#4185)"

This reverts commit 351cfe6.

* MWPW-173888 + MWPW-173919: Appending extra options for relative urls & multiple cta reopen fix (#4237)

* Korea free trial restrict (#4282)

* [MWPW-173016] - block korea free trial links/buttons

* [MWPW-173016] - korea text filter added

* [MWPW-173016] - update if

* [MWPW-173016] - code optimization

* [MWPW-173016] - improve import

* [MWPW-173016] - modal check, string check added

* [MWPW-173016] - strings added

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - remove whitespace

* [MWPW-173470] - optimize

* [MWPW-173470] - remove variable

* [MWPW-173016] - optimize code

* [MWPW-173016] - group logic

* [MWPW-173016] - merch solution added

* [MWPW-173016] - merch ref comment added

* [MWPW-173470] - null safe

* [MWPW-173470] - code optimization

* [MWPW-173470] - nav korea restrict coverage

* [MWPW-173470] - fix eslint console error

---------

Co-authored-by: Dusan Kosanovic <dusan.kosanovic@hitthecode.com>

* Revert "[MWPW-173717] Hyphenate large headings on mobile (#4199)"

This reverts commit 7b82821.

* MWPW-74582: ensure that empty primaryCtas are not being sent through to caas

* ugh rebasing stage is the worst

* omg literally the worst

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* rebase weirdness

* omg this whitespace will be the death of me

* MWPW-74582: carlos revsion

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Angelo Statescu <angelostatescu.adobe@gmail.com>
Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>
Co-authored-by: Narcis Radu <github@narcisradu.ro>
Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high-impact Any PR that may affect consumers Ready for Stage verified PR has been E2E tested by a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants
0