buda-base / public-digital-library

http://library.bdrc.io
4 stars 6 forks source link

Author in search index (from test) #940

Open roopeux opened 4 hours ago

roopeux commented 4 hours ago

Add authors as searchable fields, like authorName_bo_x_ewts. The field will not be shown in the UI, only used for matching.

eroux commented 4 hours ago

Yes, let's make things a bit more elaborate: when there is only one author (in the author field, which is the person id), the UI could display the (highlighted) matches in the authorName_* fields with the link to the person record instead of the usual prefLabel of the person record fetched after the query

eroux commented 4 hours ago

(But no change if there's no match in the authorName_* fields

eroux commented 4 hours ago

Thinking more about it, I think in the case where there are multiple authors (and thus we can't put the links on the matches, we could still display the matches in a new "authors' names" field... that would make more sense... what do you think @roopeux ?

roopeux commented 4 hours ago

Yes, I am generally against links within links, because it makes users hesitate. Since the whole search result div should be a link, it is actually better to not link from authors' names.

eroux commented 3 hours ago

ah ok, that's quite a big change (even from the old website), can you open a dedicated issue for that?

roopeux commented 3 hours ago

Yes, we can check that later. Let's see how this style works in the test. My point was in the context of adding links from names.