buda-base / public-digital-library

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

Monlam Dictionary integration UX feedback #800

Closed berger-n closed 1 year ago

berger-n commented 1 year ago

[edit: fix spelling + dictionary on when opening viewer (#800) ]

berger-n commented 1 year ago


evanyerburgh commented 1 year ago

@berger-n Great! Yep, Control+C works for me now.

as long as there's space to display text why not show it?

I agree. And I like how when the browser window gets narrower, they drop down to icons.

Testing it just now, I noticed two more things about narrow screens...

Here's a screencast from my computer. Notice also that when I highlighted text while the browser is narrow, it didn't unselect the previously selected text (which you can see when the window gets wider again.)

https://user-images.githubusercontent.com/51434640/221777768-a3629f9a-ae3b-4b90-869b-6083f92a90e1.mp4

berger-n commented 1 year ago

dictionary now enabled on mobile, you can test it here: http://library-dev.bdrc.io/show/bdr:UTIE0OPI44484488_I3CN3541?backToEtext=bdr:W3CN3328#open-viewer click on "Copy" after selecting text to open Monlam popup (not perfect in terms of UX as it sure needs to be documented, but it does not seem possible for a website to add an option in a mobile's native contextual menu I'm afraid)

screen-20230228-180019 mkv

oh and I added swipe support for closing both Monlam results panel and the bottom bar (which close button is really small)

eroux commented 1 year ago

very nice, thanks!!

evanyerburgh commented 1 year ago

Awesome, looks great! Thanks, Nicolas

JannTibetan commented 1 year ago

when searching within etexts, the initial search string is highlighted in the etext. This is a wonderful. However, this highlighted text cannot be subsequently highlighted by the Monlam Dictionary feature.

https://user-images.githubusercontent.com/44705185/222219471-fcfc2c35-3c65-4336-9cf4-4c84ba6db547.mov

berger-n commented 1 year ago

oh that's surprising, thanks! actually it already can on my computer though (using Chrome): link

simplescreenrecorder-2023-03-02_09 52 58 mkv


but let me check what's going on in Safari! (or is it Firefox?)

berger-n commented 1 year ago

ah it works for me in Safari as well...

JannTibetan commented 1 year ago

OK it did work for me once on Safari but most of the time it doesn't. It works very smoothly on Firefox. Anyway, this is an incredible new feature. People will certainly ask why they can't just query the dictionary directly but this is samsara and desires are endless...

eroux commented 1 year ago

well, they can on https://monlamdic.com/ ... but it's true that we don't have a link... perhaps the at the top of the results could have a link to it?

berger-n commented 1 year ago

OK it did work for me once on Safari but most of the time it doesn't.

it is intriguing... I'll dig into this

well, they can on https://monlamdic.com/ ... but it's true that we don't have a link... perhaps the at the top of the results could have a link to it?

(actually the title & icon at the top of the results are already links to monlamdic.com)

berger-n commented 1 year ago
berger-n commented 1 year ago

@eroux what about a new sort by "quality" on the etext results page?

eroux commented 1 year ago

good idea! Thanks for the changes, I think a good limit could be pretty high, let's start maybe at 40

JannTibetan commented 1 year ago

@berger-n I tested the link you sent and it works well on Safari. Thanks

berger-n commented 1 year ago

image

simplescreenrecorder-2023-03-07_18 14 07 mkv

eroux commented 1 year ago

this is great, thanks! perhaps 120 characters is safer. Also, maybe "OCR quality" would be good?

JannTibetan commented 1 year ago

Or, "accuracy (ranking)" Quality is probably not self explanatory enough.

berger-n commented 1 year ago

thanks @JannTibetan! what about simplifying these labels for filter values as well? something like this: \

image

JannTibetan commented 1 year ago

Sure. Great idea. Filters are much more visible than rankings so let's build in some redundancy about this so that people know what to expect from the texts. Thanks.

eroux commented 1 year ago

LGTM, thanks!