Skip to content
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

💸 Mega Menu - Reduced Bundle Size #1384

Merged
merged 3 commits into from
Sep 15, 2023
Merged

💸 Mega Menu - Reduced Bundle Size #1384

merged 3 commits into from
Sep 15, 2023

Conversation

Harry-Ross
Copy link
Contributor

@github-actions
Copy link
Contributor

Deployed changes to https://app-sswwebsite-9eb3-pr-1384.azurewebsites.net

@github-actions
Copy link
Contributor

PageSpeed Insights

Category Score
Accessibility 91
Best practices 91
Performance 31
SEO 100

View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-1384.azurewebsites.net

Copy link
Member

@bradystroud bradystroud left a comment

Choose a reason for hiding this comment

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

Megamenu images are broken

image

@bradystroud
Copy link
Member

Seems to be looking for images using a relative link

image

@Harry-Ross
Copy link
Contributor Author

Seems to be looking for images using a relative link

image

@bradystroud

Correct, as per SSWConsulting/SSW.Website.Menu.json#99 - this change will fix the images not showing up, but deployments must be coordinated with SSW.People and SSW.Rules to ensure no broken menu across each site.

@Harry-Ross Harry-Ross marked this pull request as ready for review September 15, 2023 05:14
@github-actions
Copy link
Contributor

Deployed changes to https://app-sswwebsite-9eb3-pr-1384.azurewebsites.net

@github-actions
Copy link
Contributor

PageSpeed Insights

Category Score
Accessibility 91
Best practices 95
Performance 26
SEO 100

View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-1384.azurewebsites.net

@Harry-Ross Harry-Ross merged commit af155b9 into main Sep 15, 2023
9 checks passed
@Harry-Ross Harry-Ross deleted the 1357-updated-megamenu branch September 15, 2023 06:54
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.

💸 Megamenu - Bundle Size Too Large
2 participants