feat(snap-platforms)

[chrisFrazier77]

snap-platforms added for shopify, bigcommerce, & magento2

[korgon]

This is the way.

Need to clean up the readme files and error logs to remove plugin references.

[korgon]

This is looking good. I'd like to discuss some more about the "callback" and async usage of these addToCart functions before we set the interface/usage in stone.

[korgon]

Types

• add available in ListOption type
• refactor List, Grid, Swatches to new type - add
• need class modifiers in components based on ListOption value --disabled --unavailable
• styling should be the same for class modifiers above
• disabled options should not be clickable

export type ListOption = {
	value: string | number;
	label?: string;
	disabled?: boolean;
	available?: boolean;
	default?: boolean;
	icon?: string | Partial<IconProps>;
	url?: UrlManager;
};

Grid Component

• List component has a 'title' why not this one?
• need story for overflowing grid and for options with backgrounds (image or whatever)
• prop for hiding "show less" (when you don't want to show less)
• :before pseudo element for "disabled" strike should not have fixed width

List Component

• remove clickableDisabledOptions
• instead of padding lets use gap from flex
• disabled still allows for selections
• when using "horizontal" we should put the title <h5> next to <ul> by adding styles to main wrapper: display: flex;align-items: center;

Swatches Component

• has a fixed width of 40px per slide, why?
• :before pseudo element for "disabled" strike should not have fixed width
• :before pseudo element doesn't work on white swatch
• disabled styling differs from grid / carousel - maybe unify?

VariantSelection Component

• storybook type selection doesn't work
• dropdown type needs position: relative; for disabled strike
• for disabled class, maybe use simpler 'line-through' for styling?

[korgon]

Could use some final review and testing.

[korgon]

Looks good to me.