Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Filter search results by user-selected dictionary sources #126

Closed
3 tasks done
eddieantonio opened this issue Oct 14, 2019 · 15 comments
Closed
3 tasks done

Filter search results by user-selected dictionary sources #126

eddieantonio opened this issue Oct 14, 2019 · 15 comments

Comments

@eddieantonio
Copy link
Member

eddieantonio commented Oct 14, 2019

Prerequisite: #125

Enables the "advanced search" dropdown that allows one to filter on dictionary source.

Addresses the following user story:

As a Cree language educator, I want to see search results ONLY from certain dictionary sources

  • Requires new UX [Diane]
  • Requires accessible HTML/CSS [Eddie]
  • Requires database implementation [Matt or Eddie]
@eddieantonio
Copy link
Member Author

Note: I deleted my first attempt at the HTML in this commit: ae6c0c9.

git revert ae6c0c9d7567ba77ad26718e2103c5963360a31a

To get the code back.

@aarppe aarppe changed the title Filter search results by enabled dictionary sources Filter search results by user-selected dictionary sources Jul 22, 2020
@eddieantonio
Copy link
Member Author

@nienna73 here are the old mockups that our UX deverloper made in 2019:

Screen Shot 2021-07-21 at 1 39 41 PM
Screen Shot 2021-07-21 at 1 38 31 PM
Screen Shot 2021-07-21 at 1 38 53 PM

@nienna73 nienna73 self-assigned this Jul 27, 2021
@nienna73
Copy link
Contributor

Do we want to keep this UI or add the dictionary source to the settings page?
Adding the options to the new settings page makes more sense to me.

@eddieantonio
Copy link
Member Author

Do we want to keep this UI or add the dictionary source to the settings page?
Adding the options to the new settings page makes more sense to me.

I agree; I think it makes more sense to add it to the settings page, since I think it's actually a niche feature. That being said, I have a personal attachement to the style of those checkboxes! 😂

@dwhieb
Copy link
Contributor

dwhieb commented Jul 28, 2021

I also like this UI! But also agree that the settings page is the place for them.

@nienna73
Copy link
Contributor

@aarppe is the idea that a user can select more than one dictionary source at a time? For example, CW and MD? Or should they only be able to select one, that is: CW or MD (among the other choices)?

@aarppe
Copy link
Contributor

aarppe commented Aug 19, 2021

@nienna73 The idea is that users could select any combination of sources, so CW alone or MD alone, or both CW and MD (and then more, once we get AECD in as well).

@nienna73
Copy link
Contributor

The idea is that users could select any combination of sources

Okay, that's what I've been trying to do. I'm finding it difficult to set multiple options, so I wanted to clarify this was actually the goal.

@nienna73
Copy link
Contributor

nienna73 commented Oct 4, 2021

I came across an edge case in development:
Screen Shot 2021-10-04 at 2 42 23 PM

What do we do when the search result is a form of a word from a different dictionary, like in the example above?
Should I include the definition of mîpit (in this case)?
Or completely remove nîpit from the results?

@dwhieb
Copy link
Contributor

dwhieb commented Oct 4, 2021

@nienna73 Is the above screenshot filtering what currently happens when filtering for MD entries, or for CW entries?

@nienna73
Copy link
Contributor

nienna73 commented Oct 4, 2021

That's a good question. The example above is showing only MD results.

I found a perhaps better example, still filtering to show just MD results:

Screen Shot 2021-10-04 at 3 50 22 PM

@dwhieb
Copy link
Contributor

dwhieb commented Oct 4, 2021

@nienna73 Ok so to clarify: This second example is supposed to be showing only MD results, but it includes a CW entry because ayiwinisa is recognized as a form of the CW entry ayiwin. Do I have it right?

I'm 100% fine with simply not showing either "form of ayiwin" or the entry for ayiwin in this case.

Another solution is to remove the entry for ayiwin, but keep "form of ayiwin", simply without the link. Also 100% fine with that.

@aarppe Do you have a preference?

@nienna73
Copy link
Contributor

nienna73 commented Oct 5, 2021

@dwhieb Yes, you have that correct. I think both of those options are possible, so I'll wait to see what the preferred implementation is.

@aarppe
Copy link
Contributor

aarppe commented Oct 5, 2021

This is the problem when MD doesn't also include a lemma entry for inflected word-form entries - we'll have more of these cases with some of the imperative verbs. When I was thinking about this almost three years ago, I was considering whether we ought to add "technical" lemma entries, though we might want to run those past speakers - fortunately there's only a few hundred of such forms.

Strictly speaking, if we're not showing content from the other dictionary, then it might be best that we're consistent about that, and neither show the lemma entry, i.e. not show ayiwin (like: cîmân) etc., nor the link to the lemma entry (which is in the other dictionary), i.e. form of ... etc. However, the FST analysis can still show that lemma (right now the stem, but I think we'll want to swap back to lemmas eventually), since the FST lexicon is independent of the actual sources.

@aarppe
Copy link
Contributor

aarppe commented Jan 15, 2022

@nienna73 As part of the dictionary source selection (#126, this issue is getting resolved, but to close that we need to resolve the case of empty results, cf.

image

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants