Fix: Attribute Filters: allow choosing between single/multiple selection #6620
Conversation
Requesting one more review from @Aljullu because this PR is based on his suggestion. |
Size Change: +111 B (0%) Total Size: 875 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I tested it with the All Products block and PHP templates and everything seems to be working fine.
Not introduced in this PR, but one suggestion: when the selection mode is Single
and display style List
, could we replace the checkboxes with radio inputs? I think it's better for accessibility and also a better UI. (No need to do it as part of this PR, it can be done as a follow-up)
@Aljullu I thought about this too and experimented with the |
Ah, good point. I didn't think about this. LGTM |
Fixes #1311
This PR:
AND
.single
.Save
component to avoid block mismatch issues for existing Attribute Filter blocks.Accessibility
prefers-reduced-motion
Other Checks
Screenshots
Testing
Automated Tests
User Facing Testing
Attempt block recovery
button for that block.Allow selecting multiple options?
set to Multiple by default.Single
. See theQuery Type
setting disappear.Multiple
. Set display style toDropdown
and query type toAND
.WooCommerce Visibility
Performance Impact
N/A
Changelog