Skip to content

Improve product page#337

Merged
takumiando merged 2 commits intospacecubics:mainfrom
yashi:improve-product-page
Apr 2, 2026
Merged

Improve product page#337
takumiando merged 2 commits intospacecubics:mainfrom
yashi:improve-product-page

Conversation

@yashi
Copy link
Copy Markdown
Member

@yashi yashi commented Apr 2, 2026

see commit logs.

Remove the unused `bg` argument from `product_display()`.

The shortcode no longer uses `bg` to set a background image,
so drop it from both the template and its call sites.

Signed-off-by: Yasushi SHOJI <yashi@spacecubics.com>
@yashi yashi requested a review from takumiando April 2, 2026 09:05
Add the product page link (`details_link`) to the image so visitors
can click the product image to open the product page. This improves
usability because, in the previous version, only the arrow in the
orange circle was linked.

`img_link` is no longer referenced in the template, so remove it as
well.

Signed-off-by: Yasushi SHOJI <yashi@spacecubics.com>
@yashi yashi force-pushed the improve-product-page branch from 49f061f to 30d892b Compare April 2, 2026 09:40
@takumiando takumiando merged commit 8a6a058 into spacecubics:main Apr 2, 2026
6 checks passed
@yashi
Copy link
Copy Markdown
Member Author

yashi commented Apr 2, 2026

hmm... I was expecting no visual changes... oops I didn't submit this comment... There was a visual diff and I was fixing....

@yashi
Copy link
Copy Markdown
Member Author

yashi commented Apr 2, 2026

The commit log is half baked. I should add:

Added display: block; for img under .product-image-container to
reduce visual change. Because both <a> and <img> are inline
elements, adding <a> chagnes the base-line.

@yashi yashi deleted the improve-product-page branch April 2, 2026 09:48
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.

2 participants