LucaScorpion
commented
3 years ago Bedankt voor je feedback!
Logisch, ik verwachtte idd wel dat er een reden zou zitten achter alles single-file :grin:
Aan de andere kant kan ik me wel voorstellen dat anderen losse files fijner vinden editen?
dit was voor mij inderdaad de reden om het los te trekken, en ik vind het zelf ook gewoon netjes om een html file te hebben met markup, css met style, etc... Al ben ik me ervan bewust dat dit natuurlijk een en al persoonlijke voorkeur is.
(de minuscule stukjes JavaScript zitten ook inline)
Yeahh, ik overwoog dat ook los te trekken, maar gezien hoe extreem weinig dat op het moment is zag ik daar weinig voordeel. Mogelijk dat dat logischer wordt als het groeit.
En er was altijd wel een punt gekomen dat het moet, want als we bijvoorbeeld #9 (vertalingen) of #6 (designs verbeteren) zouden oppakken zal dit waarschijnlijk sowieso nodig zijn?
Dat verwacht ik inderdaad ook. (Of misschien dat het niet nodig nodig zou zijn, maar ik denk dat als we meer gaan toevoegen het steeds lastiger wordt om alles single-file te houden :sweat_smile: )
Waar ik wel echt voor wil vechten met deze repository is dat het zo lang als mogelijk (near) zero-dependencies blijft.
Helemaal mee eens. Ik moet toegeven dat ik wel een beetje zat te spelen met de gedachte om Sass toe te voegen, maar gewoon de repo kunnen clonen, de html openen en it just works heeft zo veel voordeel dat dat soort dingen het (atm) niet waard zijn. Zelfde voor JavaScript, met vanilla JS kan je meer dan genoeg voor wat we nodig hebben met deze pagina.
Bottom line: mijn aanname hier is dat het meerendeel van mensen het fijner vindt werken met de opgespliste files, dus ik merge hem as-is :) Al waardeer ik de nostalgie wel heel erg :smiling_face_with_tear:
(Het lijken zo veel veranderingen, maar de commits zijn allemaal klein, daar kan je zien wat ik precies veranderd heb aan de CSS)