Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

[WIP] Configure kiwix to not display a searchbar and let ideascube handle it.#43

Open
mgautierfr wants to merge 1 commit into
masterfrom
kiwix_proxy
Open

[WIP] Configure kiwix to not display a searchbar and let ideascube handle it.#43
mgautierfr wants to merge 1 commit into
masterfrom
kiwix_proxy

Conversation

@mgautierfr
Copy link
Copy Markdown
Member

This PR correspond to the 590 one on ideascube ideascube/ideascube#590

@fheslouin
Copy link
Copy Markdown
Collaborator

How far are we from using this new feature ?

@mgautierfr
Copy link
Copy Markdown
Member Author

This is a bit on hold :/

It have been a lot of change on the kiwix side that somehow impact this feature:

  • We have rewritten all the build system of kiwix. (Split code in several repositories, add new master script to build everything, handle cross-compilation, ...) This works well but there is one missing feature for BSF : cross-compilation to ARM. Almost everything is in place (cross-compilation system with use of external toolschain) but the last step (use a toolchain for arm) is missing. This is not a priority on kiwix side (sadly for BSF). I could spend a day (at most) to add it. Without it we stick with a old version of kiwix-serve with which we cannot change the topbar.
  • There is a parallel work being made on the css/js for mobile. This work is in two parts : improve how we scrap wikimedia and create the zim; and rewrite the html/css of kiwix-serve to provide better responsive design. So we will rewrite the toolbar code on the kiwix side. And more than just the toolbar, we will probably update how kiwix-serve serves it.

For all of this, I think it is preferable to wait a bit for more improvement on kiwix side.
I've just open a bug on kiwix-tools for this need (kiwix/kiwix-tools#17)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants