quincy-tromp / educom-webshop-oop

0 stars 0 forks source link

Waarom heb je hier gekozen voor een dropdown i.p.v. een input type="number" #16

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

Je hebt in de cart gekozen voor een dropdown, https://github.com/quincy-tromp/educom-webshop-oop/blob/3e37179c83b5671c6bddda3bf6608a9c8d0713ad/cart.php#L87-L92 Maar de amount staat bovenaan in de dropdown met daaronder de nummers 1-10, dus er kan staan bijv 4,1,2,3,4,5,6,7,8,9,10. Als ik er nu 20 wil bestellen, dan kan dat niet. Beter zou het zijn om een <input name="amount" type="number" value="'.$amount.'" min="0"> te gebruiken.
of als je een dropdown wilt behouden om dan door de nummers 1 t/m ($amount+10) te lopen en als het $quantity overeenkomt met de $amount deze optie in HTML te 'selecteren'.