Jedna z moim pierwszych stron

Question

Witam, dziś przez cały dzień robiłem tą stronę Link Tutaj . pomimo że strona jest wizualnie słaba , zapewne pod względem kodu również... To chciałbym podziękować wam forumowicze za wsparcie na które zawsze mogę liczyć.

Proszę o wyciągniecie  największych gaf w kodzie abym mógł w przyszłości robić poprawne strony, pozdrawiam.

ps.

(świadomie nie robiłem nawigacji mobilnej )

Answer 1

Przed przeczytaniem chciałbym tylko poinformować że nie jestem znawcą i jest to tylko moje subiektywne zdanie :)

Więc według mnie strona reprezentuję się dość dobrze pod względem wizualnym (wiadomo trzeba by ją jeszcze dopracować, ale jesteś na dobrej drodze)

I u mnie pojawia się pytanko czy robiłeś ją tylko dla doświadczenia czy będziesz ją dopracowywał ponieważ jeżeli masz zamiar ją dokończyć to z tego co widzę będzie ona multi page (jeżeli się mylę to przepraszam) i czy nie lepiej zrobić z niej one page i po kliknięciu którejś z zakładek (np. skills) zjeżdżało nam po prostu na dół

 

Tak jak mówiłem jest to tylko moja opinia i nie mnie jest to oceniać bo pewnie znajdą sie osoby które się znają na tym bardziej ode mnie :)

Pozdrawiam!

Comments

czy nie lepiej zrobić z niej one page

a tak z ciekawości dlaczego uważasz, że one page to lepsze rozwiązanie?

Wiem, że dziś jest na to wielka moda, łał i w ogóle, ale mówiąc szczerze dla mnie wygodniejsze w obsłudze są strony "standardowe" (oczywiście nie rozmawiamy tu o SPA tylko o zwykłych stronkach).

Pytanko nie jako jakiś zarzut ale czysta ciekawość (chyba jestem już starej daty i jakoś nie mogę się przekonać do tych wszystkich nowych pomysłów designerskich :)

---

@Layoutowiec, Ciekawy pomysł, strona tylko dla doświadczenia.

---

Znaczy nie mam nic do stron multi page bo sam lubię wykorzystywać to u siebie w projektach tylko chodzi mi o to że jest to portfolio i moim subiektywnym zdaniem portfolia one page wyglądają troszeczkę lepiej

---

różnie to bywa, ale tak w ogóle to całkiem niedawno znajomek zmieniał robotę i jak z nim rozmawiałem to powiedział, że jako CV firma sama poprosiła go o przedstawienie jednego swojego projektu i dał im dostęp do jakiegoś CRMa na bitbucket i podobno nawet rozmowa kręciła się ogólnie wokół tego projektu.

Żadnego CV, żadnych milionów projektów itp. sam mówił, że był w lekkim szoku :)

Także chyba nie zawsze trzeba usilnie tworzyć mega portfolia, a może lepiej się skupić na jakimś konkretniejszym projekcie :)

---

Znaczy według mnie niektóre portfolia są dość fajne pod względem właśnie takich małych szczegółów, bo sam zajmuję się właśnie robieniem portfoli które później wystawiam na jakiejś stronce z layoutami dzięki czemu można sobie troszeczkę dorobić i nabiera się doświadczenia

Answer 2

Skoro nikt o kodzie jeszcze nic nie napisał to ja tak krótko to zrobię. Z tego co widzę strona jest mega prosta, ale: 

• Błędna struktura nagłówków, posiadasz więcej niż jeden h1
• Ikonki są w miarę ok, ale mimo wszystko lepiej dla semantyki użyć span zamiast i
• Nie stylujemy po id, wiąże się to ze specyficznością selektorów
• Co jakiś czas zdarzają Ci się taie potworu

  .main-nav ul li a:hover { // kod }
  

  Staraj się takie rzeczy upraszczać. W tym miejscu polecam Ci zaineterować się jakąś metodyką CSS ( a może nawet architekturą) np. BEM. Pozwala to inaczej spojrzeć na kod i jednocześnie poprawić jego strukturę.

• Responsywność pozostawia wiele do życzenia. Po pierwsze nie wrzucaj wszystkiego do jednego worka width<1000px, tylko rozbij na kilka mniejszych czyli telefony <768 i tablety <1024 itd. Oprócz tego warto częściej stosować jednostki takie jak rem, em, vw, vh zamiast tradycyjnych px. 

• Masz nadmiarowy kod np. dwa razy 

  .banner { //kod }
  
  ...
  
  .baner{ //kod }

   

Comments

Wszystko jest zrozumiałe, stalowałem po #id bo później się pogubiłem w kodzie i nie mogłem się dostać po selektorach :) , co do 

	
.main-nav ul li a:hover { // kod }
poprawnym zapisem był by ?
main-nav a:hover {
}

Dziękuję za pomoc, obiecuje sie poprawić! 

ps.

nie chce zakładać oddzielnego tematu może ktoś mi podpowie jak się nazywa efekt który pozwala zrobić łamana poziomą kreskę jak by diva?