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

Saving memory usage by creating simple dropdownlists via Javascript #108

Closed
wants to merge 2 commits into from

Conversation

dbambus
Copy link
Collaborator

@dbambus dbambus commented Jun 13, 2021

No description provided.

@dbambus dbambus requested a review from tali June 13, 2021 10:52
@dbambus dbambus added the enhancement New feature or request label Jun 13, 2021
@tali
Copy link
Member

tali commented Jun 13, 2021

Hmm, bin noch nicht ganz überzeugt, ob wir uns das antun sollten, dass wir die Webseite mit handgeschriebenem Javascript erzeugen sollen. Wenn es irgendwelche fertigen Widgets gibt die wir einbinden können und die dann in JS gebaut sind (wie der ColorPicker) find ich das in Ordnung, aber so ist das nochmal extra Code den wir supporten müssen.

@tali
Copy link
Member

tali commented Jun 13, 2021

Von der Änderung unabhängig: bitte in den eigenen Repos nur rebasen, nicht mergen. Sonst bekommen wir eine Flut an Merge-Commits.
Ich habe das bei mir in .git/config folgendermassen eingestellt:

[remote "origin"]
	url = git@github.com:tali/Wortuhr.git
	fetch = +refs/heads/*:refs/remotes/origin/*
[remote "upstream"]
	url = https://github.com/ESPWortuhr/Wortuhr.git
	fetch = +refs/heads/*:refs/remotes/upstream/*
	gh-resolved = base
[branch "dev"]
	remote = upstream
	pushRemote = origin
	merge = refs/heads/main
	rebase = true

Damit verwendet er als basis für Vergleiche immer das Hauptrepo, macht bei git pull ein Rebase der eigenen Commits und pushed aber immer nur in das eigene Repo.

@dbambus
Copy link
Collaborator Author

dbambus commented Jun 14, 2021

Zum ersten Kommentar.

ich habe halt die Angst, dass fertige Widgets, den Code nur unnötig aufblähen und das der Effekt der Codeusability weg geht. Meine Beweggründe waren hier nur Vermeidung von Copy&paste Code innerhalb der HTML.

@tali
Copy link
Member

tali commented Jul 5, 2021

Die Frage ist natürlich auch, ob ein drop-down Feld mit ewig vielen Einträgen überhaupt das richtige UI Element ist.
Vielleicht sind für die Helligkeitswerte einfache Slider besser und für die Zeiteinstellung eher ein Widget ala https://dhtmlx.com/docs/products/dhtmlxTimePicker/?

@dbambus
Copy link
Collaborator Author

dbambus commented Apr 16, 2023

@Wandmalfarbe Wäre das eine Idee ?

@dbambus
Copy link
Collaborator Author

dbambus commented Apr 19, 2023

Solved with #242

@dbambus dbambus closed this Apr 19, 2023
@dbambus dbambus deleted the main_webSave branch April 19, 2023 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants