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

Revert "Add searchform partial and function to replace default WordPress functionality" #2110

Merged
merged 1 commit into from
Sep 7, 2018

Conversation

retlehs
Copy link
Member

@retlehs retlehs commented Sep 7, 2018

Reverts #2090

Closes #2105 #2109

@retlehs retlehs merged commit c9bb05a into master Sep 7, 2018
@retlehs retlehs deleted the revert-2090-master branch September 7, 2018 13:57
@garrettw
Copy link

garrettw commented Sep 7, 2018

So is there some other forthcoming fix for the broken search form?

@retlehs
Copy link
Member Author

retlehs commented Sep 7, 2018

this is the fix...

@garrettw
Copy link

garrettw commented Sep 7, 2018

What I mean is, #2090 was supposed to fix get_search_form() so if that's been reverted, what's the new solution?

@retlehs
Copy link
Member Author

retlehs commented Sep 7, 2018

not sure what you mean? #2090 doesn't fix get_search_form - it was a new feature that introduced some bugs (#2101 #2105). this reverts it.

@garrettw
Copy link

garrettw commented Sep 7, 2018

Meh, never mind I guess. Maybe I misunderstood what was meant by "letting users use get_search_form() as normal".

@mmirus
Copy link
Contributor

mmirus commented Sep 7, 2018

@garrettw been a bit, but if memory serves, the problem solved by #2090 was that if you wanted to use a custom template for the search form you needed to filter get_search_form to work with the Sage folder structure and Blade.

We may incorporate something similar to this again in a future Sage release, but in the meanwhile you can use the changes from #2090 and #2109 if you want to override the search form partial in your project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants