Code Review - co poprawić

pytanie zadane 15 czerwca 2016 w HTML i CSS przez xmentor Nałogowiec (49,520 p.)
zamknięte 18 czerwca 2016 przez xmentor

Link do strony: http://kkondratowicz.esy.es/cr/

Proszę o wytknięcie nawet najmniejszego błędu - szczególnie chodzi mi o semantyke kodu.

komentarz zamknięcia: Uzyskałem odp

2 odpowiedzi

odpowiedź 15 czerwca 2016 przez Comandeer Guru (601,110 p.)

<meta name="keywords" content="">
		<meta name="description" content="">

Rozumiem, że to tylko w demo? ;)

<p>WORKING<br>WITH design LAB</p>
<p>We've been helping startups launch and grow their businesses. We help<br>you to fill the gap between development,<br>design and        product management.
</p>

Nie podoba mi się to br. Nie wymuszałbym przełamania tekstu wyłącznie ze względów estetycznych.

```
<a href="#main"><img class="bottom" src="img/bottom.png" alt="BOTTOM"></a>
```
Tutaj dodałbym jeszcze [title] do linku, a sam [alt] zmieniłbym na "Scroll to the main part of the page" czy coś tego typu.
```
<header><h2>About us</h2></header>
```
Tutaj starczy samo h2.
To, co masz jako fieldset > legend powinno być section > h3. A jeszcze lepiej: div z ładnym ::before (bo ten "nagłówek" to duplikacja). fieldset służy do grupowania pól formularza.
No i dlaczego nie ma tam akapitów?
Tam, gdzie masz "What we do?" poszczególne divy to sekcje.
```
<a href="#header"></a>
```
Pusty element = bezsensowny element.

komentarz 16 czerwca 2016 przez xmentor Nałogowiec (49,520 p.)

Rozumiem, że to tylko w demo? ;)

Tak :)

Wielkie dzięki za wskazówki!

odpowiedź 15 czerwca 2016 przez radek024 Szeryf (77,160 p.)

Nie rozumiem zastosowania tutaj fieldsetu - tego się używa w formularzach, czytaj więcej tutaj.
Prace powinny mieć wypełniony alt - nie jest to element dekoracyjny. I czy to nie powinna być lista?
br też średnio - mogłeś zrobić po prostu oddzielne paragrafy,
nie stylizuj po tagach
IMO nadużywasz ilości tagów, np.: w main, w article na pewno z mniejszą ilością dałbyś radę to zrobić równie dobrze.

Ogółem: jest całkiem ok, poza tym fieldsetem.