search

cmsimple-xh / xhshop

A simple shop for CMSimple_XH
GNU General Public License v3.0
4 stars 3 forks source link

Add hi_fancybox to catalogue images in catalog.tpl? #211

Closed manu37 closed 3 years ago

manu37 commented 4 years ago

Frage an die xhshop Cracks: Ist es möglich, die hi_fancybox auch bei den preview Bildern im catalog.tpl anzuwenden? Ich hatte gehofft, dies im productDetails.tpl abzugreifen. Aber so einfach ist es wohl nicht. Antworten gerne willkommen.

frase-git commented 4 years ago

Darüber hat es eine lange Diskussion gegeben (https://github.com/cmsimple-xh/xhshop/issues/28 und https://github.com/cmsimple-xh/xhshop/issues/40). Denn ursprünglich war das genau so.

An welcher Stelle Christoph dann letztendlich geändert hat, weiß ich jetzt nicht. Mal sehen, ob ich das noch finde.

manu37 commented 4 years ago

Anscheinend hab Ihr hier diese Möglichkeit definitiv ausgetrieben, schade. Aber wenn doch alles so schön template-basiert ist, sollte das doch möglich sein. Im template habe ich Zugriff auf die FrontEndController Instanz. Aber von da ab verliert sich die Spur, wie ich zum image link komme.

frase-git commented 4 years ago

Vielleicht kommst du mit dieser Funktion weiter? Ich verstehe Code nicht ansatzweise :(

manu37 commented 4 years ago

Ja das wäre schön, aber an den href resp. an das Detailbild komme ich nicht ran. Zumindest im catalog.tpl in den products Eigenschaften nicht.

Ich verstehe Code nicht ansatzweise :(

Na ja. OOP und Abstrahierung ist ja ok. Aber komplett ohne Dokumentation ist das schon ein Irrgarten.

Ich finde den xhshop ein irre gutes plugin, bin aber in den templates etwas enttäuscht, wie eingeschränkt auf die dynamischen Daten zugegriffen werden kann. Aber das wurde wohl aus guten Gründen so designed und ist gut so.

Mein Vorschlag wäre: Wenn keine Detailseite, aber ein Detailbild vorhanden ist, dann im Katalog das previewbild mit Detailbild mit zoom verlinken. Das wäre dann irgendwo hier zu erweitern, oder ein zusätzliche Product Eigenschaft ('preview with zoom') mitzugeben.

frase-git commented 4 years ago

Mein Vorschlage wäre: ...

Schön, da wirst du aber auf cmb warten oder es selbst machen müssen. Bei mir reicht's dazu nicht ;-)

frase-git commented 4 years ago

Sorry, hatte den falschen Button erwischt und ge"closed".

manu37 commented 4 years ago

Hier mein PR

frase-git commented 4 years ago

Hier mein PR

Super. Funktioniert bei mir einwandfrei.

manu37 commented 3 years ago

Da hier kein BC Break zu erwarten ist, würde ich diese Erweiterung gerne im 1.1 sehen.