Ocena strona

pytanie zadane 20 sierpnia 2016 w HTML i CSS przez Boshi VIP (100,240 p.)

Mógłbym prosić o ocenę? głównie design- choć moje umiejętnosci doboru koloru i graficzne są żadne ;) Jak ktoś ma chwilę, to kod (spaghetii wymieszane z grzybami) też może ocenić. Będę wiedział na przyszłość co zmienic, co dodać.

Znajomy prosił o bardzo prostą stronkę bez bajerów.

Na razie działą galeria, strona głowna i formularz. Ogólnie strona jeszcze testowa, bardzo testowa

http://janpokoje.hekko24.pl/

komentarz 20 sierpnia 2016 przez xmentor Nałogowiec (49,520 p.)

Zastanawiam się czy kolejność w menu nie powinna być na odwrót.

W arkuszu stylów jest coś takiego:

Menu
#menu, #menu ul {
    margin: 0;
    padding: 0;
    list-style: none;
}

U Ciebie jedynie jest ul o id = menu, więc nie rozumiem tego: #menu ul

I przez to:

Menu

nad

#menu, #menu ul

nie styluje tego elementu, sprawdź sam klikająć na to menu ppm - > zbadaj element.

Taką prostą karuzele(slider) można zrobić w samym CSS i myśle, że to fajne rozwiązanie.

No i styluj po klasach.

komentarz 20 sierpnia 2016 przez Eimens Maniak (69,240 p.)

RWD lipa

hover w menu mnie przeraził dalej nie oceniam :o

komentarz 20 sierpnia 2016 przez Boshi VIP (100,240 p.)

@Patyl, niby czemu hover przeraził ?

komentarz 21 sierpnia 2016 przez Eimens Maniak (69,240 p.)

Tragicznie to wygląda, pojeździłeś myszką po menu?

komentarz 21 sierpnia 2016 przez Boshi VIP (100,240 p.)

Fakt, coś tam pogrubia po zjechaniu, zamiast bordera before użyje najwyżej.

komentarz 21 sierpnia 2016 przez Eimens Maniak (69,240 p.)

możesz dla li dać takiego samego bordera co po :hover ale zamiast koloru daj transparent :) wtedy animacja płynnie działa :p

komentarz 21 sierpnia 2016 przez Boshi VIP (100,240 p.)

Ok, przyjrzę się, dzięki ;)

4 odpowiedzi

odpowiedź 20 sierpnia 2016 przez IBB Gaduła (3,020 p.)
wybrane 20 sierpnia 2016 przez Boshi

Najlepsza

Ok, to bez zaglądania w kod - strona główna:

pierwsza rzecz, która rzuciła mi się w oczy, to że nigdzie na stronie głównej nie jest wspomniane, gdzie owe 'Pokoje Jan' się znajdują - nad morzem? w górach? w Polsce? w Australii? To jest dość podstawowa informacja. Owszem, na dole jest mapka, ale jest na tyle powiększona, że jedyna widoczna informacja dotycząca lokalizacji to 'Osiedle Słoneczne 8'
czcionka, którą używasz do nagłówków ('Pokoje Jan', 'Opis') jest jak dla mnie średnio czytelna
menu wygląda dość niechlujnie jeśli chodzi o wyrównanie elementów. Pierwszy link ma zbyt dużo odstępu po lewej stronie. Pionowe linie oddzielające linki dotykąją górnej granicy menu, ale już nie dolnej
zamieszczenie linku do strony głównej na samym końcu menu jest dość dziwne, obowiązującą praktyką jest zamieszczenie go jako pierwszego linku w menu (bądź pominięcie go i użycie do tego celu loga)
dodaj przycisk umożliwiający zatrzymanie slidera (albo nie zmieniaj automatycznie slajdów)
dodaj padding wokół głównej treści (w sekcji 'Opis') - tekst jest zbyt ciasno wciśnięty pomiędzy górny a dolny border
linko-przyciski ('Więcej', 'Kontakt') mają taki sam wygląd jak nagłówki panelów ('Atrakcje', 'Dojazd'). Skąd użytkownicy mają wiedzieć, które z tych elementów są interaktywne, a które statyczne, jeśli wyglądają dokładnie tak samo?
tekst linku 'Więcej' - więcej czego? Warto by go zmienić na nieco bardziej opisowy
linki zamieszczone na ciemniejszym tle (np. w stopce) są zbyt ciemne on hover/focus

Mobile:

słowo 'Menu' w przycisku jest zbyt wysunięte w prawo
warto by dać trochę odstępu pomiędzy dolną granicą sekcji 'Opis' a kolejną sekcją ('Atrakcje')
sekcje 'Dojazd' i Kontakt są ucięte z lewej strony

Tablet:

powyżej pewnej szerokości sekcja 'Atrakcje' wydaje się zbyt szeroka i długa. Albo wyświetl ją w lewej kolumnie obok sekcji 'Dojazd', albo wyświetl kilka zdjęć w jednym rzędzie, a nie jedno pod drugim
sekcja 'Dojazd' jest wyświetlona po prawej stronie, zostawiając dużą lukę po lewej stronie

odpowiedź 20 sierpnia 2016 przez Comandeer Guru (600,810 p.)

Ale tę faviconkę H5BP to byś zmienił… :D

<!--[if lt IE 7]>
<html class="no-js lt-ie9 lt-ie8 lt-ie7" lang=""> <![endif]-->
<!--[if IE 7]>
<html class="no-js lt-ie9 lt-ie8" lang=""> <![endif]-->
<!--[if IE 8]>
<html class="no-js lt-ie9" lang=""> <![endif]-->

Potrzebne Ci te klasy? Jak nie, zostaw jedno html i tyle. A jak potrzebne, to uzupełnij [lang].

Potrzebny Ci Modernizr?
```
<nav id="menu-trigger">Menu</nav>
```
To button.
```
<header>
                    <h1 class="logo">Opis</h1>
                </header>
                <section id="content">
```
To raczej powinien być nagłówek tej sekcji. I samo h1 w zupełności wystarczy. Inna rzecz, że to section to bardziej main a nagłówek h2 (struktura na samych h1 została de facto usunięta z HTML5).
Za obrazki bez [alt] powinno się wykłuwać oczy :)