liqd / adhocracy3

Adhocracy 3 with customizations for various projects
https://adhocracy3.readthedocs.io
Other
50 stars 11 forks source link

Add config options to allow deployment specific frontend css without making a new release #2897

Closed joka closed 6 years ago

joka commented 6 years ago

New config options:

adhocracy:
 frontend:
   # Absolute file system path to optional folder with custom css files.
    # All files are published under the url path `/static/custom/`
    # In this directory a file named `main.css` is expected. It is imported
    # after a3.css to allow overriding default css.
    custom_static_folder: ''"
dwt commented 6 years ago

Gefällt mir sehr gut. Eine Frage die ich mir gestellt habe ist ob wir es beim Deployment leichter haben, wenn man anstatt das CSS direkt ins config file zu schreiben, man auch für die Custom config direkt ein File angibt?

Was hältst Du davon? (Bin nicht überzeugt das das eine gute Idee ist.)

joka commented 6 years ago

Ein extra file ist etwas mehr Aufwand beim Deployment als einfach die vorhandene config zu erweitern.

dwt commented 6 years ago

Ich dachte an so ein Scenario:

+ custom_css/
| main.css
| logo.png

Ordner und logo müssen eh angelegt werden, css da drin zu haben hört sich einfacher zu bearbeiten an (um Änderungen zu testen)

dwt commented 6 years ago

Ich überlege ob es sogar am besten ist für das main css file einfach einen relativen Pfad in dem Verzeichnis anzugeben.

joka commented 6 years ago

ah, ja das macht sinn wenn man das custom_css verzeichnis sowiso anlegt. Mein Szenario war dass das oft nicht nötig ist.

joka commented 6 years ago

Geändert, es gib jetzt nur noch die options custom_static_folder:
Im spezifizierten Ordner wird die Datei "main.css" erwartet