Ocena strony szkoły z poziomu front endu #2 [ankieta]

pytanie zadane 10 lipca 2016 w Nasze projekty przez Adrian1999 Nałogowiec (34,570 p.)

Witam wszystkich serdecznie, możliwe że ktoś przypomni sobie przeglądając tą stronę że taki temat już był. Był ale był dawno, wpuściłem od danego czasu kilka modyfikacji i myślę już że strona z poziomu wyglądu jest zakończona. Jak ktoś śledzi moje posty, zauważył ktoś że pracuje nad back endem moje pytanie jest takie, co mógłbym zmodyfikować oraz dodać by strona była w pełni funkcjonalna i nie sprawiała problemów ze strony wyglądu. Oraz co wy byście dodali ze strony estetycznej, by wyglądało ładniej dziękuje z góry. Dodaję ankietę w kwestii oceny strony, by wiedzieć co myśli o tym szersze grono odbiorców bo zawsze wypowie się garstka(jeszcze do tych co wybiorą opcje "Jest okej, ale jeszcze bym coś dodał") wypowiedzcie się i napiszcie dokładnie czy macie coś na myśli.

http://niedoswiadczeni.pl/index.php#

Możliwe odpowiedzi:

Bardzo fajne (2 głosów, 5%)

Całkiem spoko (1 głos, 3%)

Jest okej, ale jeszcze bym coś dodał (8 głosów, 21%)

Strona odpychająca, większość do zmiany (28 głosów, 72%)

6 odpowiedzi

odpowiedź 10 lipca 2016 przez Comandeer Guru (600,730 p.)

Menu, z powodu swojej półprzezroczystości, jest bardzo słabo widoczne – zwłaszcza, gdy najedzie na tekst.
Część elementów interaktywnych (zwłaszcza linki otwierające dropdowny) nie ma odpowiedniego outline'u.
Czcionki z Google Fonts można spokojnie dociągnąć jednym żądaniem.
Skrypty powinny być na końcu body.
Po co Ci skrypt na menu skoro go nie używasz?
```
<div class="Header" id="Duzy">
```
czy
```
<div class="col-xs-6 col-md-3 kolumna">
```
Spójrz jak rozpada się cała konwencja nazewnicza. Dodatkowo unika się stylowania po [id].
Z rozkładu nagłówków wynika, że "Hala sportowa" jest cześcią "Celu szkoły".

<div class="col-xs-6 col-md-3 kolumna">
<article>

to można spokojnie zamienić na

<article class="col-xs-6 col-md-3 kolumna">

```
<div style="clear:both;"></div>
```
Bootstrap udostępnia clearfix – użyj go zamiast bzdurnych, pustych divów.

komentarz 10 lipca 2016 przez Adrian1999 Nałogowiec (34,570 p.)

Zastosowałem się do twoich rad, jak teraz oceniasz ? Jeszcze jakieś poprawki oferujesz?

komentarz 10 lipca 2016 przez Comandeer Guru (600,730 p.)

Tak średnio się zastosowałeś…

.clearfix jest po to, by nie robić pustych divów – Ty wstawiłeś go jako pusty div.
nagłówki dalej są źle rozłożone
.kolumna dalej w kodzie
akurat wywalenie klasy i stylowanie po header też nie jest dobre (tym sposobem utrudniłeś sobie znacząco stylowanie np. nagłówków artykułów)

komentarz 10 lipca 2016 przez Adrian1999 Nałogowiec (34,570 p.)

Gdzie tego "clearfix" powinienem umieścić, bo gdzie go nie umieszczam efekt jest taki że tytuł wtapia mi się z treścią

1	komentarz 10 lipca 2016 przez Comandeer Guru (600,730 p.) A spróbuj coś typu: <h2 class="clearfix">Nagłówek sekcji</h2>

komentarz 10 lipca 2016 przez Adrian1999 Nałogowiec (34,570 p.)

Dzięki jakoś działa, projekt wznowiłem przeważnie po jakimś czasie okazało się że to było winą moją że w kodzie miałem float:right; spójrz teraz, i rozwiń jak jest jakiś błąd i gdzie dokładnie bo "nagłówki są źle rozłożone" troszkę źle rozumiem by nie powiedzieć że w ogóle. I dzięki za pomoc i tworzenie dobrych nawyków

komentarz 10 lipca 2016 przez Comandeer Guru (600,730 p.)

gdzie dokładnie bo "nagłówki są źle rozłożone"

http://tutorials.comandeer.pl/html5-blog.html#artykul-outline

komentarz 10 lipca 2016 przez Adrian1999 Nałogowiec (34,570 p.)

O to m.in chodziło? clearfix dodany w takim razie, jeżeli z nagłówkami chodziło Ci o to co zrobiłem również, oraz kolumnę zlikwidowałem z kodu

odpowiedź 10 lipca 2016 przez niezalogowany

odpowiedź 10 lipca 2016 przez GrubaPanda Gaduła (3,920 p.)

Ten heder.png skompresuj np. za pomocą https://tinypng.com/ .Nie pogorszysz jakości obrazu, a zaoszczędzisz dodatkowe cenne KB ( w tym przypadku 491 KB ).
Skoro używasz swojego pliku ze stylami to bardzo ważne jest też, abyś swój style.css umieścił po bootstrap.min.css, dzięki czemu nie będziesz musiał dodawać do niektórych deklaracji !important. Po wprowadzeniu tej zmiany zauważysz, że dopiero teraz w menu działa Ci czcionka 'Itim'.