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

Theme Search: Make theme search more flexible to provide more results #4680

Open
rachelmcr opened this issue Jan 14, 2016 · 3 comments
Open

Comments

@rachelmcr
Copy link
Contributor

The theme search is very restrictive, which may cause problems for mobile users. Comment from Adam during beta testing:

Searching for a theme name with a space after it returns no results. Compare a search for:
“wedding ”
… with a search for:
“wedding”

Seems like we should work around this since some mobile keyboards automatically put spaces after words when autocompleted or whatever.

It's also worth noting that a search for "wedding" in Calypso on a desktop returns many other themes (such as wedding-topic themes like Forever) that aren't returned in the app's search results. It would be nice to include those kinds of results as well.

@koke
Copy link
Member

koke commented Mar 13, 2017

This is potentially "Low Hanging Fruit". The API for themes includes a tags array which we can store locally and use for search. I tried doing this a couple weeks ago, but couldn't figure out a Core Data crash when using a predicate to search in the tags (any tags contains[c] %@).

@koke koke assigned koke and unassigned koke Mar 15, 2017
@koke
Copy link
Member

koke commented Mar 15, 2017

I figured out the crash: Theme.tags isn't really an array, since there's no such thing in Core Data. It's a "Transformable" attribute and we can't use that for the search. Some possible solutions:

  • Make a ThemeTag entity and use that. That'd allow local search but add some overhead on sync to create all the new entities.
  • Forget about local, and make search always hit the API for results.

@bummytime bummytime self-assigned this Mar 20, 2017
@bummytime bummytime removed their assignment Jul 6, 2017
@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

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

No branches or pull requests

8 participants