Prośba o ocenę kodu

Cześć,
starałem się dostać na staż i miałem wysłać swoje rozwiązanie zadania związane z tworzeniem strony o smoothie. Niestety się nie dostałem, bo miałem za dużo błędów. Miałby ktoś chwilkę czasu zerknąć na kod i podać jakieś sugestie. Myśle, że skoro to miała być strona o smoothie to na pewno powinienem inny wygląd dać, takie bardziej 'owocowy'.Z góry dzięki.

https://github.com/quarhodron/quarhodron.github.io/tree/master/SmoothieDelight

A jaka była dokładnie treść zadania? Jak na staż chciałeś się dostać... to powiem że widziałem gorsze dzieła, nawet przy aplikowaniu na regulara. A tutaj.... w formularzu w "action" dałbym raczej # - tak samo przy linkach. Reszta na pierwszy rzut oka wydaje się ok.

A mogę podać treść zadania? To nie jest jakoś chronione prawami?

Trzeba zawrzeć benefity tegoż produktu i marki(hmm... ja chyba nic nie mam wspomniane o marce) oraz zebrać maile do newslettera. Mogłem na swojej stronie wykorzystać te hasła, które są w suwaku oraz obrazek butelki.

Prawdopodobnie chcieli aby ta strona wyglądała mniej więcej tak: https://www.gowebbaby.com/wp-content/uploads/2011/12/smoothie.png
W sumie to mogłem stworzyć jakieś zakładki w stylu about us, contact ale nie wiedziałem co tam napisać. Może tam właśnie powinienem dać tak jak na tej stronie wyżej, lorem ipsum i tak dalej

na staż i masz stronę robić, jacyś janusze wyzysku chyba...

Popieram, możliwe że trafiłeś do janush-zone. A co chodzi o zadanie to może nacisk był kładziemy na interakcje? czyli te zbieranie maili do newslettera. Staż jako frontend? Może zabrakło jakiegoś angulara czy jQuery?

Nie, to nie firma janusza. Korpo z całkiem dobrymi opiniami.
Miałem jeszcze opisać jak podszedłem do tematu SEO i accessibility, więc napisałem coś takiego, może jakieś błędy gramatyczne popełniłem.
Nie mieli nic o jQuery albo o Angularze w opisie swojej oferty.

Utilized ways to improve SEO:
-filled metadata with a few necessary things like title, description,
-keyword ‘Smoothie Delight’ included in the title, description and header,
-images provided with an alternative text,
-optimized images to increase page load speed,
-easy to read content for better readability,
-clear navigation to reduce bounce rate,
-installed social sharing buttons,
-mobile-friendly website.

Utilized ways to improve accessibility:
-font size is big enough,
-used highly contrasting colours for text and background,
-images with an alternative text,
-the website has a language assigned.

niby tak, ale z założenia staż oznacza, że dopiero się uczysz więc wymagania są po prostu niesmaczne

Ogólnie to ciężko jest znaleźć jakieś praktyki/staż w front-endzie, więc myślę, że mają bardzo dużo chętnych, więc muszą jakoś ich przefiltrować.
Chciałbym podnieść swoje umiejętności a nie chce mi się klepać stronek bez sensu, mogę gdzieś znaleźć jakiś otwarty projekt i tam pod opieką kogoś doświadczonego pisać?

Trochę nie na temat: A jakie masz doświadczenie? Może dałbyś radę gdzieś aplikować jako "junior" po prostu...? Ew. po douczeniu się kilku rzeczy (np. ze znajomością podstaw angulara/innego modnego frameworka zaszedłbyś dalej).

Na temat: wrażenia to nie robi, ale działa. Wiele zależy od tego jakie są wymagania, ale na juniora by starczyło, tym bardziej na staż.