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

Composer Search Field Simple > Advanced #200

Open
musicus opened this issue Feb 26, 2016 · 4 comments
Open

Composer Search Field Simple > Advanced #200

musicus opened this issue Feb 26, 2016 · 4 comments

Comments

@musicus
Copy link
Contributor

musicus commented Feb 26, 2016

When switching from the simple to advanced search, the searched term needs to transfer to the advanced column (this may or may not be possible).

@agpar
Copy link
Contributor

agpar commented Mar 1, 2016

I'm not sure this is possible... We could guess which field the user was trying to search on, but it will not be a very good guess.

@musicus
Copy link
Contributor Author

musicus commented Mar 1, 2016

The composers are easy, you can query whether the composer exists and fill the composer into the composer field. The title would be everything else that is not in the composer field, but I agree it would be a guess at best.

@agpar
Copy link
Contributor

agpar commented Mar 1, 2016

The auto-suggest on the general search field just pulls results from all the suggestion dictionaries and does not know where they came from. The JS itself does not have any flag or notion that one of its suggestions has been input.

Without re factoring extensively, the best we could do would be to run like 12 ajax queries when you change tabs (1 to each suggestion dictionary) and see if an exact match is returned by any... And that would only work when there's an exact match, and no conflicting matches.

@musicus
Copy link
Contributor Author

musicus commented Mar 1, 2016

It is a difficult task indeed. Continue thinking about it. It may not be possible, as I previously had stated, thus to come up with a solution would be quite impressive.

@musicus musicus assigned vee-L and unassigned AFFogarty Sep 12, 2016
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

4 participants