overview icon indicating copy to clipboard operation
overview copied to clipboard

Ensure that all readers support bookmarking of page with fragment in URL

Open benoit74 opened this issue 1 year ago • 6 comments

With the move to Vue.JS of youtube and freecodecamp scrapers (and soon kolibri and maybe others), all "items" are accessed with a URL containing a fragment like ./index.html#/watch/heating_a_tiny_home_in_winter_with_propane-8CT9?list=winter_prepping-0Xxa. For kiwix-serve, see e.g. https://library.kiwix.org/viewer#canadian_prepper_winterprepping_en_2024-08/index/heating_a_tiny_home_in_winter_with_propane-8CT9 (which redirects to https://library.kiwix.org/content/canadian_prepper_winterprepping_en_2024-08/index.html#/watch/heating_a_tiny_home_in_winter_with_propane-8CT9?list=winter_prepping-0Xxa with the viewer iframe loaded)

This might have an impact on bookmarking feature of readers, which need to save to whole URL including the fragment.

We need to test all readers to confirm which ones are working and/or open proper issues.

  • [ ] kiwix-serve => KO, https://github.com/kiwix/libkiwix/issues/1120
  • [ ] kiwix-desktop => KO, https://github.com/kiwix/kiwix-desktop/issues/1195
  • [x] kiwix-android
  • [x] kiwix-apple
  • [x] kiwix-js

benoit74 avatar Sep 02 '24 16:09 benoit74

  • kiwix-android 3.11.1 on Android 13 works OK
  • kiwix-apple version 3.6.0 (173) on MacOS Sonoma 14.5 works OK

benoit74 avatar Sep 03 '24 08:09 benoit74

@benoit74 There is no issue open IMHO, you can open one.

kelson42 avatar Sep 03 '24 08:09 kelson42

  • kiwix-js : looks like there is no bookmarking concept / button, you cannot reload a ZIM entry from only a saved URL in your browser, so there is no issue (@Jaifroid did I missed something?)

benoit74 avatar Sep 03 '24 08:09 benoit74

Sorry @benoit74, I'm on an idyllic Greek island, so not really following issues much atm! I confirm there's no bookmark function in KJS. This isn't possible due to the fact that you have to give the browser access to the ZIM each time you restart a session except in the case of browsers that support the File System Access API -- which we support if the browser supports it, but only Chrome(-ium) atm. Do we need to support this for YouTube scraper?

Jaifroid avatar Sep 03 '24 08:09 Jaifroid

I'm on an idyllic Greek island, so not really following issues much atm!

No worries, and enjoy (and there wasn't any hurry, I just wasn't sure you where following kiwix/overview at all so preferred to tag you ^^)

I confirm there's no bookmark function in KJS.

Good thank you ! I agree it would be complex to implement and maybe not even worth it

Do we need to support this for YouTube scraper?

Nope, it is the opposite, we want to be sure that readers supporting bookmarking are still working with recent Youtube scraper changes. kiwix-serve is not for instance. Since kiwix-js simply does not support bookmarking, then we are fine.

If possible, just keep in mind that should you decide to implement a bookmarking feature one day, then you should consider youtube / freecodecamp ZIMs as good test candidates due to their particularities (I know I would probably not remember it, so no worries if you don't ^^).

benoit74 avatar Sep 03 '24 09:09 benoit74

  • kiwix-desktop: KO, I'll open an issue and update first comment with link

benoit74 avatar Sep 03 '24 09:09 benoit74