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

Feature request: change background via admin panel #57

Closed
koenigd79 opened this issue Jul 19, 2019 · 10 comments
Closed

Feature request: change background via admin panel #57

koenigd79 opened this issue Jul 19, 2019 · 10 comments

Comments

@koenigd79
Copy link

Toll fände ich die Möglichkeit über das super Admin panel auch noch das Hintergrundbild zu ändern. Zum einen ein pulldown Menü für die schon vorhandenen Bilder und zum anderen noch die Möglichkeit weitere Bilder hochzuladen. So könnte man die Box für jede Feier anpassen, ohne sich tiefer im System anmelden zu müssen.

Gruß Daniel

@tobiashaas
Copy link
Contributor

Fände ich auch gut, ebenfalls vielleicht auch die Hintergrund Farbe zb. in der Galerie!
Gut fände ich auch wenn man im Admin Panel auch sagen könnte wo die Fotos überall gespeichert werden.

Nur Kamera
Nur Raspi
oder beides

Grüße Tobi

@andi34
Copy link
Collaborator

andi34 commented Jul 20, 2019

@koenigd79 glaube nicht so einfach, da es im style.css definiert ist.
https://github.com/andreknieriem/photobooth/blob/master/resources/css/style.css#L264
https://github.com/andreknieriem/photobooth/blob/master/resources/css/style.css#L479

Aber denke es ist nicht schwer einfach selbst das bg.img zu ersetzen wenn gewünscht. Andere Hintergrundbilder gibt es ja auch nicht im Source.

@tobiashaas Instruktionen um das Bild auf der Cam zu behalten ist ja in der README. Muss ja die cmd Line angepasst werden und das geht bereits im Admin Panel.
Je nach Cam muss ja ein anderes target definiert sein, also als generelle Option zur Auswahl mMn nicht gut.

@tobiashaas
Copy link
Contributor

Stimmt, sorry das hab ich übersehen, Danke dir

@andi34
Copy link
Collaborator

andi34 commented Jul 20, 2019

Die cmd Line wäre dann nicht per hand änderbar. Oder du bräuchtest eine Checkbox zusätzlich.

Checkbox: Cmd Line per Hand aktivieren = true
--> Auswahl aus Liste hat keine Funktion. Cmd Line kann angepasst werden.

Checkbox: Cmd Line per Hand aktivieren = false
--> Auswahl aus Liste aktiv, cmd Line nicht änderbar per hand
Nur Kamera: macht keinen Sinn da Galerie dann überflüssig und kein Bild auf Bildschirm
Nur Pi: $config['take_picture']['cmd'] = 'sudo gphoto2 --capture-image-and-download --filename=%s images';
Pi und Cam1: $config['take_picture']['cmd'] = 'sudo gphoto2 --capture-image-and-download --keep --filename=%s images';
Pi und Cam2: $config['take_picture']['cmd'] = 'sudo gphoto2 --set-config capturetarget=0 --capture-image-and-download --keep --filename=%s images';
Pi und Cam3: $config['take_picture']['cmd'] = 'sudo gphoto2 --set-config capturetarget=1 --capture-image-and-download --keep --filename=%s images';

Ob 1, 2 oder 3 würde man nur durch probieren erfahren. Da ist es einfacher die Instruktionen im README zu befolgen. Jede Cam hat im Zweifel ein anderes target.

Korrigiert mich wenn ich falsch liege.

@tobiashaas
Copy link
Contributor

Meine Überlegung war dass man 2 Checkboxes anlegt und die CMD Line komplett rausnimmt.
So dass man nur noch anwählen muss was man haben möchte.

  • save only on Pi
  • save on Pi & Camera

Aber wenn das CaptureTarget je nach Kamerahersteller unterschiedlich ist, macht das kein Sinn, da hast du Recht! ;-)

@andi34
Copy link
Collaborator

andi34 commented Jul 20, 2019

Ja, gibt cams mit internem Speicher soweit ich weiss. Auch ist capturetarget glaub nicht bei allen cams unterstützt.

@koenigd79
Copy link
Author

Ja so mache ich da aktuell auch, einfach die bg.jpg entsprechend anzupassen.

Dachte halte wäre ne feine sache, das über das Admin panel zu erledigen. Aber klar liegt im ccs und nicht in den php files, das lässt sich dann nicht so einfach realisieren....

Ansonsten sind deine weiterentwicklungen super andi. Habe mir vom https://www.reinhard-rebernig.at/website/websites/fotokasterl/ schicken lassen. Schau mal ob ich die filter ans laufen kriege.

@andi34
Copy link
Collaborator

andi34 commented Jul 20, 2019

@koenigd79 hab ich schon am laufen mit Filter auswahl. Auch etwas andere Filter eingebaut. Sind nur noch Feinheiten. Denke nächste , spätestens übernächste Woche lade ich es hoch für alle

@andi34
Copy link
Collaborator

andi34 commented Jul 20, 2019

Aber wenn du Lust hast: Lad doch dein bg.img Mal hoch. Dann können auch andere es nutzen als Alternative.

@andi34 andi34 changed the title Admin panel: Hintergrundbild ändern Feature request: change background via admin oanel Sep 5, 2019
@andi34 andi34 changed the title Feature request: change background via admin oanel Feature request: change background via admin panel Sep 7, 2019
@andi34
Copy link
Collaborator

andi34 commented Sep 9, 2019

discussion/development continues here #68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants