Skip to content

Comments

fix masonry bug caused by #123#212

Merged
Elfangor93 merged 1 commit intoJoomGalleryfriends:mainfrom
reilldesign:Masonry-Bug
Apr 2, 2024
Merged

fix masonry bug caused by #123#212
Elfangor93 merged 1 commit intoJoomGalleryfriends:mainfrom
reilldesign:Masonry-Bug

Conversation

@reilldesign
Copy link
Collaborator

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

@MrMusic please test the pull request

This adds back the necessary information that was deleted in pull request JoomGalleryfriends#123.
@MrMusic
Copy link
Member

MrMusic commented Mar 20, 2024

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

@eumel1602
Copy link
Collaborator

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
Copy link
Member

MrMusic commented Apr 1, 2024

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
Copy link
Collaborator

eumel1602 commented Apr 1, 2024

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
Copy link
Member

Thank you...

@Elfangor93 Elfangor93 merged commit 96eb5c3 into JoomGalleryfriends:main Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants