JoomGalleryfriends / JG4-dev

Development repository for JoomGallery v4.x
GNU General Public License v3.0
10 stars 6 forks source link

fix masonry bug caused by #123 #212

Closed reilldesign closed 3 months ago

reilldesign commented 3 months ago

This adds back the necessary information that was deleted in pull request #123.

@MrMusic please test the pull request

MrMusic commented 3 months ago

Works as before. I have tested this item ✅ successfully. Thank you very much.

eumel1602 commented 3 months ago

mit dem test von Alwin, hat sich der PR erledigt??? wenn nicht, bitte genaue anweisung, da in dem PR 123 für mich nicht mehr erkenntlich ist was das $ genau für einen fehler gemacht hat... , bzw was zu testen ist... ansonsten bitte pr schließen?

MrMusic commented 3 months ago

mit dem test von Alwin, hat sich der PR erledigt???

Jeder Test ist hilfreich.

Das Problem: Mit der Alpha2 sieht es aktuell so aus (Lerraum unterhalb der Bilder einer Spalte): masonry-mit-bug

Es sollte aber so aussehen (Bilder sollten ohne Leerraum untereinander anschließen):

masonry-ohne-bug

eumel1602 commented 3 months ago

Ok ! Alpha 2 installiert, alles erstellt und fehler gesehen. PR drübergebügelt und diese speziellen Fehler im FE nicht mehr gefunden bzw wurden der Effekt nun angewandt. J.4.4.3 / PHP 8.1.19 / Datenbankversion 5.7.25 / nur Onlineseite getestet / WIN rechner

Elfangor93 commented 3 months ago

Thank you...