-
Notifications
You must be signed in to change notification settings - Fork 83
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
Not capturing the correct selected words for the highlight #142
Comments
Note to self: |
I investigated quickly, the saved text seems correct: {
"before": "to manifest it over all religions, although the Mushrikoon hate it.ʺ [Sooratut-Tawbah, 9:33] Audio not supported!\nThe guidance is the beneficial knowledge, while the religion of truth is the ",
"highlight": "righteous action",
"after": " that is based upon sincerity to Allaah and following His Messenger (ﷺ).\nAnd beneficial knowledge includes any knowledge that contains a benefit to the Ummah for its life in the Dunyaa and"
} Although we can see an interesting issue: "Audio not supported" is part of the highlight snippet even though it's not desired for user presentation: <audio xmlns="http://www.w3.org/1999/xhtml" src="..." controls="controls">Audio not supported!</audio> I'm not sure what's wrong with the We might need to escalate this issue to Hypothesis, once this discussion is settled. |
@mrifni but how are you even able to play the audio? I am trying to do the same, but couldn't find any documentation/sample that can do it. |
@og-abdul-mateen You don't have anything to do to support it. However on FXL EPUB it's currently broken because of this issue: #131 |
That's interesting. I've been scratching my head over trying to support audiobook + text highlight support (basically Epub3 with MediaOverlay), but couldn't find a sample or documentation for it. Will try it on some Ebooks to see how it goes. Thanks @mickael-menu. |
Media overlays is not the same as |
Umm okay. I have generated some Ebook files of my own using syncabook (plaintext + audio -> SMIL + XHTML -> EPUB3 with MediaOverlay), but as you mentioned, it might not work atm. Is there a spec for the supported PS: Sorry for bloating this thread with unrelated questions. |
I would suggest waiting for the Media overlays implementation, this is the most user friendly and easiest to generate. |
Do we have any ETA for that? |
Should be in 2023 |
@mickael-menu do you have sample Ebooks with audio that work with Readium Kotlin Toolkit on Android? I couldn't make any Ebook audio format work with Readium Android. |
https://github.com/IDPF/epub3-samples/releases/download/20230704/cc-shared-culture.epub This one is supposed to work. If not (with some patience), please report a detailed issue. |
@qnga I tried this sample Ebook. Video works fine, but audio doesn't work. Even it would, this is not what I'm looking for. I guess I'll wait for the official |
@mickael-menu do we have any update on the EPUB3 with MediaOverlay support in Readium Kotlin Toolkit? If it's far away in future, can you suggest another solution that essentially does the same thing? |
It should be available in 2024, I can't be more precise than that. I'm not sure about any alternative, it depends what you need and if you produce your own publications. If you must consume EPUBs containing media overlays, then no there's no alternative besides implementing it yourself. |
running the latest develop branch on a real device (oneplus 5)
please see the attached video
bug.record.mov
The text was updated successfully, but these errors were encountered: