Skip to content
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

Fix fixed tariff configuration UI #626

Merged

Conversation

MartinRinas
Copy link
Contributor

Implement a user interface for configuring fixed tariff hours, including options for adding, removing, and updating tariffs and their active times.

UI for openWB/core#2167

@MartinRinas
Copy link
Contributor Author

@benderl hier bräuchte ich ein wenig Hilfe. Grundsätzlich funktionert das alles, allerdings ist das UI noch nicht rund.

Idealerweise hätte ich die Aktiven Zeiten gern im Sinne einer Tabelle organisiert. Start, Ende und Löschen als eine Zeile so wie das jetzt ist, aber natürlich mit sinnvollerer Aufteilung des Platz. Also viel für die Dropdowns mit Beschriftung, nur das unbedingt notwendige fürs Icon zum Löschen.
Davon bin ich aktuell ein Stück weit entfernt.
image

Ich habe mich bewusst gegen eine weitere Karte entschieden, das werden sonst zu viele ineinander geschachtelt, ich finde das so wie es aktuell ist schon fast zu viel. Zudem hätte die Karte wirklich nur zwei Elemente welche insgesamt viel zu viel Platz verbrauchen. Die tabellarische Sicht würde das m.E. deutlich eleganter lösen.

@benderl
Copy link
Contributor

benderl commented Jan 23, 2025

@MartinRinas
Ich erstelle dazu mal einen Feature-Branch sowohl im Core als auch hier. Dann kann ich dabei besser helfen.

@benderl benderl changed the base branch from main to feature-fixed-hour-tariff January 23, 2025 11:19
@benderl benderl marked this pull request as ready for review January 23, 2025 11:20
@benderl benderl merged commit 8cc78e0 into openWB:feature-fixed-hour-tariff Jan 23, 2025
1 check passed
@benderl
Copy link
Contributor

benderl commented Jan 23, 2025

Dort geht es weiter: #627

@MartinRinas MartinRinas deleted the feature-scheduledET branch January 23, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants