Skip to content
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

Instant Search: add sort widget #13614

Merged
merged 10 commits into from
Oct 3, 2019

Conversation

bluefuton
Copy link
Contributor

Changes proposed in this Pull Request:

This enables sorting of search results from Jetpack Instant Search.

Is this a new feature or does it add/remove features to an existing part of Jetpack?

Yes, this adds a sorting UI to Jetpack Instant Search.

Testing instructions:

Add define( "JETPACK_SEARCH_PROTOTYPE", true ); to your wp-config.php.
Ensure that your site has the Jetpack Pro plan and has Jetpack Search enabled.
Add a Jetpack Search widget to the Search page sidebar.
Enter a query into a search widget. Alternatively, navigate to a search page like /?s=privacy.
Ensure that you can use the sort dropdown to change sort order.

Proposed changelog entry for your changes:

Not required.

@bluefuton bluefuton force-pushed the add/instant-search-sorting-ui branch from 4e5652b to 5541f7c Compare October 2, 2019 22:14
@bluefuton
Copy link
Contributor Author

bluefuton commented Oct 2, 2019

Todo:

  • add correct sort options
  • enable sorting based on current option

This PR does not use the user-specified default sort - we'll add in a followup PR.

@bluefuton bluefuton marked this pull request as ready for review October 3, 2019 01:56
@bluefuton bluefuton requested a review from a team as a code owner October 3, 2019 01:56
@bluefuton bluefuton added [Status] Needs Review To request a review from Crew. Label will be renamed soon. and removed [Status] In Progress labels Oct 3, 2019
Copy link
Member

@gibrown gibrown left a comment

Choose a reason for hiding this comment

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

WFM. Minor suggestion.

Co-Authored-By: Greg Ichneumon Brown <greg@automattic.com>
Copy link
Member

@jsnmoon jsnmoon left a comment

Choose a reason for hiding this comment

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

Looks good.

@bluefuton bluefuton merged commit 90716e6 into instant-search-master Oct 3, 2019
@bluefuton bluefuton deleted the add/instant-search-sorting-ui branch October 3, 2019 22:02
@matticbot matticbot removed the [Status] Needs Review To request a review from Crew. Label will be renamed soon. label Oct 3, 2019
jsnmoon added a commit that referenced this pull request Oct 23, 2019
* Implement minimal search results and spelling correction (#13365)
* Add filtering display (#13371)
* Fix search result display bugs and make improvements (#13393)
* Add rudimentary support for filtering on post types (#13430)
* Add support for filtering on categories and tags (#13505)
* Add instant search sorting based on the URL (#13377)
* Add support for filtering on dates (#13545)
* Add custom taxonomy filtering (#13605)
* add sort widget (#13614)
* fix many theme incompatibilities (#13602)
* Add infinite scrolling (#13684)
* Add caching to the api requests (#13714)
* Clean up some design bugs/issues (#13721)
* Fix labels for post types when we have them. (#13750)
* Add localization and formatting of all dates (#13748)
* search from any page on the site (#13713)
* Hook up default options (inc. sort) (#13742)
* Add TrainTracks analytics (#13730)
* Create PostTypeIcon component (#13790)
* Upgrade to Preact 10 (#13794)
* Add comments component (#13797)
* Address review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Search For all things related to Search [Type] Feature Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants