-
Notifications
You must be signed in to change notification settings - Fork 600
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
Add option to display custom fields under "Additional Information" tab on PDP #1480
Conversation
Autotagging @bigcommerce/storefront-team @davidchin |
@@ -1285,6 +1285,20 @@ | |||
"force_reload": true, | |||
"id": "show_product_details_tabs" | |||
}, | |||
{ |
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.
@bigcommerce/merc-team
@@ -7,6 +7,11 @@ | |||
<a class="tab-title" href="#tab-warranty">{{lang 'products.warranty'}}</a> | |||
</li> | |||
{{/if}} | |||
{{#all product.custom_fields theme_settings.show_custom_fields_tabs}} | |||
<li class="tab"> | |||
<a class="tab-title" href="#tab-specs">{{theme_settings.pdp-custom-fields-tab-label}}</a> |
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.
Can we build the href based on the label so it matches what we are actually calling the tab.
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.
@@ -22,6 +27,16 @@ | |||
{{{product.warranty}}} | |||
</div> | |||
{{/if}} | |||
{{#all product.custom_fields theme_settings.show_custom_fields_tabs}} | |||
<div class="tab-content" id="tab-specs"> |
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.
Can we build the href based on the label so it matches what we are actually calling the tab.
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.
@junedkazi Ran the string through |
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.
LGTM
config.json
Outdated
@@ -304,6 +305,7 @@ | |||
"pdp-sale-price-label": "Now:", | |||
"pdp-non-sale-price-label": "Was:", | |||
"pdp-retail-price-label": "MSRP:", | |||
"pdp-custom-fields-tab-label": "Specifications", |
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.
Can we also change the label to be more generic like "Additional Information" ? 🍹
What?
Adds an option to display custom fields under a "Additional Information" tab.
Tickets / Documentation
Screenshots (if appropriate)
False, default
False, default - tab region
True
True, tab region - closed
True, tab region - open
True, tab region - open, mobile
True, tab region - closed