FriendsOfREDAXO / Modulsammlung

Ermöglicht es leicht und schnell, einige REDAXO-Module zu installieren.
MIT License
64 stars 15 forks source link

Kleiner css Verschreiber bei parallax #105

Closed Geri2017 closed 4 years ago

Geri2017 commented 5 years ago

Bildschirmfoto 2019-07-08 um 09 14 41

olien commented 5 years ago

Danke. werde ich fixen (wobei ein PR willkommen ist :-))

Geri2017 commented 5 years ago

Ich habe schon nach dem Code gesucht, aber nicht gefunden :-(

skerbis commented 5 years ago

@Geri2017 https://github.com/FriendsOfREDAXO/Modulsammlung/blob/674d7cfad531e0cb1e67a66409bff028af33249a/lib/module/0105_parallax_skrollr/styles_scss.inc#L1

Die Volltextsuche in Github hilft Dir auch dabei

olien commented 5 years ago

Wobei die Class im Output auch so heisst. Also wenn dann beides korrigieren :-))

https://github.com/FriendsOfREDAXO/Modulsammlung/blob/674d7cfad531e0cb1e67a66409bff028af33249a/lib/module/0105_parallax_skrollr/output.inc#L51

Geri2017 commented 5 years ago

WTF? Bildschirmfoto 2019-07-08 um 13 10 35

skerbis commented 5 years ago

Kurz oben links auf Master stellen

Geri2017 commented 5 years ago

Danke für den Tip mit master. Man lernt jeden Tag dazu :-) Alles erledigt!

skerbis commented 5 years ago

Du kannst das issue automatisch closen lassen wenn du im PR fixes link zum issue eingibst. Dann wird das issue geschlossen, sobald gemerged wurde

olien commented 5 years ago

Hallo @Geri2017 ich habe die PRs gemerkt. Danke dafür. Um es rund zu machen muß noch

Magst Du das durchspielen oder soll ich das machen (dann aber frühestens am WE :-))

LG Oliver

Geri2017 commented 5 years ago

Versionsnummer -> erledigt Eintrag im Changelog -> erledigt neues Release gemacht und neue Version in den Installer geladen ???? Keine Ahnung wie man das macht, aber ich lerne gern dazu :-)

olien commented 5 years ago

Top. Die PRs kannst Du gerne übernehmen...

Der Rest steht hier: https://github.com/FriendsOfREDAXO/friendsofredaxo.github.io

olien commented 4 years ago

Ist das erledigt? dann kann das hier zu?

skerbis commented 4 years ago

scheint ja so