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

REGISTRIERUNG: Konfigurierbare Texte #2

Closed
christianbarkowsky opened this issue Apr 30, 2018 · 9 comments
Closed

REGISTRIERUNG: Konfigurierbare Texte #2

christianbarkowsky opened this issue Apr 30, 2018 · 9 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@christianbarkowsky
Copy link
Contributor

Label - sowie Infotext inkl. Link sollen konfigurierbar werden.

@christianbarkowsky christianbarkowsky added the enhancement New feature or request label Apr 30, 2018
@christianbarkowsky christianbarkowsky self-assigned this Apr 30, 2018
@christianbarkowsky christianbarkowsky added this to the 0.2.0 milestone May 2, 2018
christianbarkowsky added a commit that referenced this issue May 2, 2018
christianbarkowsky added a commit that referenced this issue May 2, 2018
@frontendschlampe
Copy link
Member

Wie besprochen wäre folgende Idee sinnvoll:

aktuell wird bei einem Standard Checkbox Input folgendes ausgegeben:

  • Feldbezeichnung als legend - Eingabe: input.text
  • Checkbox als input
  • Optionen > Bezeichnung als label - Eingabe: input.text

Für unsere Zwecke müsste es folgendermaßen aussehen:

  • Feldbezeichnung als Überschrift in legend- Eingabe: input.text mit Überschriftswähler (hier würde ich aber mal noch mit @MacKP Rücksprache halten wg. der Semantik ggfs. nur ein strong)
  • ausführliche Beschreibung in legend - Eingabe über TinyMCE zum Setzen von Links auf die Datenschutzvereinbarung etc.
  • Checkbox bleibt wie im Standard
  • Optionen > Bezeichnung bleibt wie im Standard

@frontendschlampe frontendschlampe changed the title Konfigurierbare Texte REGISTRIERUNG: Konfigurierbare Texte May 8, 2018
@christianbarkowsky
Copy link
Contributor Author

Die Texte können wir entweder über die Root-Page einbinden - oder wir müssen ein Widget bauen. Mehr bietet die Registrierung lnicht. 🌵

@frontendschlampe
Copy link
Member

hmmm ... es gibt auch grad Diskussionen bei Contao contao/core-bundle#1512 (comment)

@frontendschlampe
Copy link
Member

Mit einem eigenen Registrierungswidget geht's nicht? Zu viel Aufwand?

@christianbarkowsky
Copy link
Contributor Author

So wie es jetzt aufgebaut ist - nein.

@MacKP
Copy link

MacKP commented May 9, 2018

Wie gestern kurz mit @frontendschlampe besprochen wäre ein HTML Aufbau in der Art ganz praktisch (vereinfachte Fassung):

<div class="single-checkbox">
  <div class="von-contao-schon-vorhanden">
    <input>
    <label>
  </div>
  <div class="special-content">
    <h>Headline of the Content (could be used for an accordion)</h>
    <div class="special-content-content">
      Here is the Text (and a link ore something else).
    </div>
  </div>
</div>

Das ganze ist etwas mehr, als nur ein Link... Ich hab da eben etwas allgemeiner gedacht. Als label oder legend würde ich das nicht unterbringen, da man dort eher andere Dinge enthalten hat. Das von mir vorgeschlagene ist im Grunde eine Variante um das ganze recht einfach nutzen zu können. Um das ganze noch etwas anzureichern könnte man noch mit aria-describedby und einer ID arbeiten.

Viele Grüße

@leofeyer
Copy link

leofeyer commented May 9, 2018

Wir müssen wohl das Frontend-Modul ModuleRegistration in $GLOBALS['FE_MOD'] durch unser eigenes Modul ersetzen, damit wir das Feld hinzufügen können (siehe contao/core-bundle#1512 (comment)).

christianbarkowsky added a commit that referenced this issue May 22, 2018
christianbarkowsky added a commit that referenced this issue May 22, 2018
christianbarkowsky added a commit that referenced this issue May 22, 2018
christianbarkowsky added a commit that referenced this issue May 22, 2018
christianbarkowsky added a commit that referenced this issue May 22, 2018
@frontendschlampe
Copy link
Member

@christianbarkowsky Ich habe es gerade einmal installiert und es muss eigentlich genau andersrum sein von den Texten her.

Das "Datenschutz-Hinweis Label" (<legend>) muss den TinyMCE enthalten und die Beschriftung für die Checkbox "Datenschutz-Hinweis" ein einfaches Textfeld.

Es ist ja im Prinzip dann so:

Hiermit bestätigen ... Datenschutz (mit Verlinkung) ... usw.
(Checkbox) Ich stimme zu

Kannst du das kurzfristig ändern?

christianbarkowsky added a commit that referenced this issue May 23, 2018
christianbarkowsky added a commit that referenced this issue May 23, 2018
christianbarkowsky added a commit that referenced this issue May 23, 2018
@frontendschlampe
Copy link
Member

passt soweit ... muss dann nur noch in die 3.5 portiert werden

christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 23, 2018
christianbarkowsky pushed a commit that referenced this issue May 24, 2018
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

No branches or pull requests

4 participants