l-uss / pizza-zadani Goto Github PK

První povinná část rozhodně splněná a druhá taky, takže uznáno a je pár tipů:

• v Header komponentě si mohla použít kontext místo posílání hodnot jako propsy

• pizza-zadani/src/components/ToppingsSelect/index.jsx

  Lines 17 to 21 in cb278de

  	const handleSelect = (index,checkedTopping)=>{
  	const newToppings = [...toppings];
  	newToppings[index].selected = checkedTopping;
  	setToppings(newToppings);
  	}

  tady si mohla zjednodušit funkci a neposílat do ní checkedTopping, stačilo by newToppings[index].selected = !toppings[index].selected;

• pizza-zadani/src/components/Check/index.jsx

  Line 21 in cb278de

  className={isEnabled()?"check":"check--disabled"}

  tady potřebuješ nechat i původní třídu check, protože obsahuje styly určující rozměry atp., takže className={isEnabled()?"check":"check check--disabled"}

• pizza-zadani/src/components/Check/index.jsx

  Lines 8 to 12 in cb278de

  	const isEnabled=()=>{
  	if(veganOnly)
  	return isVegan;
  	else return true;
  	}

  tohle je hodně krkolomná a zbytečná funkce. Stačilo by const isDisabled = veganOnly && isVegan; a pak otočit tu podmínku u className