Proszę o opinie strony www

pytanie zadane 7 marca 2016 w Nasze projekty przez NowyProgramista Użytkownik (950 p.)

Witam

chciałbym prosić o szczegółowe opinie ewentualnie poprawki stron internetowych napisanych przez Nowego Programistę.

http://strona-niedlugo.esy.es/WWW----/

http://strona-niedlugo.esy.es/WWW-----/

EDIT: Wiem wiem nie powinienem skorzystać z id w przypadku 1 strony przepraszam ale nie chce mi się tego już zmieniać to już się nie powtórzy.

6 odpowiedzi

odpowiedź 7 marca 2016 przez Ivan Maniak (60,650 p.)
wybrane 7 marca 2016 przez NowyProgramista

Najlepsza

Nawet muszę przyznać, że strona wygląda "dobrze". Może nie jest to najlepszy design, ale mi nawet przypadł do gustu.
Ale, żeby nie było za miło to parę "sugestii" na temat kodu.

Każde zdjęcie <img> MUSI zawierać atrybut ALT,
Rozumiem, że to po COŚ po lewej stronie to nawigacja. Nawigacja "powinna" być listą. [Jeżeli już chcesz używać super nowych tagów z HTML5 to użyj <nav>]. Nawet nie będę oceniał tego co tam jest. Ale article, nie służy do "zamknięcia" linku, a tym bardziej aside. Aside to odrębna część strony, a nawigacja nią nie jest.
Nie używaj znaczników <br>. W końcu po coś jest ten margines oraz padding
Jeżeli stylizujesz jakiś element strony to rób to za pomocą klasy, a nie ID. [Nie jest to błąd ale..] Nawet jeżeli jesteś przekonany, że nigdy nie zrobisz drugiego takiego samego elementu to "nigdy nie mów nigdy
A no i ty tak zrobiłeś, zrobiłeś 100 ID które mają takie same właściwości [Chodzi mi o to coś z lewej strony, bo nawigacja to na pewno nie jest. Patrząc na kod oczywiście]
Używasz <aside>, <article> [Pominę, że źle], słyszałeś, że jest coś takiego jak <footer>?

komentarz 8 marca 2016 przez HaKIM Szeryf (87,590 p.)

Nie używaj znaczników <br>. W końcu po coś jest ten margines oraz padding

Równie dobrze mogę napisać:

Nie używaj margin i padding, od czegoś są te <br>.

komentarz 8 marca 2016 przez makoso Mądrala (7,380 p.)

i chyba nie <br> a <br /> ? ;)

komentarz 8 marca 2016 przez Ivan Maniak (60,650 p.)

<br>, <br/>
Nie ma różnicy w HTML5

komentarz 9 marca 2016 przez makoso Mądrala (7,380 p.)

Dobrze wiedzieć, ktoś kiedyś mnie opierniczył za zły zapis :O

odpowiedź 7 marca 2016 przez PietaS Bywalec (2,570 p.)

odpowiedź 7 marca 2016 przez radek024 Szeryf (77,160 p.)

Poza tym co dodał Ivan:

każde zdjęcie musi posiadać artybut alt, ale nie musi(w pewnych przypadkach nie jest nawet zalecane) być wypełniony treścią.
ID jest do stylizowania niepowtarzalnej rzeczy, natomiast klasa - do wielu. Powszechnie jednak całość dzieli się na dwa obozy: wg jednych powinno pisać się w CSSie zarówno klasy jak i ID, wg drugich ID powinno się używać w JSie jak uchwyt. Na dodatek, każde ID menu ma identyczne wartości, więc jak najbardziej powinno być to klasą.
tagi używasz, ale źle z ich przeznaczeniem. Zobacz na to: http://tutorials.comandeer.pl/html5-blog.html - treść słowna strony(mam na myśli to po prawej) powinna być ile akapitów tekstu tyle tego tagu. Sam tekst natomiast lepiej się czyta, gdy jest wyjustowany.
co jeżeli tekstu w #content będzie więcej niż zakłada znacznik height? Należy użyć min-height, aby w razie czego pole było odpowiednio dłuższe.
masz znacznik style w head, który jest pusty.

Całość bym poprawił :P

odpowiedź 8 marca 2016 przez Mateusz11 Pasjonat (22,910 p.)

Strony w nazwie nie powinny mieć spacji.

Litery w stopce trochę duże.

.KT{
color: #000000;
text-decoration: none;
transition: 0.2s;
}

.KT:hover {
background-color: #89A6C4;
color: #fff;
}

#NY{
	float: left;
	height: 40px;
	background-color: #89A6C4;
	text-align: center;
	width: 161px;
	margin-right: 2px;
	margin-top: 19px;
	font-family: 'Playfair Display', serif;
	font-size: 18px;
	padding-top: 15px;
	padding-bottom: 5px
}  

.NY{
color: #000000;
text-decoration: none;
transition: 0.2s;
}

.NY:hover {
background-color: #89A6C4;
color: #fff;
}

Raz używasz klas a raz id :(

Pozdrawiam