search

hofff / contao-calendarfield

GNU Lesser General Public License v3.0
18 stars 3 forks source link

Läst das Kalender Icon ausblenden #4

Closed Aziz-JH closed 11 years ago

Aziz-JH commented 12 years ago

Es ist damit möglich das Kalender Icon im Backend auszublenden und stattdessen wird der datepicker direkt über das Textfeld ausgelöst.

aschempp commented 12 years ago

Besten Dank für den Input. Allerdings gefallen mir ein paar Dinge noch nicht:

  1. Warum nennst du das Feld dataClick. Wie wär's mit dateImage?
  2. Verstehe ich richtig dass das Bild nur noch angezeigt wird, wenn die Checkbox gesetzt ist? Das wäre nicht rückwärtskompatibel... Vielleicht könnte man das auch gleich so bauen, dass man auch ein eigenes Icon wählen kann?
Aziz-JH commented 12 years ago

stimmt dateImage wäre wirklich besser, lässt sich ja schnell umändern, nur müsste ich dafür ein neues pull request Starten oder kann man das auch jetzt noch abändern? Ich dachte die Checkbox ist automatisch gesetzt, das muss ich gleich mal ausprobieren, ich glaube das hab ich ganz vergessen. Ich hatte auch schon sowas im Kopf das mit einer Bildauswahl zu verknüpfen.

Edit: Ja die Checkbox ist automatisch gesetzt oder soll das anderes sein?

aschempp commented 12 years ago

Jeder weitere Pull-Request im gleichen Branch wird hier automatisch angehängt. Also gerne anpassen ;-)

Aziz-JH commented 12 years ago

So die Iconauswahl ist nun auch drin, ich weis nur nicht wie das mit den Contao Hooks funktioniert. Man müsste ja nun den "executePostActions" noch einbauen. Ich habe bei mir Lokal teilweise beim klicken auf das Bild Probleme manchmal muss man mehrere male den Datepicker öffnen damit er Funktioniert.

Edit: Ich trottel habe die 'showTemplateVars' ausgegeben und hatte vergessen das er ja dann die Sache zwei mal drin hatte. Trotzdem gibt es auch noch Probleme und zwar wenn man zwei mal hintereinander versucht die Urzeit zu ändern gibt er geht das nicht richtig.

aschempp commented 12 years ago

Ich habe den Pull Request in einen Feature-Branch gemergt und etwas angepasst: https://github.com/aschempp/contao-calendarfield/tree/feature/date-icon

Kannst du mal prüfen ob noch alles wie gewünscht funktioniert?

Aziz-JH commented 12 years ago

Im BE funktioniert alles einwandfrei, ich muss mir das merken mit dem subpalettes.
Ich habe hier erst mal das teil Online gestellt:
http://www.lavoute.de/reservierung-lavoute.html
wenn man die Uhrzeit zwei mal hintereinender ändern will geht das nicht bzw, er zeigt eine komplett andere Uhrzeit an. Ich habe da leider immer noch nicht den Fehler entdecken können. Dieses Problem tritt aber nur auf wenn man nur die Uhrzeit auswählt. Ich glaube das da eventuell irgend was zu viel ist beim JS.

aschempp commented 12 years ago

Bei mir klappt das mit der Uhrzeit.

@Toflar hast du eventuell noch Feedback?

Toflar commented 12 years ago

Ich kann leider nicht sehen, was sich genau bei diesem PR geändert hat. Ausserdem habe ich keinen Testcase, sprich keine Möglichkeit das zu testen. Mit Datum & Zeit hat aber damals bei mir alles funktioniert. Nur Zeit hab ich noch nie angewendet, wüsste auch nicht wie das gehen soll. Das Feld ist ja ein Timestamp und der besteht immer aus einem Datum :-)

aschempp commented 12 years ago

Mit dem hier auch nicht? https://github.com/aschempp/contao-calendarfield/compare/develop...feature/date-icon

aschempp commented 11 years ago

Die Funktion ist in der nächsten Version drin.