search

Ahwxorg / LibreY

Framework and JS free privacy respecting meta search engine
GNU Affero General Public License v3.0
185 stars 22 forks source link

[Feature Request] Ability to specify default theme #135

Closed OdinVex closed 3 months ago

OdinVex commented 6 months ago

I'm not one for AMOLED, the ability to specify a CONFIG_... for default theme would be nice. Currently hijacked the Docker's entrypoint to sed replace ?? "amoled" to ?? "auto".

Edit for others (replace auto with whatever theme name you want, use the HTML value): entrypoint: /bin/sh -c "sed -i \"s/?? \\\"amoled\\\"/?? \\\"auto\\\"/g\" /var/www/html/misc/header.php; sed -i \"s/?? \\\"amoled\\\"/?? \\\"auto\\\"/g\" /var/www/html/misc/search_engine.php; docker/entrypoint.sh"

codedipper commented 6 months ago

Willing to implement this. Also noting files in /misc/search_engine.php and /settings.php need to also be changed.

OdinVex commented 6 months ago

I forgot about /misc/search_engine.php, I'll put that in the above sed.

davidovski commented 6 months ago

could be good for to add as option in config.php too of course

codedipper commented 3 months ago

This should be closed.

OdinVex commented 3 months ago

This should be closed.

https://github.com/Ahwxorg/LibreY/pull/151#issuecomment-2180834968

Is setting the theme possible via an environmental variable or setting without needing to hijack the entrypoint to sed or modifying files?

codedipper commented 3 months ago

Is setting the theme possible via an environmental variable or setting without needing to hijack the entrypoint to sed or modifying files?

Yes! Eventually it'll be merged into main from #167

OdinVex commented 3 months ago

Is setting the theme possible via an environmental variable or setting without needing to hijack the entrypoint to sed or modifying files?

Yes! Eventually it'll be merged into main from #167

Alrighty, tentatively closing and looking forward to #167 merging into main.

Ahwxorg commented 2 months ago

It has been merged!