-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add possibility to filter by discounts, on sale and new products #880
Conversation
6218850
to
263ed3a
Compare
d961aa5
to
263ed3a
Compare
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.
Only one question.
I kinda like "Highlights", but a few alternatives I see:
- Products types (although it's confusing with product types like pack, combination, etc.)
- Products selections
- Shopping choices
- Specific products
ps_facetedsearch.php
Outdated
|
||
if (!isset($doneCategories[(int) $idCategory]['q'])) { | ||
$filterData['layered_selection_highlights'] = ['filter_type' => Converter::WIDGET_TYPE_CHECKBOX, 'filter_show_limit' => 0]; | ||
$doneCategories[(int) $idCategory]['e'] = true; |
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.
e
like "extra properties"? 🤔
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.
Yep, forgotten
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.
Small request: We should separate the new and the discounts part 🙏
@MatShir what do you mean? it is about showing products with different state, on discount, on sale, new products - how would you like to split this? |
Hello @Hlavtox :-) As @MatShir said, it would be nice to have two separate parts for the two "types" of labels so it's not too confusing. By that, I mean that So, considering this, here's what we propose: Instead of the The Deals section is dedicated to The Selection section contains a selection of specific products. In this case, the So it would look like this: Deals:
Selection:
What do you think? 🏎 |
2e9c1fe
to
457d2f0
Compare
8a87f43
to
14190cb
Compare
6f56bf0
to
946ca0e
Compare
@MatShir @kpodemski It's ready for review.
|
Looks good to me ! |
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.
Hello @Hlavtox ,
Tested on 8.0.x
, 8.1.x
and develop
branches.
Upgrade to 3.14.0 OK ✅
New filter Selections works fine ^^
There's just this option that I do not have :
When installing this module, I moved the stock, price and selections filter to the top. As they are the most important.
Could you check ? ^^
Hi @florine2623, it works only on clean installs when there never was faceted search. For existing installs, there will be no change. |
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.
@Hlavtox ,
Indeed after Uninstalling the module and reinstalling with the PR, the 3 elements are on top the the filter list ^^
QA ✅
Thanks!
well done 👏🏻 thanks @Hlavtox |
Even after a module upgrade ? what needs to be done for this part to be on top ? @Hlavtox |
Description
Screenshot
This change is