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

Template für CE Catalog Filterformular anpassen #45

Closed
andre5tz opened this Issue Jul 27, 2017 · 4 comments

Comments

Projects
None yet
2 participants
@andre5tz
Copy link

andre5tz commented Jul 27, 2017

Hallo,

ich habe mich nun daran gewagt, einen Filter über den Filtergenerator zu erstellen. Bei der Anpassung der Ausgabe mit css habe ich jetzt ein kleines Problem mit der Ausgabe der Checkboxen.

Aktuell kommt zuerst der Text und dann die Checkbox in catalog-manager/templates/ctlg_form_field_checkbox.html5:

<?php foreach( $this->options as $strKey => $strTitle ): ?>
      <div class="checkbox_container">
         <label for="id_form_<?= $this->name ?>_<?= $intIndex ?>"><?= $strTitle ?></label>
         <input type="checkbox" id="id_form_<?= $this->name ?>_<?= $intIndex ?>" name="<?= $this->name ?>[]" value="<?= $strKey ?>" <?= ( !empty( $this->value ) && in_array( $strKey, $this->value ) ? 'checked' : '' ) ?> <?= $this->onchange ?>>
      </div>
      <?php $intIndex++; ?>
<?php endforeach; ?>

Wäre es nicht besser, wenn label mit input getauscht wird, damit zuerst die Checkbox kommt und dann der Text?

<input type="checkbox" id="id_form_<?= $this->name ?>_<?= $intIndex ?>" name="<?= $this->name ?>[]" value="<?= $strKey ?>" <?= ( !empty( $this->value ) && in_array( $strKey, $this->value ) ? 'checked' : '' ) ?> <?= $this->onchange ?>>
         <label for="id_form_<?= $this->name ?>_<?= $intIndex ?>"><?= $strTitle ?></label>

Gleiches im Template catalog-manager/templates/ctlg_form_field_radio.html5.

Ich habe erst kurz versucht, das ganze mit float zu lösen, aber danach verrutscht die Textbox in er Höhe zum Text, sieht nicht schön aus.

VG
André

@alnv

This comment has been minimized.

Copy link
Owner

alnv commented Jul 27, 2017

Geht klar. Sollte ab v1.5 implementier sein.

@alnv alnv added the todo label Jul 27, 2017

@andre5tz

This comment has been minimized.

Copy link
Author

andre5tz commented Jul 28, 2017

In Contao 4.4 ist die Checkbox im Template form_checkbox.html5 übrigens so gelöst.

      <?php if ($option['type'] == 'option'): ?>
        <span><input type="checkbox" name="<?= $option['name'] ?>" id="opt_<?= $option['id'] ?>" class="checkbox" value="<?= $option['value'] ?>"<?= $option['checked'] ?><?= $option['attributes'] ?>> <label id="lbl_<?= $option['id'] ?>" for="opt_<?= $option['id'] ?>"><?= $option['label'] ?></label></span>
      <?php endif; ?>
@alnv

This comment has been minimized.

Copy link
Owner

alnv commented Aug 1, 2017

Ist erledigt in v1.5

@alnv

This comment has been minimized.

Copy link
Owner

alnv commented Aug 1, 2017

Das mit dem zusätzlichem span-Tag ist jedem selbst überlassen. Persönlich war ich nie Fan davon. Aber eventuell gibt es einen praktischen Grund dafür den ich nicht kenne ;)

@alnv alnv added discussion and removed todo labels Aug 1, 2017

@alnv alnv closed this Aug 1, 2017

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