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

Enable gift card recipient form by default on featured product section #2666

Merged

Conversation

fredma
Copy link
Contributor

@fredma fredma commented May 22, 2023

PR Summary:

This enables by default the recipient form for gift cards when showcased on a featured product section.

Why are these changes introduced?

Fixes one of the priority tasks in #2454

What approach did you take?

Add block settings to buy_buttons in featured-product.liquid section like we already do for main-product.liquid

Other considerations

No

Visual impact on existing themes

Surfaces the recipient form checkbox on the featured product section if gift card is selected.

Screenshot 2023-05-22 at 17 28 47

Testing steps/scenarios

  • In theme editor, add featured product section on homepage and select a gift card product. The recipient form checkbox should be enabled by default.
  • Load storefront to view featured product section.

Demo links

  • Store (Scroll to bottom to view featured product section)
  • Editor

Checklist

@fredma fredma marked this pull request as ready for review May 22, 2023 16:36
@fredma fredma requested a review from ludoboludo May 22, 2023 16:38
Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

Thank you 👍

Copy link
Contributor

@LucasLacerdaUX LucasLacerdaUX left a comment

Choose a reason for hiding this comment

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

LGTM

@YoannJailin YoannJailin self-assigned this May 23, 2023
@ludoboludo ludoboludo merged commit ced607f into main May 23, 2023
@ludoboludo ludoboludo deleted the gift_cards/enable_recipient_form_on_featured_product_section branch May 23, 2023 16:15
@fredma fredma mentioned this pull request May 25, 2023
11 tasks
pangloss added a commit to pangloss/dawn that referenced this pull request Jun 14, 2023
* shopify/main: (59 commits)
  [Announcement bar] Add social icons (Shopify#2497)
  Update theme version to match the pubic release (Shopify#2698)
  Add release/v10.0.0 branch fixes to main (Shopify#2693)
  fix default UI for dropdown and mega menu (Shopify#2644)
  Fix link formatting in Related Products heading (Shopify#2680)
  Update 2 translation files (Shopify#2637)
  Enable gift card recipient form by default on featured product section (Shopify#2666)
  Gift cards/enable recipient form by default (Shopify#2618)
  Add a Color Scheme setting for Menus-Header (Shopify#2622)
  Made mobile drawer full width by default-header (Shopify#2625)
  Allow multiple announcement bars in Header group (Shopify#2619)
  [Feat Product] Add rating styling sheet (Shopify#2620)
  Fix password page variables (Shopify#2607)
  Fix transform applied when it should not for sliders (Shopify#2606)
  Modify info string for gift card recipient checkbox (Shopify#2588)
  [3D lift animation] Raise hovered card above adjacent cards (Shopify#2589)
  Remove fallback color scheme info text (Shopify#2602)
  Fix CSS specifity issue to cancel animation for theme editor events (Shopify#2605)
  Remove settings daya for icon color (Shopify#2601)
  Follow ups for accessibility of announcements slider (Shopify#2580)
  ...
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
This pull request was closed.
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