• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Ocena jakości kodu i funkcjonalności strony.

VPS Starter Arubacloud
+2 głosów
318 wizyt
pytanie zadane 19 marca 2017 w Nasze projekty przez wafel34 Nowicjusz (180 p.)
Witam wszystkich,

To mój pierwszy post, więc chciałem się przywitać i życzyć dobrego tygodnia:)

W tym pierwszym poście chciałem prosić o ocenę jednego z moich projektów, który niedawno ukończyłem.

Generalnie uczę się front-endu od jesieni, ale przez cały ten czas pracowałem sam i zostawiałem swoje projekty tylko sobie. Czuję, że przez takie podejście dużo tracę, bo nie mam doświadczenia aby wyłapać pewnie masę błędów i standardów/wzorców, więc teraz chciałbym abyście ocenili co poszło okej, a co nie w poniższym projekcie.

Głównym celem było przećwiczenie responywności bez użycia frameworków i zrobienie pierwszego projektu,który składałby się więcej niż z jednej strony. W podfolderze /zabawa-webiste zamieściłem też stronkę z użyciem dynamicznego przeładowania stron w Ajax, choć nie wiem czy wyszło to idealnie.

http://zabawa-countryside.eu.pn/

Dzięki z góry za feedback!

4 odpowiedzi

+1 głos
odpowiedź 19 marca 2017 przez xmentor Nałogowiec (49,520 p.)
edycja 19 marca 2017 przez xmentor
  • Brak otwierającego znacznika html z atrybutem lang.
  • skrypt wypadałoby przenieść przed </body>
  • <i class="icon-info-outline"></i>
    itd. - https://comandeer.github.io/blog/a11y/html-css/2017/02/28/o-ikonkach-slow-kilka.html
  • raczej bym unikał używania atrybutu target z wartością _blank dla linków:  https://css-tricks.com/use-target_blank/
  • Puste znaczniki, które nie niosą za sobą żadnej treści, nie wiadomo od czego, co robią:
    <button id="attractions-btn1"></button>
    <button id="attractions-btn2"></button>
    <button id="attractions-btn3"></button>
    
    <aside class="fixed-arrow" id="fixed-arrow">
      <div class="arrow"></div>
    </aside>
  • przypisz style z :hover do :focus - poruszam się klawiaturą po stronie i nie wiem gdzie jestem.
0 głosów
odpowiedź 19 marca 2017 przez Kamil Naja Nałogowiec (27,330 p.)
Strona skromna, ale nie widzę większych błędów. Dałbym nieco mniejsze logo. W sekcji Author sporo błędów w tekście.Na 320 px szerokości elementy stopki nachodzą na siebie. Na niektórych podstronach jest nieprawidłowa struktura tagów hx - brakuje np h2 i h3, od h1 przechodzi od razu do h4.

Moim zdaniem, jest dobrze.
0 głosów
odpowiedź 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Brakuje fav-icon

Skrypty JS wrzuć przed </body>

Tyle zauważyłem, nie zagłębiałem się za bardzo :p
0 głosów
odpowiedź 20 marca 2017 przez jaca121212 Nałogowiec (40,760 p.)

Zamiast  twojego codu użyj tego ładniej wygląda

nav a[aria-haspopup="true"]::after{
	content: "\25bc";
	display: block;
	position: absolute;
	width:50%;
	
	top: 25%;
	left: 25%;
	color:black;
	transform: translate(-25%, -25%);
	transform:rotate(0deg);

	}

 

Podobne pytania

+2 głosów
1 odpowiedź 299 wizyt
pytanie zadane 25 lipca 2017 w Nasze projekty przez niko404 Użytkownik (910 p.)
0 głosów
5 odpowiedzi 629 wizyt
0 głosów
2 odpowiedzi 176 wizyt
pytanie zadane 26 października 2017 w PHP przez Sławek Obywatel (1,270 p.)

92,454 zapytań

141,263 odpowiedzi

319,099 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...