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

Nested dictionary lookup uses original sentence instead of using respective sentence from dictionary #11754

Open
artjomsR opened this issue May 5, 2024 · 3 comments

Comments

@artjomsR
Copy link

artjomsR commented May 5, 2024

  • KOReader version: 2024.04
  • Device: Kobo sage

Issue

When you save a word in vocab builder, it saves the word in context of its original sentence. So if you look up a word from a book, everything works as expected.

However, you can still do dictionary lookups within dictionary's lookup. The problem is that in this case, the wrong sentence is used as context: it uses the original book sentence, rather than the dictionary one

I rely on these sentences for context so it's an important feature, especially since I'm using Anki's SRS to come back to these sentences+words multiple times

Steps to reproduce

  1. Do a dictionary lookup from any book
  2. Do another lookup from the dictionary definition popup
  3. Add the word to the vocab builder
  4. Close all popups, go to Vocabulary builder from the main menu, find the newly added word.
  5. Press on the 3 dots which will show the word in the context of the sentence
  6. Observe that the word from the dictionary lookup was used (from the 2nd step) but the original sentence was used (from the 1st step)
IMG_3549.mp4
@poire-z
Copy link
Contributor

poire-z commented May 5, 2024

Pinging @weijiuqiao - I guess we didn't think about this case :/

@weijiuqiao
Copy link
Contributor

We didn't indeed. In this case, the context should theoretically be the surrounding sentence in the definitions? I'm not sure if its extractable. If not, then it can fallback to null context. I'm also not sure if the context of the bottom of the stack of words can be retained.

@artjomsR
Copy link
Author

I agree, worst case scenario, if the context can't be extracted, it's better to have a word on its own with no sentence, rather inject the new (nested) word into the original sentence

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

No branches or pull requests

3 participants