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

Update template CSS variables #1091

Merged
merged 42 commits into from
Jan 12, 2022
Merged

Update template CSS variables #1091

merged 42 commits into from
Jan 12, 2022

Conversation

KaichenWang
Copy link
Contributor

Why are these changes introduced?

A few CSS variables were added/modified in theme.liquid in order to add global settings. password.liquid and gift_card.liquid must be updated to match the CSS variables in theme.liquid.

What approach did you take?

  • password.liquid: Copy all CSS variables from theme.liquid
  • gift_card.liquid: Update existing CSS variables to match those in theme.liquid (not all theme CSS variables are needed on the gift card template)

Other considerations

Demo links

Checklist

Base automatically changed from all-global-settings to main January 5, 2022 19:55
martinamarien and others added 29 commits January 5, 2022 14:56
* adjust featured blog grid

* ajust collage grid gap

* adjust featured collection grid

* update blog page grid

* add settings, css variables and update grid class

* Fix multicolumn vertical spacing

* address reviewer feedback
* Text boxes global settings setup
* Add section-wrapper/borders to applicable sections

* Add section spacing

* Remove default border thickness.

* Fix collage spacing
* Add settings to schema

* Edit template

* Add styles

* Use decimal for border-radius

* Refactor markup

* Adjust styles for product card

* Add styles

* Add settings

* Update setting and CSS variable names

* Update cards for search results

* Clean up CSS

* Update product card placeholder

* Fix visual bugs. Adjust shadow setting min

* Update markup for collection list cards

* Update hover styles

* Adjust collection arrow

* Adjust collection arrow

* Rename snippet for product card. Create snippet for collection card

* Remove separate placeholder for product card

* Adjustments for image ratio

* Add support for auto height for collage

* Clean up CSS

* Fix typo

* Fix pick up availability 'Unavailable' (#855)

* Fix pick up availability 'Unavailable'

* reuse similar check used in the same file

* Remove redundant section settings

* Update 1 translation file (#866)

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>

* Product image seo (#812)

* Version bump to 2.4.0 (#884)

* Add translation strings for en

* Adjust styles

* Update snippets/card-product.liquid

* Adjust settings organization

* Update layout/theme.liquid

Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* Remove redundant padding

* Address feedback - Clean up unused CSS

* Fix interaction with border and background

* Fix collage

* Adjust price layout

* Adjust price layout

Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>
* Move vert spacing and remove other section settings

* Include collage
* Enable textbox settings for image with text section. Clean up CSS

* Update from Shopify for theme dawn/apply-settings-image-with-text

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/apply-settings-image-with-text"

This reverts commit 7dea1d2.

* Add back bottom margin

* Remove redundant media query

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
* Split popup and drawer settings

* Align naming
* Update setting ranges and options for Buttons, Inputs, Layout, Media, Content containers

* Fix typo

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Reorder settings. Adjust mobile grid range and defaults

* Update settings naming

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>
* Fix rich text margins for bg-1

* Remove section level spacing CSS

* Update and clean up spacing naming conventions

* Clean up

* Fix spacing for special consecutive sections with no heading

* Remove comment

* Update collapse grid naming

* Update behaviour of special collapsing sections

* Fix bugs

* Remove product-grip bottom margin

* Adjust setting range. Set image-with-text as collapsible section

* Update from Shopify for theme dawn/fix-section-vertical-spacing

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/fix-section-vertical-spacing"

This reverts commit a0771e5.

* Adapt consistent spacing for all sections. Remove special cases

* Revert usage of section spacing for elements other than sections

* Edit settings names and order for spacing. Add mobile vertical section spacing setting

* Revert value for margin-bottom

* Revert additional padding

* Revert additional padding

* Revert heading logic

* Revert heading logic

* Revert heading logic

* Address feedback

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>
* Add and update settings and copy

* add css for variant pill settings
…onflicts

* Collapse borders/corners and prevent shadow overlap

* Revamp approach

* Address media-only borders case

* Adjust conditional

* Prevent transparent borders from occupying physical space

* Revert border width change
* Apply shadow filter and background-none alternative

* Prevent border radius on background-none

* Update background class conditional

* Move shadows to pseudo element

* Whitespace

Co-authored-by: Ludo <ludo.segura@shopify.com>

Co-authored-by: Ludo <ludo.segura@shopify.com>
* Section paddings settings
* Update article cards to use card structure
* Add theme level grid setting (#885)

* adjust featured blog grid

* ajust collage grid gap

* adjust featured collection grid

* update blog page grid

* add settings, css variables and update grid class

* Fix multicolumn vertical spacing

* address reviewer feedback

* Settings popup and drawer (#865)

* Add media settings to non-card non-textbox elements (#893)

* Text boxes global settings setup (#889)

* Text boxes global settings setup

* Add global sections settings (#891)

* Add section-wrapper/borders to applicable sections

* Add section spacing

* Remove default border thickness.

* Fix collage spacing

* Global settings - buttons, inputs (#844)

Buttons + inputs

* Added back section settings tmeporarily (#899)

* Add global settings cards (#886)

* Add settings to schema

* Edit template

* Add styles

* Use decimal for border-radius

* Refactor markup

* Adjust styles for product card

* Add styles

* Add settings

* Update setting and CSS variable names

* Update cards for search results

* Clean up CSS

* Update product card placeholder

* Fix visual bugs. Adjust shadow setting min

* Update markup for collection list cards

* Update hover styles

* Adjust collection arrow

* Adjust collection arrow

* Rename snippet for product card. Create snippet for collection card

* Remove separate placeholder for product card

* Adjustments for image ratio

* Add support for auto height for collage

* Clean up CSS

* Fix typo

* Fix pick up availability 'Unavailable' (#855)

* Fix pick up availability 'Unavailable'

* reuse similar check used in the same file

* Remove redundant section settings

* Update 1 translation file (#866)

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>

* Product image seo (#812)

* Version bump to 2.4.0 (#884)

* Add translation strings for en

* Adjust styles

* Update snippets/card-product.liquid

* Adjust settings organization

* Update layout/theme.liquid

Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* Remove redundant padding

* Address feedback - Clean up unused CSS

* Fix interaction with border and background

* Fix collage

* Adjust price layout

* Adjust price layout

Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* update secondary button background color (#903)

* Clean up popup drawer feed (#902)

* Revert global section settings (#911)

* Move vert spacing and remove other section settings

* Include collage

* Remove text box padding setting (#912)

* Image with text section - Apply global textbox and media settings (#918)

* Enable textbox settings for image with text section. Clean up CSS

* Update from Shopify for theme dawn/apply-settings-image-with-text

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/apply-settings-image-with-text"

This reverts commit 7dea1d2.

* Add back bottom margin

* Remove redundant media query

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>

* Split popup and drawer settings (#917)

* Split popup and drawer settings

* Align naming

* Update global setting range values (#954)

* Update setting ranges and options for Buttons, Inputs, Layout, Media, Content containers

* Fix typo

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Reorder settings. Adjust mobile grid range and defaults

* Update settings naming

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Fix section vertical spacing (#976)

* Fix rich text margins for bg-1

* Remove section level spacing CSS

* Update and clean up spacing naming conventions

* Clean up

* Fix spacing for special consecutive sections with no heading

* Remove comment

* Update collapse grid naming

* Update behaviour of special collapsing sections

* Fix bugs

* Remove product-grip bottom margin

* Adjust setting range. Set image-with-text as collapsible section

* Update from Shopify for theme dawn/fix-section-vertical-spacing

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/fix-section-vertical-spacing"

This reverts commit a0771e5.

* Adapt consistent spacing for all sections. Remove special cases

* Revert usage of section spacing for elements other than sections

* Edit settings names and order for spacing. Add mobile vertical section spacing setting

* Revert value for margin-bottom

* Revert additional padding

* Revert additional padding

* Revert heading logic

* Revert heading logic

* Revert heading logic

* Address feedback

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Refactor collage

* Update BEM for video modal

* Adjust image spacing for mobile

* Clean up

* Set grip gap to global setting values

* Fix collage focus states

* Revert block order in liquid

* Revert block order in liquid

* Update file name

* Add special case for single product/collection card

* Remove ratios

* Remove single video case

* Update auto height setting for product and collection cards

* Move card shadow to pseudo element (#1012)

* Move product/card media shadows to pseudo element (#1013)

* Add variant pills settings (#1005)

* Add and update settings and copy

* add css for variant pill settings

* Image with text border/corner/shadow updates (#1003)

* Collapse borders/corners and prevent shadow overlap

* Revamp approach

* Address media-only borders case

* Adjust conditional

* Prevent transparent borders from occupying physical space

* Revert border width change

* Clean up

* Adjust implementation for card height extend

* Adjust implementation for card height extend

* Fix focus and clickable area for video which includes padding

* Address PR comment. Fix 2 item mobile collage

* Add position relative to collage-card for shadows

* Update mobile collage layout

* Use minmax to prevent grid expansion beyond viewport

Co-authored-by: Martina Marien <martina.marien@shopify.com>
Co-authored-by: Sofia Matulis <sofiamatulis@users.noreply.github.com>
Co-authored-by: Ken Meleta <30790058+kmeleta@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>
Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>
* update the order of high level setting

* remove style icons header

* fix my mistake
* update the order of high level setting

* update range values

* adjust buttons values

* update min value

* update range values

* adjust buttons values

* update min value
kmeleta and others added 10 commits January 5, 2022 14:57
* Move pill shadow to pseudo element

* Fix radius value
* update the naming and clean up

* adjust behavior

* re-add decorative elements

* fix code

* remove unsused translations and add pop-ups paragraph

* rebase 1

* rebase 2

* fix code

* remove s to border

* fix feedback
* testing values

* adjust buttons and blur
* Move button styling to pseudo elements

* Fix shadow depth in affected sections

* Add border width/opacity condition

* Fix branded checkout button heights

* Fix border radius/border width calculations

* Ignore facets

* Cart and customer button fixes

* Fix tertiary dimensions and opacity
@KaichenWang KaichenWang force-pushed the update-template-css-variables branch from 5354fed to f46b337 Compare January 5, 2022 19:57
…mplate-css-variables

# Conflicts:
#	layout/password.liquid
#	templates/gift_card.liquid
tyleralsbury
tyleralsbury previously approved these changes Jan 10, 2022
Copy link
Contributor

@tyleralsbury tyleralsbury left a comment

Choose a reason for hiding this comment

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

Looks good - I tested by comparing the variables between the theme and password pages and it looks like they all match. The gift card one has fewer, as mentioned in the description, so I just cross-referenced it with the classes being used on that page and it looks good.

@sofiamatulis sofiamatulis self-assigned this Jan 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.

Looks great 🎉 Just added a few small comments

--popup-shadow-vertical-offset: {{ settings.popup_shadow_vertical_offset }}px;
--popup-shadow-blur-radius: {{ settings.popup_shadow_blur }}px;

--drawer-border-width: {{ settings.drawer_border_thickness }}px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Drawers don't exist in the password page since they are only for the filter, cart, pick up availability and menu (password page has a different menu). Do we need it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point! I'm ok with removing --drawer- variables from password page. One argument for keeping them is to ensure the same CSS variables are available across then theme to be used in the Custom Liquid section, but that is a very limited use-case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed 91e1420. We can always add them back if needed.


--badge-corner-radius: {{ settings.badge_corner_radius | divided_by: 10.0 }}rem;

--popup-border-width: {{ settings.popup_border_thickness }}px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Same with popup. Do we need it? It seems to be only used in the cart, search, filter, footer, product page and header which dont exist in the password page

Copy link
Contributor Author

@KaichenWang KaichenWang Jan 10, 2022

Choose a reason for hiding this comment

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

The featured product section add-to-cart should in theory trigger a cart notification, which is a popup, but that functionality is currently not working on password page. Opening an issue #1144

Copy link
Contributor

@sofiamatulis sofiamatulis Jan 10, 2022

Choose a reason for hiding this comment

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

I believe the reason it doesn't work is because technically if a site is password protected, it means we cant access the other pages. So even though we allow merchants to add any section to the password page, it's not encouraged to add the featured product (?). So I am not sure if I would consider that an issue

cc @melissaperreault @Oliviammarcello

Copy link
Contributor

Choose a reason for hiding this comment

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

Hum, @nicklepine might have more context on that one, I don't remember. 🤔

Choose a reason for hiding this comment

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

@sofiamatulis given the site is behind a password, adding to cart from a featured product section should not work on the password page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the -popup variables from password page f04b9cd

--grid-desktop-horizontal-spacing: {{ settings.spacing_grid_horizontal }}px;
--grid-mobile-vertical-spacing: {{ settings.spacing_grid_vertical | divided_by: 2 }}px;
--grid-mobile-horizontal-spacing: {{ settings.spacing_grid_horizontal | divided_by: 2 }}px;
--variant-pills-radius: {{ settings.variant_pills_radius }}px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Same with variants. They seem to only exist in the product page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe variant pills can exist in a featured-product section on the password page as well

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.

Thanks for the changes 🚢

@KaichenWang KaichenWang merged commit 61970ca into main Jan 12, 2022
@KaichenWang KaichenWang deleted the update-template-css-variables branch January 12, 2022 18:10
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* Add theme level grid setting (Shopify#885)

* adjust featured blog grid

* ajust collage grid gap

* adjust featured collection grid

* update blog page grid

* add settings, css variables and update grid class

* Fix multicolumn vertical spacing

* address reviewer feedback

* Settings popup and drawer (Shopify#865)

* Add media settings to non-card non-textbox elements (Shopify#893)

* Text boxes global settings setup (Shopify#889)

* Text boxes global settings setup

* Add global sections settings (Shopify#891)

* Add section-wrapper/borders to applicable sections

* Add section spacing

* Remove default border thickness.

* Fix collage spacing

* Global settings - buttons, inputs (Shopify#844)

Buttons + inputs

* Added back section settings tmeporarily (Shopify#899)

* Add global settings cards (Shopify#886)

* Add settings to schema

* Edit template

* Add styles

* Use decimal for border-radius

* Refactor markup

* Adjust styles for product card

* Add styles

* Add settings

* Update setting and CSS variable names

* Update cards for search results

* Clean up CSS

* Update product card placeholder

* Fix visual bugs. Adjust shadow setting min

* Update markup for collection list cards

* Update hover styles

* Adjust collection arrow

* Adjust collection arrow

* Rename snippet for product card. Create snippet for collection card

* Remove separate placeholder for product card

* Adjustments for image ratio

* Add support for auto height for collage

* Clean up CSS

* Fix typo

* Fix pick up availability 'Unavailable' (Shopify#855)

* Fix pick up availability 'Unavailable'

* reuse similar check used in the same file

* Remove redundant section settings

* Update 1 translation file (Shopify#866)

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>

* Product image seo (Shopify#812)

* Version bump to 2.4.0 (Shopify#884)

* Add translation strings for en

* Adjust styles

* Update snippets/card-product.liquid

* Adjust settings organization

* Update layout/theme.liquid

Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* Remove redundant padding

* Address feedback - Clean up unused CSS

* Fix interaction with border and background

* Fix collage

* Adjust price layout

* Adjust price layout

Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* update secondary button background color (Shopify#903)

* Clean up popup drawer feed (Shopify#902)

* Revert global section settings (Shopify#911)

* Move vert spacing and remove other section settings

* Include collage

* Remove text box padding setting (Shopify#912)

* Image with text section - Apply global textbox and media settings (Shopify#918)

* Enable textbox settings for image with text section. Clean up CSS

* Update from Shopify for theme dawn/apply-settings-image-with-text

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/apply-settings-image-with-text"

This reverts commit 7dea1d2.

* Add back bottom margin

* Remove redundant media query

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>

* Split popup and drawer settings (Shopify#917)

* Split popup and drawer settings

* Align naming

* Update global setting range values (Shopify#954)

* Update setting ranges and options for Buttons, Inputs, Layout, Media, Content containers

* Fix typo

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Reorder settings. Adjust mobile grid range and defaults

* Update settings naming

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Fix section vertical spacing (Shopify#976)

* Fix rich text margins for bg-1

* Remove section level spacing CSS

* Update and clean up spacing naming conventions

* Clean up

* Fix spacing for special consecutive sections with no heading

* Remove comment

* Update collapse grid naming

* Update behaviour of special collapsing sections

* Fix bugs

* Remove product-grip bottom margin

* Adjust setting range. Set image-with-text as collapsible section

* Update from Shopify for theme dawn/fix-section-vertical-spacing

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/fix-section-vertical-spacing"

This reverts commit a0771e5.

* Adapt consistent spacing for all sections. Remove special cases

* Revert usage of section spacing for elements other than sections

* Edit settings names and order for spacing. Add mobile vertical section spacing setting

* Revert value for margin-bottom

* Revert additional padding

* Revert additional padding

* Revert heading logic

* Revert heading logic

* Revert heading logic

* Address feedback

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Move card shadow to pseudo element (Shopify#1012)

* Move product/card media shadows to pseudo element (Shopify#1013)

* Add variant pills settings (Shopify#1005)

* Add and update settings and copy

* add css for variant pill settings

* Image with text border/corner/shadow updates (Shopify#1003) + rebase merge conflicts

* Collapse borders/corners and prevent shadow overlap

* Revamp approach

* Address media-only borders case

* Adjust conditional

* Prevent transparent borders from occupying physical space

* Revert border width change

* Multicolumn shadow updates (Shopify#1023)

* Apply shadow filter and background-none alternative

* Prevent border radius on background-none

* Update background class conditional

* Move shadows to pseudo element

* Whitespace

Co-authored-by: Ludo <ludo.segura@shopify.com>

Co-authored-by: Ludo <ludo.segura@shopify.com>

* fix rebase issues

* Section padding settings (Shopify#1006)

* Section paddings settings

* Blog cards update (Shopify#1036)

* Update article cards to use card structure

* Fix collage section product and collection card image size (Shopify#1014)

* Add theme level grid setting (Shopify#885)

* adjust featured blog grid

* ajust collage grid gap

* adjust featured collection grid

* update blog page grid

* add settings, css variables and update grid class

* Fix multicolumn vertical spacing

* address reviewer feedback

* Settings popup and drawer (Shopify#865)

* Add media settings to non-card non-textbox elements (Shopify#893)

* Text boxes global settings setup (Shopify#889)

* Text boxes global settings setup

* Add global sections settings (Shopify#891)

* Add section-wrapper/borders to applicable sections

* Add section spacing

* Remove default border thickness.

* Fix collage spacing

* Global settings - buttons, inputs (Shopify#844)

Buttons + inputs

* Added back section settings tmeporarily (Shopify#899)

* Add global settings cards (Shopify#886)

* Add settings to schema

* Edit template

* Add styles

* Use decimal for border-radius

* Refactor markup

* Adjust styles for product card

* Add styles

* Add settings

* Update setting and CSS variable names

* Update cards for search results

* Clean up CSS

* Update product card placeholder

* Fix visual bugs. Adjust shadow setting min

* Update markup for collection list cards

* Update hover styles

* Adjust collection arrow

* Adjust collection arrow

* Rename snippet for product card. Create snippet for collection card

* Remove separate placeholder for product card

* Adjustments for image ratio

* Add support for auto height for collage

* Clean up CSS

* Fix typo

* Fix pick up availability 'Unavailable' (Shopify#855)

* Fix pick up availability 'Unavailable'

* reuse similar check used in the same file

* Remove redundant section settings

* Update 1 translation file (Shopify#866)

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>

* Product image seo (Shopify#812)

* Version bump to 2.4.0 (Shopify#884)

* Add translation strings for en

* Adjust styles

* Update snippets/card-product.liquid

* Adjust settings organization

* Update layout/theme.liquid

Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* Remove redundant padding

* Address feedback - Clean up unused CSS

* Fix interaction with border and background

* Fix collage

* Adjust price layout

* Adjust price layout

Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>

* update secondary button background color (Shopify#903)

* Clean up popup drawer feed (Shopify#902)

* Revert global section settings (Shopify#911)

* Move vert spacing and remove other section settings

* Include collage

* Remove text box padding setting (Shopify#912)

* Image with text section - Apply global textbox and media settings (Shopify#918)

* Enable textbox settings for image with text section. Clean up CSS

* Update from Shopify for theme dawn/apply-settings-image-with-text

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/apply-settings-image-with-text"

This reverts commit 7dea1d2.

* Add back bottom margin

* Remove redundant media query

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>

* Split popup and drawer settings (Shopify#917)

* Split popup and drawer settings

* Align naming

* Update global setting range values (Shopify#954)

* Update setting ranges and options for Buttons, Inputs, Layout, Media, Content containers

* Fix typo

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Reorder settings. Adjust mobile grid range and defaults

* Update settings naming

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Fix section vertical spacing (Shopify#976)

* Fix rich text margins for bg-1

* Remove section level spacing CSS

* Update and clean up spacing naming conventions

* Clean up

* Fix spacing for special consecutive sections with no heading

* Remove comment

* Update collapse grid naming

* Update behaviour of special collapsing sections

* Fix bugs

* Remove product-grip bottom margin

* Adjust setting range. Set image-with-text as collapsible section

* Update from Shopify for theme dawn/fix-section-vertical-spacing

Committed from shop: OS 2.0

* Revert "Update from Shopify for theme dawn/fix-section-vertical-spacing"

This reverts commit a0771e5.

* Adapt consistent spacing for all sections. Remove special cases

* Revert usage of section spacing for elements other than sections

* Edit settings names and order for spacing. Add mobile vertical section spacing setting

* Revert value for margin-bottom

* Revert additional padding

* Revert additional padding

* Revert heading logic

* Revert heading logic

* Revert heading logic

* Address feedback

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Update config/settings_schema.json

Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Refactor collage

* Update BEM for video modal

* Adjust image spacing for mobile

* Clean up

* Set grip gap to global setting values

* Fix collage focus states

* Revert block order in liquid

* Revert block order in liquid

* Update file name

* Add special case for single product/collection card

* Remove ratios

* Remove single video case

* Update auto height setting for product and collection cards

* Move card shadow to pseudo element (Shopify#1012)

* Move product/card media shadows to pseudo element (Shopify#1013)

* Add variant pills settings (Shopify#1005)

* Add and update settings and copy

* add css for variant pill settings

* Image with text border/corner/shadow updates (Shopify#1003)

* Collapse borders/corners and prevent shadow overlap

* Revamp approach

* Address media-only borders case

* Adjust conditional

* Prevent transparent borders from occupying physical space

* Revert border width change

* Clean up

* Adjust implementation for card height extend

* Adjust implementation for card height extend

* Fix focus and clickable area for video which includes padding

* Address PR comment. Fix 2 item mobile collage

* Add position relative to collage-card for shadows

* Update mobile collage layout

* Use minmax to prevent grid expansion beyond viewport

Co-authored-by: Martina Marien <martina.marien@shopify.com>
Co-authored-by: Sofia Matulis <sofiamatulis@users.noreply.github.com>
Co-authored-by: Ken Meleta <30790058+kmeleta@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>
Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>

* Slider spacing fixes (Shopify#1037)

* Update the high level order of the global settings (Shopify#1029)

* update the order of high level setting

* remove style icons header

* fix my mistake

* add border radius to dynamic checkout buttons (Shopify#1024)

* Update global settings range values (Shopify#1042)

* update the order of high level setting

* update range values

* adjust buttons values

* update min value

* update range values

* adjust buttons values

* update min value

* Pill shadow updates (Shopify#1052)

* Move pill shadow to pseudo element

* Fix radius value

* Settings content naming update (Shopify#1044)

* update the naming and clean up

* adjust behavior

* re-add decorative elements

* fix code

* remove unsused translations and add pop-ups paragraph

* rebase 1

* rebase 2

* fix code

* remove s to border

* fix feedback

* Update layout range and values for mobile (Shopify#1055)

* Add content container to sections and update styling (Shopify#1050)

* Revise range values (Shopify#1069)

* testing values

* adjust buttons and blur

* Refactor button styles (Shopify#1064)

* Move button styling to pseudo elements

* Fix shadow depth in affected sections

* Add border width/opacity condition

* Fix branded checkout button heights

* Fix border radius/border width calculations

* Ignore facets

* Cart and customer button fixes

* Fix tertiary dimensions and opacity

* Fix content container shadow when set to transparent (Shopify#1075)

* New section - Collapsible content (Shopify#956)

* Update 13 translation files

* Update template CSS variables for password and gift card

* Remove drawer variables from password page

* Remove popup variables from password page

Co-authored-by: Martina Marien <martina.marien@shopify.com>
Co-authored-by: Sofia Matulis <sofiamatulis@users.noreply.github.com>
Co-authored-by: Ken Meleta <30790058+kmeleta@users.noreply.github.com>
Co-authored-by: Tyler Alsbury <60230011+tyleralsbury@users.noreply.github.com>
Co-authored-by: Ludo <ludo.segura@shopify.com>
Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
Co-authored-by: shopify-online-store[bot] <79544226+shopify-online-store[bot]@users.noreply.github.com>
Co-authored-by: melissaperreault <melissa.perreault@shopify.com>
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.

8 participants