Skip to content

Shop: Add alphabetical sorting options to product listing - #43

Merged
JKinnez merged 1 commit into
mainfrom
feature/sc-3044-shop-update-sorting-options
Sep 14, 2026
Merged

JKinnez merged 1 commit into
mainfrom
feature/sc-3044-shop-update-sorting-options

Conversation

@JKinnez

@JKinnez JKinnez commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Added two new sorting options to the product listing sort selector: alphabetical ascending (A→Z) and descending (Z→A). These use the name and -name values respectively, aligning with the existing price sorting pattern.

Part of Shop: Update sorting options

Copilot AI lite review requested due to automatic review settings September 9, 2026 09:33
@JKinnez JKinnez added review Please look at my code shop-team Shop and commerce labels Sep 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change is small and consistent with the existing sort option/value pattern, and it cleanly extends the selector without altering other behavior.

Pull request overview

Adds alphabetical sorting choices to the shop product listing sort dropdown, expanding the existing sort selector pattern (default/price/newest) with name-based options.

Changes:

  • Added “Alphabetical (A→Z)” sort option using sort_by=name.
  • Added “Alphabetical (Z→A)” sort option using sort_by=-name.
File summaries
File Description
snippets/sorting-select.liquid Adds two new <option> entries for name-based (ascending/descending) sorting in the sort selector.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@JKinnez
JKinnez merged commit 689e1a1 into main Sep 14, 2026
1 check passed
@dabrorius dabrorius removed the review Please look at my code label Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

shop-team Shop and commerce

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants