-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gewählter Zeitraum wird bei Switch in anderen Tab vergessen #88
Comments
Die Tabs für die Navigation zwischen den Seiten wird von Redaxo selbst generiert. Ich lasse das Issue offen und bin natürlich auch über Pull requests glücklich :-) |
Hi Andreas, verstehe ;-) Da gibt es sicher einen ExtensionPoint, wo man das eingestellte Datum (aus der Url extrahiert z.B. "&date_start=2022-02-15&date_end=2022-04-02") anhängen könnte. Fragt sich nur welcher das ist. |
Mache es dir einfach und speichere die Auswahl in der session. |
Das ist eine sehr gute Idee :) |
Ansonsten... das JS macht den Job, solange man nur zwischen den drei relevanten Tabs wechselt: `// --------------------------------------------------------------------------------------------------------------------------
});` |
Es liest die date-inputs aus. Wenn du die beiden Dati in der Session speicherst und die inputs wieder aus der Session befüllst, funktioniert es auch bei wiederkehr nach einem Wechsel zu anderen Seiten. In jedem Fall hängt es die nötigen Parameter an die Links an. |
Ups, ich war zu vorschnell. Ist noch ein Bug drin. Sofern interesse besteht, mach ich es aber noch gerne fertig. |
War nur ein Flüchtigkeitsfehler. hier das Update nun auch mit api + media: `
}); |
@oioix kannst du dazu einen PR erstellen? |
Gerade gemacht. In Datei: "statistics.js" eingefügt. Hab leider im comment versehendlich das falsche Issue "#92" statt "#88" angegeben. Also bitte den comment einfach ignorieren. |
@oioix hier ist kein PR. |
See issue AndiLeni#88
Evtl. hast du den PR in deinen lokalen Fork gepackt statt hier. Das kann man beim PR-Erstellen auswählen. Danke schonmal! |
Ja, vermutlich. Ist mir das letzte mal auch schon passiert. Aber jetzt passt es ja. |
Cooles Addon
Schön wäre es, wenn der individuell gewählte Zeitraum bei Switch in einen anderen Reiter (z.B. von Statistik zu Seitenaufrufe) erhalten bliebe.
LG, Chris
(enhancement)
The text was updated successfully, but these errors were encountered: