-
Notifications
You must be signed in to change notification settings - Fork 4
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
Rename career-hero to hero-career to follow hero block guidelines #78
Conversation
This also fixes a layout issue with this block
Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
|
🔸 1 visual difference detected
The diff images are attached in the artifact |
|
|
|
🔸 1 visual difference detected
The diff images are attached in the artifact |
heroRight.classList.add('hero-career-right'); | ||
heroDiv.appendChild(heroRight); | ||
addTextEl('blockquote', cfg.quote, heroRight, 'hero-career-quote'); | ||
addTextEl('h6', 'Career background', heroRight, 'hero-career-careerbgtitle'); // TODO i18n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how are we tracking these todos ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will have to have a separate effort (probably fairly small) to make these pages work with Japanese. This issue will become evident then.
|
0cf0e45
to
aef9361
Compare
|
🔸 2 visual differences detected
The diff images are attached in the artifact |
🔸 2 visual differences detected
The diff images are attached in the artifact |
This fixes a layout problem with the hero block for career testimonials
Fixes #15
Test URLs: