Claude/wizardly hopper dd235d - #76
Open
lgcavalini wants to merge 11 commits into
Open
Conversation
Adds an image_mobile field to the _slide block schema so merchants can upload a separate image for mobile devices. Uses a <picture> element with a <source media="(max-width: 749px)"> to serve the mobile image on small screens. Falls back to the desktop image when no mobile image is set, preserving all existing slide behaviour. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed from shop: Mono Produto
Committed from shop: Mono Produto
Bug: the <picture> element introduced for mobile images broke the full-screen layout because the CSS used direct-child selectors (>) that no longer matched the <img> nested inside <picture>. Fixed by adding explicit rules for picture and picture > .slide__image inside .slide__image-container. Feature: the adapt_image height mode (already wired in block logic) is now exposed as a schema option and set as the default. The min-height calculation is split by breakpoint so desktop uses the desktop image aspect ratio and mobile uses the mobile image ratio (falling back to desktop if no mobile image is set). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed from shop: Mono Produto
When slide_height is 'auto' and the slide has an image but no text/buttons, the image is position:absolute so it doesn't contribute to the content height and the slide collapses to a minimal bar. Fix: apply the image-based min-height calculation (split by breakpoint for desktop/mobile images) whenever slide_height is 'auto' and an image exists. Removes the separate 'adapt_image' schema option — 'auto' now covers both content-driven and image-driven height in one mode. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed from shop: Mono Produto
Committed from shop: Mono Produto
Responsive height: move {% style %} outside {% capture %} so the tag is
processed at the block render level (reliable). Replace min-height vw
with aspect-ratio W/H — the browser maintains the correct proportions
as viewport width changes, per desktop and mobile breakpoints
independently.
Clickable slides: add a `link` URL field to the _slide block schema.
Renders an absolute-positioned transparent <a> inside the image container
(inset: 0, z-index: 1). Uses CSS :has(.slide__link) to set
pointer-events: none on .slide__content so clicks pass through to the
link, while <a> and <button> children restore pointer-events: auto.
Fade transition: add `transition_style` select (fade/slide, default fade)
to the slideshow section schema. CSS sets overflow: hidden on
slideshow-slides (hides the scroll without breaking the JS's
IntersectionObserver which uses root: scroller), then animates opacity
via aria-hidden — the attribute the JS already toggles — producing a
cross-fade without any new JavaScript.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Committed from shop: Mono Produto
|
🔎 View this PR in Shipit Next. ℹ️ Expand to learn how to deploy and handle emergencies using Shipit NextOverviewShipit Next will merge your code on your behalf because this repository uses Shipit Next and its merge queue. To ship this PR, you can either:
Comment Commands
Commands exclusive to Deploy Before Merge
Documentation
Questions or feedback?
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.