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

MWPW-167207: Plans card pixel perfect #3704

Open
wants to merge 23 commits into
base: stage
Choose a base branch
from
Open

Conversation

st-angelo-adobe
Copy link
Contributor

@st-angelo-adobe st-angelo-adobe commented Feb 19, 2025

Resolves: MWPW-167207

Test URLs:
Before: https://mwpw-164492--milo--adobecom.aem.page/docs/library/kitchen-sink/merch-card?martech=off
After: https://mwpw-167207--milo--adobecom.aem.page/docs/library/kitchen-sink/merch-card?martech=off

Plans Gallery:
mwpw-167207--milo--adobecom.aem.page/libs/features/mas/docs/plans.html?martech=off

Copy link
Contributor

aem-code-sync bot commented Feb 19, 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 19, 2025

@st-angelo-adobe st-angelo-adobe marked this pull request as ready for review February 19, 2025 13:56
@DKos95
Copy link
Contributor

DKos95 commented Feb 19, 2025

  1. After link leads to 404
  2. Lighthouse score is failing
  3. There are conflicts for :
    libs/deps/mas/mas.js
    libs/features/mas/dist/mas.js

@narcis-radu
Copy link
Contributor

hey @st-angelo-adobe , I noticed some visual issues. Please see the figma specs, I think the info icon should still be top right and the info container should have different sizing.
Screenshot 2025-02-20 at 19 53 29
Screenshot 2025-02-20 at 19 53 34

Copy link
Contributor

@narcis-radu narcis-radu 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

Copy link
Contributor

github-actions bot commented Mar 1, 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 1, 2025
Base automatically changed from MWPW-164492 to stage March 3, 2025 10:27
@3ch023 3ch023 removed the Stale label Mar 3, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/c8c26347d39fb02898042bb6aeb9e25f178a4e81/libs/deps/mas/merch-whats-included.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants