Kleinigkeit: Zusätzliches DIV

[Nina]

Ein klitzekleines Feature-Request: Es wäre gut, wenn um den Slider herum (inklusive den Buttons) noch ein Gesamt-<div> wäre. Dadurch könnte man dann dieses DIV z. B. mit position: relative versehen und die Buttons absolut positionieren o. ä.

[the_scrat]

Kann man das nicht auch direkt im Template erledigen?

ce_noobslide_setup.(xhtml/html5) -> öffnendes DIV
ce_noobslide_end.(xhtml/html5) -> schließendes DIV

[schman]

Kann man sicher, allerdings denke ich Nina meint das es standardmäßig Sinn machen würde.
Deshalb wäre es fein wenn es direkt der Extensionentwickler ändern könnte.

[magicsepp]

ich denke das ist bereits gegeben bei der aktuellen Version sieht die Strucktur so aus:
XHTML:
<div class="ce_noobSlide block" style="width:500px; height:350px; ">
<div class="ce_noobSlide_previews">
<div class="ce_noobSlide_container" style="width: 1000px; height: 350px;">
</div>
HTML:
<section class="ce_noobSlide block" style="width:500px; height:350px; ">
<div class="ce_noobSlide_previews">
<div class="ce_noobSlide_container" style="width: 1000px; height: 350px;">
</section>
Wozu noch ein <div> herum, reicht ce_noobSlide nicht?

[plastiko]

Die .control_buttons sind aber laut Demoseite nicht innerhalb von .ce_noobSlide.

[magicsepp]

Achso, naja um das ixtensia Beispiel aus dem anderen Thread nachzubauen bedarf es keiner Buttons. Die stören nur....
Dafür reicht setup-preview1-preview2-section1-section2-ende und ein bischen css.

[Nina]

Ich bezieh mich nicht auf die andere Sache, sondern allgemein. Ein Rahmen-DIV um die gesamten Elemente inkl. Buttons würde einfach für bestimmte Designs noch weitere Möglichkeiten mit sich bringen. Das ist ja auch der Grund, weshalb die Contao-Core-Module normalerweise alle noch ein Rahmen-DIV haben.

[philipp.kaiblinger]

Hallo Nina,

könntest du einen Feature-Request machen? Die Extension ist nun auf Github zu finden. Wenn ich diesen Beitrag früher gelesen hätte, hätte ich es gleich in die neue Version einbauen können.

https://github.com/kaipk/contao-pk_noobSlide/

lg Philipp

[philipp.kaiblinger]

Hallo Leute,

ich habe jetzt die Control-Buttons alle in ein div zusammengefasst und diese sind nun im ce_noobSlide div.
Könnt Ihr das mal testen? Anbei das neue Template. Meiner Meinung nach fehlt dann noch eine CSS-Anweisung für die Control-Buttons da .ce_noobSlide schon position:relative hat.

z.B.:

.control_buttons {bottom:0;}

Würde euch das so passen? So würden wir uns ein zusätzliches Div ersparen.

lg
Philipp

[magicsepp]

für mich und mein Beispiel passt das, danke auch für den firrst last bugfix.

[plastiko]

Würde euch das so passen? So würden wir uns ein zusätzliches Div ersparen.

Es klappt soweit gut.
Hat aber den Nachteil, daß sich die .control_buttons jetzt überhaubt nicht mehr außerhalb von .ce_noobSlide positionieren lassen (werden mit overflow: hidden; für .ce_noobSlide abgeschnitten).

Am besten fände ich ein zusätzliches div ohne overflow: hidden; welches position: relative; bekommen kann, wenn Mann das möchte.

[philipp.kaiblinger]

Stimmt, nur wie soll das neue DIV heißen? Mir fällt kein klassenname dazu ein.


Sent from my iPhone using Tapatalk

[plastiko]

Weiß ich auch nicht.
Vielleicht .ce_noobSlide_group oder .group_ce_noobSlide