openzim / ted

Provide the best of TED.com for offline usage!
https://download.kiwix.org/zim/ted/
GNU General Public License v3.0
13 stars 8 forks source link

Select language is not persistent enough #208

Open kelson42 opened 1 week ago

kelson42 commented 1 week ago

If I pick French in language selector at https://library.kiwix.org/viewer#ted_mul_blockchain_2024-06/, then it works. But if I click on the topbar button to go to the homepage of my ZIM, the Inloose my language selection, it should not.

benoit74 commented 1 week ago

This information is stored in a query parameter, so definitely not persistent at all.

I think we should store this information:

@rgaudin do you have any information on which kind of storage should be used for proper operation across kiwix readers? Or should I create a test ZIM with this to confirm what is working?

rgaudin commented 3 days ago

Well without implementation spec/reqs, it's hard to know (I know I repeat myself). I am not sure about the actual answer to your question. Clearly deserves assessment and documentation. Cookie is the reglar safest over HTTP but on readers using a custom scheme it might not be enabled. LocalStorage support is very wide and we use it already in nautilus across readers so that should be OK but there's no guarantee

kelson42 commented 3 days ago

Cookies should work, otherwise this is a bug.

benoit74 commented 1 day ago

Let's rely on local storage for now, see https://github.com/openzim/overview/issues/39#issuecomment-2209220564