Visa inte alltid subcategory-val

[Ejdamm]

Subbcategory-dropdown ska bara visas efter man har valt category. Kanske kan ändra hidden-attribut med javascript och sortera ut subcategories utefter vad man valt för category

[envetn]

Lekte runt lite och hittade en lösning: https://stackoverflow.com/a/20478291/5318606 Fungerar med jquery-3.4.1,

Blir inte så snyggt att blanda twig/js/html, men blir nog svårt att undvika. Yay or nay?

[Ejdamm]

Det ska inte vara några problem att blanda twig med js, och bootstrap kör redan med jqeury så det kan vi fortsätta med. Så om lösningen funkar är det yay

[envetn]

Pushar upp den ikv då så får du se hur det blev.

[Ejdamm]

Funkade ju bra. Skrev lite kommentarer i commiten och hittade en liten bugg i scriptet bara. Om man väljer defaultvalet "Choose category" så borde det återställas så alla subcategories visas igen. Dessutom behöver vi et "Choose subcategory"-val också.

Jag har läst diskussioner om var det är bäst att lägga sina scripttaggar och då pratar om själva koden och inte länken för det har jag kommenterat redan i commiten. Man ska ha det i antingen i head eller body och om man har det i body ska det vara längst ner. Här är en snubbe som rekommenderar att lägga funktioner som vi har i head https://stackoverflow.com/questions/38407962/when-to-use-the-script-tag-in-the-head-and-body-section-of-a-html-page

En fin effekt med att använda relationstabeller som vi gör i databasen är att när vi lägger till en listing skickar vi bara in subcategory_id och sen hämtar den category_id själv. Då slipper vi bry oss om att nån försöker lägga till Göteborgsvarvet som en skidtävling. Eller vi behöver ju bry oss om felhantering såklart. Alltså ge ajabaja-feedback till användaren och kanske göra en warninglogg.

Från början hade jag tänkt att subcategorydropdownen skulle var hidden fram tills man väljer en category. Men då kan ju stylen hoppa runt och sidan kommer inte funka utan javascript. Så det är bättre som det är nu.

[Ejdamm]

Jag insåg precis problemet med var man ska lägga scriptet när twig är uppdelat som det är men förhoppningsvis går det att lösa på ett bbra sätt

[envetn]

Ska fixa det. Av någon anledning missade jag helt din kommentar ovan när jag skapade pull-requeten, wops.

Tycket githubs gui är lite rörigt:P

[Ejdamm]

Tycket githubs gui är lite rörigt:P

Jag är beredd att hålla med