Modul catGallery - Upload Galerie
Re: Modul catGallery - Upload Galerie
Sorry hat jetzt doch ein wenig länger gedauert als gedacht ... hast ne eMail mit den geänderten Dateien erhalten.
Auf eine Light-Box habe ich vorerst verzichtet ...
Wie sieht es eigentlich mit einer Doku zur Variante aus, braucht ihr so etwas?
Auf eine Light-Box habe ich vorerst verzichtet ...
Wie sieht es eigentlich mit einer Doku zur Variante aus, braucht ihr so etwas?
Re: Modul catGallery - Upload Galerie
Bist du hier vielleicht schon weitergekommen ?
creativecat hat geschrieben: ↑Do 9. Jan 2020, 09:07Gerne per PN. Der Fehler mit dem Attribut Message ist seltsam. Ich versuch das mal zu rekonstruieren...
- creativecat
- Beiträge: 1430
- Registriert: Mi 6. Feb 2013, 12:41
- Kontaktdaten:
Re: Modul catGallery - Upload Galerie
Sooooo ich hab es endlich geschafft!
Pre-Release der 3.0 mit jeder Menge Varianten:
https://github.com/BlackCatDevelopment/ ... /v3.0-beta
Ich hab so viel geändert, dass ich hoffe, die Version lässt sich sauber installieren
EDIT: Wer noch Varianten hat, die er gerne drin haben möchte, einfach mir zukommen lassen ... wie auch immer
Pre-Release der 3.0 mit jeder Menge Varianten:
https://github.com/BlackCatDevelopment/ ... /v3.0-beta
Ich hab so viel geändert, dass ich hoffe, die Version lässt sich sauber installieren
EDIT: Wer noch Varianten hat, die er gerne drin haben möchte, einfach mir zukommen lassen ... wie auch immer
Re: Modul catGallery - Upload Galerie
Sauber
Installation hat ohne Fehler funktioniert (xampp PHP 7.3),
Varianten lassen sich speichern, aber bin mir nicht sicher wie es einzeln im Frontend aussehen sollte ...
In der /default/modify/set_image.tpl auf Github ist, so glaube ich, ein "copy&paste" Fehler entstanden:
Und zwar im Input von Adjust vertical > resize_x
<input type="text" class="cc_In100px" name="resize_y" value="{if $options.resize_x}{$options.resize_y}{else}407{/if}" />
Und ich hätte da noch zwei Bootstrap Varianten im Angebot > bs_Carousel und bs_Gallery.
Die Variante bs_Carousel habe ich Dir als Pull Request auf Git zukommen lassen (heute in der finalen Version).
Die Variante bs_Gallery noch nicht, würde natürlich auch gehen ...
Installation hat ohne Fehler funktioniert (xampp PHP 7.3),
Varianten lassen sich speichern, aber bin mir nicht sicher wie es einzeln im Frontend aussehen sollte ...
In der /default/modify/set_image.tpl auf Github ist, so glaube ich, ein "copy&paste" Fehler entstanden:
Code: Alles auswählen
<p class="cc_catG_dreispalten">
<span class="cc_In200px">{translate('Adjust horizontal')}:</span>
<input type="text" class="cc_In100px" name="resize_x" value="{if $options.resize_x}{$options.resize_x}{else}724{/if}" /> px<br/>
<span class="cc_In200px">{translate('Adjust vertical')}:</span>
<input type="text" class="cc_In100px" name="resize_y" value="{if $options.resize_x}{$options.resize_y}{else}407{/if}" /> px<br/>
</p>
<input type="text" class="cc_In100px" name="resize_y" value="{if $options.resize_x}{$options.resize_y}{else}407{/if}" />
Und ich hätte da noch zwei Bootstrap Varianten im Angebot > bs_Carousel und bs_Gallery.
Die Variante bs_Carousel habe ich Dir als Pull Request auf Git zukommen lassen (heute in der finalen Version).
Die Variante bs_Gallery noch nicht, würde natürlich auch gehen ...
Re: Modul catGallery - Upload Galerie
So sieht die Variante bs_Carousel im Backend aus:
Re: Modul catGallery - Upload Galerie
Und so sieht die Variante bs_Gallery im Backend aus:
- creativecat
- Beiträge: 1430
- Registriert: Mi 6. Feb 2013, 12:41
- Kontaktdaten:
Re: Modul catGallery - Upload Galerie
Danke für den Pull Request Habs in den Master übernommen! Dann teste ich das heute selbst noch mal und dann könnte das schon online gehen. Und ja, ich werde auf meiner Seite noch die neuen Varianten einbauen, damit man sieht, wie es im Frontend aussehen könnte/ sollte
EDIT: Copy&Paste-Fehler ist auch ausgebessert
EDIT: Copy&Paste-Fehler ist auch ausgebessert
Re: Modul catGallery - Upload Galerie
Immer gerne, ich freue mich ja wenn ich auch mal etwas dazu beitragen kann und nicht nur Eure Hilfe in Anspruch nehme ...
Einen Vorschlag möchte ich noch machen:
In der set_image.tpl können die Werte für resize_x und resize_y vorgegeben werden. Die meisten Varianten greifen auf die set_image.tpl der "default" Variante zurück, außer sliderpro, bs_gallery und bs_carousel, die haben "eigene".
So oder so macht es aber keinen Unterschied weil beim ersten Aufruf immer die Werte aus der inc/class.catgallery.php genommen werden:
Können der resize_x und der resize_y Wert nicht dort aus den InitOptions entfernt werden, so das die Werte aus der jeweiligen set_image.tpl greifen?
Die eigentlichen Werte sind auch in der "default" Variante anders als in den InitOptions. Nur wie gesagt ohne Auswirkung.
Noch ein Wort zu beiden Bootstrap Varianten (du hast doch beide im Pull Request bekommen oder?)
Beide brauchen die lib_bootstrap_4 um "vernünftig" auszusehen. Die Fonts-CSS und JS habe ich direkt in den Varianten verknüpft, die eigentliche bootstrap.css nicht. Die muß über das Template "kommen".
So soll es dann aussehen >> Carousel
Und so für >> Gallery mit Hoover Effekt
>> Gallery Modal (Lightbox)
Einen Vorschlag möchte ich noch machen:
In der set_image.tpl können die Werte für resize_x und resize_y vorgegeben werden. Die meisten Varianten greifen auf die set_image.tpl der "default" Variante zurück, außer sliderpro, bs_gallery und bs_carousel, die haben "eigene".
So oder so macht es aber keinen Unterschied weil beim ersten Aufruf immer die Werte aus der inc/class.catgallery.php genommen werden:
Code: Alles auswählen
protected static $initOptions = [
"variant" => "default",
"effect" => "random",
"random" => "0",
"animSpeed" => "500",
"pauseTime" => "4000",
"label" => "1",
"resize_x" => "781",
"resize_y" => "350",
"auto_play" => "1",
];
Die eigentlichen Werte sind auch in der "default" Variante anders als in den InitOptions. Nur wie gesagt ohne Auswirkung.
Noch ein Wort zu beiden Bootstrap Varianten (du hast doch beide im Pull Request bekommen oder?)
Beide brauchen die lib_bootstrap_4 um "vernünftig" auszusehen. Die Fonts-CSS und JS habe ich direkt in den Varianten verknüpft, die eigentliche bootstrap.css nicht. Die muß über das Template "kommen".
So soll es dann aussehen >> Carousel
Und so für >> Gallery mit Hoover Effekt
>> Gallery Modal (Lightbox)
- creativecat
- Beiträge: 1430
- Registriert: Mi 6. Feb 2013, 12:41
- Kontaktdaten:
Re: Modul catGallery - Upload Galerie
Also ich brauch initiale Werte im Code. Aus der Variante kann ich die eigentlich nicht ziehen - man muss tatsächlich einmal speichern, damit andere Werte gesetzt sind - insofern machen die anderen Werte aus der default-Variante keinen Sinn. Die sind da mal reingerutscht, sollten aber gleich mit den init-Werten sein.DG70 hat geschrieben: ↑Mo 12. Apr 2021, 18:26Können der resize_x und der resize_y Wert nicht dort aus den InitOptions entfernt werden, so das die Werte aus der jeweiligen set_image.tpl greifen?
Die eigentlichen Werte sind auch in der "default" Variante anders als in den InitOptions. Nur wie gesagt ohne Auswirkung.
Hab beide Varianten bekommen und in die Master übernommen. Hab sie gerade mal getestet, aber bei mir funktionieren die nicht. Die Bilder werden einfach untereinander weg angezeigt. Mein Template verwendet allerdings auch kein Bootstrap.
Ich hab die lib_bootstrap_4 installiert. Allerdings findet das Skript die bootstrap-icons.css nicht. Ich finde sie ebenfalls nicht
Die scheint nicht in der lib drin zu sein, oder?
EDIT: Ich hab gerade noch mal in dein Repository geschaut. Da fehlt tatsächlich der ganze font-Ordner. Ist deine Version auf GitHub noch aktuell?
Re: Modul catGallery - Upload Galerie
Die Fonts habe ich mit der V1.3 eingefügt, aktuell auf Github ist die V1.4. Im BC Addons Katalog ist noch die V1.2, aus Zeitgründen, wie shadowcat schrieb.creativecat hat geschrieben: ↑Di 13. Apr 2021, 07:00Ich hab die lib_bootstrap_4 installiert. Allerdings findet das Skript die bootstrap-icons.css nicht. Ich finde sie ebenfalls nicht
Die scheint nicht in der lib drin zu sein, oder?
EDIT: Ich hab gerade noch mal in dein Repository geschaut. Da fehlt tatsächlich der ganze font-Ordner. Ist deine Version auf GitHub noch aktuell?
Ich habe lange überlegt ob ich die bootstrap.css auch fest mit in die Varianten aufnehme, war (bin) aber unsicher wie die Priorität beim Laden sich auswirkt (?). z.B. wenn man im eigentlichen Template dann eine Bootswatch CSS verwenden möchte.
Alles klar, dann wäre es sinnvoll die Varianten sliderpro und beide bs_ dementsprechend zu ändern. Also die "default" set_image einzubinden.creativecat hat geschrieben: ↑Di 13. Apr 2021, 07:00Also ich brauch initiale Werte im Code. Aus der Variante kann ich die eigentlich nicht ziehen - man muss tatsächlich einmal speichern, damit andere Werte gesetzt sind - insofern machen die anderen Werte aus der default-Variante keinen Sinn. Die sind da mal reingerutscht, sollten aber gleich mit den init-Werten sein.DG70 hat geschrieben: ↑Mo 12. Apr 2021, 18:26Können der resize_x und der resize_y Wert nicht dort aus den InitOptions entfernt werden, so das die Werte aus der jeweiligen set_image.tpl greifen?
Die eigentlichen Werte sind auch in der "default" Variante anders als in den InitOptions. Nur wie gesagt ohne Auswirkung.
Könnte ich heute Abend kurz machen und dir einen weiteren Pull Request senden ...
Was hältst du davon die Werte vom Init und der default set_image auf ein gängiges Format zu ändern?
z.B. WVGA - 720x540 (4:3) oder SVGA - 800x600 (4:3)