Skip to content

Commit 96bb525

Browse files
committed
MC-5410: Banner & Slide Are Missing <A> Tag For Link Attribute On Storefront
Convert data-target attribute to target
1 parent 7d9ba9e commit 96bb525

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

app/code/Magento/PageBuilder/view/adminhtml/web/js/property/link.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

app/code/Magento/PageBuilder/view/adminhtml/web/ts/js/property/link.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ export default class Link implements PropertyReaderInterface {
3434

3535
return {
3636
[attributeLinkType]: href,
37-
setting: (element.getAttribute("data-target") || element.getAttribute("target")) === "_blank",
37+
setting: element.getAttribute("target") === "_blank",
3838
type: attributeLinkType,
3939
};
4040
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<div data-element="main" data-role="row" data-appearance="contained"><div data-element="inner"><div data-element="main" data-role="banner" data-appearance="poster" data-show-button="never" data-show-overlay="never" class="one two" style="background-size: auto; background-repeat: no-repeat; background-attachment: scroll; border: 1px none; border-radius: 0px; margin: 0px;"><div data-element="link" href="http://domain.com/" data-target="_blank"><div data-element="wrapper" data-background-images="{\&quot;desktop_image\&quot;:\&quot;{{media url=wysiwyg/m/a/magento1.png}}\&quot;,\&quot;mobile_image\&quot;:\&quot;{{media url=wysiwyg/m/a/magento1.png}}\&quot;}" style="min-height: 300px; background-size: auto; background-repeat: no-repeat; background-attachment: scroll;" class="pagebuilder-banner-wrapper"><div data-element="overlay" class="pagebuilder-poster-overlay" data-overlay-color="transparent" style="background-color: transparent; min-height: 300px; padding: 40px;"><div class="pagebuilder-poster-content"><div data-element="content"></div></div></div></div><div data-element="overlay" class="pagebuilder-poster-overlay" data-overlay-color="transparent" style="background-color: transparent; min-height: 300px; padding: 40px;"><div class="pagebuilder-poster-content"><div data-element="content"></div></div></div></div></div></div></div></div>
1+
<div data-element="main" data-role="row" data-appearance="contained"><div data-element="inner"><div data-element="main" data-role="banner" data-appearance="poster" data-show-button="never" data-show-overlay="never" class="one two" style="background-size: auto; background-repeat: no-repeat; background-attachment: scroll; border: 1px none; border-radius: 0px; margin: 0px;"><div data-element="link" href="http://domain.com/" target="_blank"><div data-element="wrapper" data-background-images="{\&quot;desktop_image\&quot;:\&quot;{{media url=wysiwyg/m/a/magento1.png}}\&quot;,\&quot;mobile_image\&quot;:\&quot;{{media url=wysiwyg/m/a/magento1.png}}\&quot;}" style="min-height: 300px; background-size: auto; background-repeat: no-repeat; background-attachment: scroll;" class="pagebuilder-banner-wrapper"><div data-element="overlay" class="pagebuilder-poster-overlay" data-overlay-color="transparent" style="background-color: transparent; min-height: 300px; padding: 40px;"><div class="pagebuilder-poster-content"><div data-element="content"></div></div></div></div><div data-element="overlay" class="pagebuilder-poster-overlay" data-overlay-color="transparent" style="background-color: transparent; min-height: 300px; padding: 40px;"><div class="pagebuilder-poster-content"><div data-element="content"></div></div></div></div></div></div></div></div>

0 commit comments

Comments
 (0)