Javascript-Edits zum laufen bringen

puzzle / decidim-zuerich

Mitwirkungsportal für die Stadt Zürich, basierend auf Decidim

https://mitwirken.stadt-zuerich.ch

GNU Affero General Public License v3.0

8 stars 2 forks source link

Javascript-Edits zum laufen bringen #240

Closed larsUE closed 2 years ago

larsUE commented 2 years ago

[x] Process redirects
[x] Jump down bei Prozessmenü: funktioniert nicht ⇨ betrifft unsere JS Erweiterung @carlobeltrame
[x] Menü Down-Slider: funktioniert nicht (Menü Down-Slider ist die Funktion, dass das Top-Menü (mit Suchbar) mit dem runterscrollen verschwindet. Siehe hier: https://github.com/puzzle/decidim-zuerich/issues/83)

larsUE commented 2 years ago

Notizen Process Redirects:

Prozess Redirects testen: Funktionieren derzeit nicht, weil Decidim eine neue Funktion eingeführt hat und ein Warnfenster bei externen Links einblendet.
EDIT: auch mit Whitelist-Domains (z.B. stadt-zuerich.ch) funktionieren Redirects derzeit nicht.
Edit @Kagemaru: Ich konnte dieses Verhalten nicht nachstellen. Mit dem Whitelisten der vollen Domain scheint dies zu funktionieren. ACHTUNG: stadt-zuerich.ch und mitwirken.stadt-zuerich.ch sind unterschiedliche Domains und müssen beide eingetragen sein. (https://mitwirken.stadt-zuerich.ch/ funktioniert nicht, da es nur den Domainteil prüft.)
Edit @larsUE: @Kagemaru: Das Whitelisting funktioniert gut. Aber die automatischen Redirects per Konvention, wie @carlobeltrame hier umgesetzt hat, scheinen nicht zu laufen. Beispiel siehe hier: https://mitwirken.integ.stadt-zuerich.ch/processes/extern.

carlobeltrame commented 2 years ago

@larsUE ich konnte diese 3 Dinge flicken, war jeweils derselbe Fix für alle drei. Testbar auf Integration.

bsantschi-zz commented 2 years ago

Hoi @lars, Carlo konnte es umsetzen und auf Integration deployen.

larsUE commented 2 years ago

Cool, danke! Process Redirects funktionieren super. @carlobeltrame Könntest du beim Jump-Down noch etwas weiterspringen? Schön wäre, dass das Menü bündig zum oberen Seitenrand ist. Im Moment schauts so aus, auch wenn die Seite gegen unten noch mehr Platz hätte:

Beispiel hier: https://mitwirken.integ.stadt-zuerich.ch/processes/testprozess/f/181/budgets/8/projects

Beim Header Down gibts noch ein Konflikt mit dem allgegenwärtigen Banner, wenn es eingeschaltet ist. Ich löse das aber gleich wie auf der MeinQuartierPlattform und nehme das Banner einfach an den unteren Seitenrand. Da musst du nichts mehr unternehmen.

carlobeltrame commented 2 years ago

@larsUE klar, kann ich probieren. Das ist aber auf der Produktion auf decidim 0.24 auch schon so: https://mitwirken.stadt-zuerich.ch/processes/probstei/f/98/ Ich vermute dass das durch das Zusammenspiel des Scroll-Features und des slidenden Hauptmenüs entsteht.

carlobeltrame commented 2 years ago

Habe eine einfache Lösung gefunden. Werde diese dann bald mal noch auf Integration ausrollen, wenn wir noch mehr Fixes haben.

larsUE commented 2 years ago

Yeah, supernice!, danke!