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

[Featured Collection] Update maximum products to 24 #2096

Merged
merged 2 commits into from
Nov 30, 2022

Conversation

LucasLacerdaUX
Copy link
Contributor

PR Summary:

We updated the maximum amount of products displayed on the Featured Collection to be 24.

Why are these changes introduced?

Fixes #2093

What approach did you take?

Update the setting maximum value to 24.

Testing steps/scenarios

  • Add a featured collection section to any page on your theme
  • Change values from 13-24 and ensure they all look as expected on desktop
    • On mobile, test with both Enable swipe on mobile on and off

Demo links

Checklist

@LucasLacerdaUX LucasLacerdaUX changed the title [Featured Collection] Update maximum products to show to 24 [Featured Collection] Update maximum products to 24 Nov 3, 2022
@eugenekasimov eugenekasimov self-requested a review November 4, 2022 22:44
@eugenekasimov
Copy link
Contributor

eugenekasimov commented Nov 4, 2022

Hey Lucas, everything works as expected. The only thing that looks odd to me now and it was even before this PR is the fact that on desktop 24 products never perfectly match 5 columns. It feels like it should be 25 products x 5 columns or 24 products x 4 or 6 columns. Otherwise it looks this way which feels wrong:
Screen Shot 2022-11-04 at 3 49 59 PM

I know this is something about UX but I couldn't pass by :)

@sofiamatulis sofiamatulis self-assigned this Nov 10, 2022
@sofiamatulis sofiamatulis self-requested a review November 10, 2022 18:03
sofiamatulis
sofiamatulis previously approved these changes Nov 10, 2022
Copy link
Contributor

@sofiamatulis sofiamatulis left a comment

Choose a reason for hiding this comment

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

Works as expected 👍 I am curious about the decision as to why we are changing it 👀

eugenekasimov
eugenekasimov previously approved these changes Nov 10, 2022
Copy link
Contributor

@eugenekasimov eugenekasimov left a comment

Choose a reason for hiding this comment

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

I approve it now. But I still have the concern I mentioned above. I'll leave it for you and UX :)

@allylane
Copy link
Contributor

@LucasLacerdaUX It is generally working as expected but I'm getting this error when I change settings, despite it having the intended effect when I make a change.
Screenshot 2022-11-14 at 10 39 54 AM

@allylane
Copy link
Contributor

@eugenekasimov I think that's a good catch. Let's go with 25.

Copy link
Contributor

@eugenekasimov eugenekasimov left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

@sofiamatulis sofiamatulis left a comment

Choose a reason for hiding this comment

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

Works as expected 🎉

Do you mind sharing some context around this decision? 👀

@LucasLacerdaUX LucasLacerdaUX merged commit 4cdcd90 into main Nov 30, 2022
@LucasLacerdaUX LucasLacerdaUX deleted the featured-collection-max branch November 30, 2022 16:08
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* Update featured collection max to 24

* Update maximum to 25
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.

🌐 Featured Collection: Increase Maximum Products
4 participants