Merged
Conversation
- Replace placeholder text group with CC0 photo (why-nail-salon.webp, StockSnap.io/Freestocks.org) in the nail-salon-why-us pattern image column - Simplify image container group: remove explicit padding/blockGap, switch layout to default for natural fill - Add overlay-grid-wrap CSS rules: overflow:hidden for border-radius clipping, margin reset, and object-fit:cover for full-bleed rendering - Add image attribution to readme.txt; bump stable tag and theme version to 3.9.1
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.
This release upgrades the nail salon vertical's "Why Choose Us" pattern (version 3.9.1) by replacing a static placeholder with a real, CC0-licensed photograph sourced from StockSnap.io/Freestocks.org, depicting hands receiving a nail treatment. The image is served via a
wp:imageblock usingget_template_directory_uri()to comply with WP.org standards, with the container's layout type changed fromconstrainedtodefaultso the photo fills the column naturally. Accompanying CSS additions to theoverlay-grid-wrapcomponent ensureoverflow: hiddenclips the image to the group'sborder-radiuspreset, and amargin: 0reset eliminates unintended spacing inside the image container. All three version files (style.css,readme.txt,CHANGELOG.md) have been updated to 3.9.1, and image attribution has been added toreadme.txtper WP.org licensing requirements.Nail Salon Pattern Enhancement:
patterns/nail-salon-why-us.php: Replaced the placeholder group block (containing centered "Salon Photo" text with large padding) with a realwp:imageblock referencingwhy-nail-salon.webp, providing a production-ready two-column "Why Choose Us" layout.constrainedtodefault, and explicitpadding/blockGapattributes were removed, allowing the photo to fill the 45% column without artificial constraints.Asset Addition and CSS Fix:
patterns/images/nail-salon/why-nail-salon.webp: Added bundled CC0-licensed photo (StockSnap.io / Freestocks.org) — meets WP.org GPL-compatible licensing requirements and avoids any hardcoded external URL.overlay-grid-wrapCSS withoverflow: hiddenon the image-bearing first child and amargin: 0reset on the innerwp-block-image, ensuring proper border-radius clipping without layout bleed.Version and Attribution Updates:
style.cssandreadme.txtbumped to version 3.9.1;CHANGELOG.mdupdated with a new[3.9.1] - 2026-04-24entry documenting all additions and changes.why-nail-salon.webpadded toreadme.txtattributions section in compliance with WP.org submission guidelines.Files Changed:
CHANGELOG.md(Modified)patterns/nail-salon-why-us.php(Modified)readme.txt(Modified)style.css(Modified)patterns/images/nail-salon/why-nail-salon.webp(Added)