Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
A couple of things need to happen on the price slider:
1) The price (number) field needs to not be user entry, but just displayed above or below the slider
2) The go button goes away (the field updates via ajax anyway)
3) Ideally we have an ajax loader indicator, because it take awhile for the search results to update
You can find examples of this type of price slider design on kayak.com or airbnb.com. Here's the one from airbnb:
Comment | File | Size | Author |
---|---|---|---|
#8 | Capture d’écran 2012-08-06 à 11.39.07.png | 5.71 KB | GuGuss |
#8 | Capture d’écran 2012-08-06 à 11.39.21.png | 7.47 KB | GuGuss |
#1 | Capture d’écran 2012-07-23 à 16.01.13.png | 5.36 KB | Cookiz |
Screen Shot 2012-07-20 at 2.33.18 PM.png | 14.2 KB | aleighs |
Comments
Comment #1
Cookiz CreditAttribution: Cookiz commentedNeeds review : https://code.drupalcommerce.org/199
Done in this commit :
1) The price (number) field needs to not be user entry, but just displayed above or below the slider
2) The go button goes away (the field updates via ajax anyway)
Concerning :
3) Ideally we have an ajax loader indicator, because it take awhile for the search results to update
If we need to add a loader for the price range, do we need to add it also on the other faceted search blocks ?
Comment #2
bojanz CreditAttribution: bojanz commentedMerged the commit.
#3 needs feedback.
Comment #3
GuGuss CreditAttribution: GuGuss commented3) I agree with Cookiz to have an ajax loader when you select any facet.
Comment #4
GuGuss CreditAttribution: GuGuss commentedComment #5
vasikefor 3) what about Search API ajax module.
is there anyone who use it for a Drupal commerce system?
Comment #6
jsacksick CreditAttribution: jsacksick commentedI already used it, a lot of progress has been made but there are still a few bugs, let's say it's 80 - 90% working if you're using drupal blocks and views/search_api_pages for your search pages which I think is the case in kickstart.
Also the code is not super clean...
Comment #7
bojanz CreditAttribution: bojanz commentedLet's skip that module for now then.
Are we done with this issue then?
Comment #8
GuGuss CreditAttribution: GuGuss commentedThis looks good and behaves fine.
Comment #10
romainT CreditAttribution: romainT commentedRight now both Small and Normal screen layouts are present at the same time, making it very confusing, and the "Go" button is present.