[Code Review] jQuery shopping cart

Question

W ramach treningu napisałem prostą aplikacje w rodzaju to do list. Prosiłbym o drobny przegląd kodu. Każda rada mile widziana :)  

Github
Demo

Answer 1

Działa spoko, ale naucz się korzystać z bibliotek CSS, bo użyłeś Materialize, ale udało ci się to zrobić dość nieumiejętnie. Nie wiem, co chcesz potem robić, ale jeśli front-end, to musisz nad tym popracować. Poszukaj w internecie gotowych szablonów i przykładów stron internetowych i np. próbuj napisać podobne, albo po prostu się nimi inspiruj tworząc własne. Jeśli będziesz się tym zajmował, przez dłuższy czas, to w końcu dojdziesz do takiego momentu, że będziesz wiedział co wygląda dobrze, a co nie.

Comments

Dzięki. Ogółem już trochę zakodowałem szablonów, ale średnio mam oko do tworzenia designu, a tym bardziej spędzania na tym czasu. Wolałem na szybko wrzucić materialize :)

---

Mógłbyś jeszcze napisać co rozumiesz poprzez nieumiejętne użycie Materialize?

---

Użyłeś Materialize, żeby twoja aplikacja wyglądała dobrze, a osiągnąłeś efekt odwrotny do zamierzonego.

---

Czyli tylko o tego desing chodzi. Możesz powiedzieć co Ci się w tym nie podoba?

---

Kiepsko dobrane kolory, słaby wygląd, słaba czytelność.

---

ok, niech będzie :)

Answer 2

Czemu mogę ściągnąć pustego PDFa? Imo przydałby się jakiś ładny timestamp w nazwie pliku.

Comments

Wiesz, jakoś nie zwróciłem uwagi na pustego PDFa. Założyłem że użytkownik wrzuci coś do listy przed wygenerowaniem pliku. Dobry pomysł z timestamp. Poprawie te dwie rzeczy.