Skip to content

Markpadbe branch - #8

Open
markpadbe wants to merge 7 commits into
mainfrom
markpadbe_branch
Open

Markpadbe branch#8
markpadbe wants to merge 7 commits into
mainfrom
markpadbe_branch

Conversation

@markpadbe

Copy link
Copy Markdown
Collaborator
  • Update init console message

Resolves: MWPW-NUMBER

Test URLs:

@aem-code-sync

aem-code-sync Bot commented Feb 6, 2024

Copy link
Copy Markdown

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

@markpadbe
markpadbe marked this pull request as draft February 6, 2024 22:37
@markpadbe
markpadbe marked this pull request as ready for review February 6, 2024 22:54
@markpadbe
markpadbe marked this pull request as draft February 6, 2024 22:54

@akobrina akobrina left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Requesting change to console log message.

Comment thread libs/blocks/marquee/marquee.js Outdated
console.log('blah blah');
}

if (el.classList.contains('new-variant')) console.log('got it...in the new branch!');

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks great!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@akobrina, Thank you. I've updated the code.

@ivanvatadobe

Copy link
Copy Markdown
Collaborator

this looks good to me

@markpadbe
markpadbe requested a review from akobrina February 6, 2024 23:09

@akobrina akobrina left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Much better! Thanks.

@markpadbe
markpadbe marked this pull request as ready for review February 6, 2024 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants