Ocena strony internetowej

Question

Prosze o ocenę strony http://atelierfashion.pl/pl/

Comments

Masz straszny bałagan w kodzie...wszystko jest jakieś porozwalane... Wciąż mieszasz też kod JS z html... nie lepiej skrypty JS połączyć np. w 1-2 pliki i po prostu zminifikowane wersje dociągnąć na końcu?

To samo z html i pewnie css też (nie sprawdzałem)... daj najlepiej na produkcję wersję zminifikowaną, a kod do oceny np. na GH, bo ten kod strasznie ciężko się analizuje...

---

@penera, dzięki za odpowiedź

Answer 1

UX:

• Baner zdecydowanie za duży, przytłacza wszystko
• Menu zdecydowanie za duże, nie widzę treści muszę scrollować.
• font w listach zdecydowanie za duży
• wyszukiwarka, lista opcja zbyt mocno ściśniona
• Słabo wygląda te najedżenie loga.
• Kiedy nie znajduje produktu, focus znika i mruga logo.
• Angielski z polskim nigdy nie jest dobry
• Nieklikalny koszyk
• zaloguj nie mieści się w jednym ekranie i jeszcze z boku przez co to jest słabo widoczne
• Wszędzie arial a tu time-new-roman na panelu do logowania się
• Ponownie polsko-angielskie nazewnictwo. Nie jest to żadna sklejanka?
• zamknięcie okna logowania, odświeża stronę. Po co?
• zmiana waluty odświeża stronę. Można to lepiej zrobić.
• nie po https. Niedługo może Ci się świecić smutny znaczek ostrzeżenia.
• Po co taka duża przerwa pod banerem?
• Można było wybrać lepszą jakość zdjęcia
• Brak hovera na kategori produktu. Czy to jest zdjęcie??
• Logo bez wyciętego białego

Sporo białych przestrzeń zbudowanym kodzie.

Dalej nie oceniam z daleka widać, że sklejanka.

Answer 2

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

Coś tutaj trzeba wpisać

<link rel="stylesheet" href="http://atelierfashion.pl/themes/theme1464/assets/cache/theme-31e5f03.css" type="text/css" media="all">

Nie musimy korzystać z type="text/css" (mamy HTML5) oraz możemy użyć względnego adresu do pliku css

<script type="text/javascript">

Nie trzeba pisać type="text/javascript"

<main>
      
              

      <header id="header">

Nie wiem czy header może być w mainie, poza tym ten header to trochę nav (bo widzę w kodzie że jest dropdown menu)

<div class="wrapper it_QYYJQAATWHTI header-nav container">
<div class="row it_OUAHNTMYNQKJ ">
<div class="it_OEPXDZNREEJF    col-lg-4  hidden-md-down">

Szczerze? Nie wiem jak generowałeś nazwy tych klas, ale powinny one mówić co robią

 <div class="">

              <input
                class="form-control"
                name="firstname"
                type="text"
                value=""
                                                required        >

1. Klasa diva jest pusta. Poza tym dlaczego required jest tak daleko?

<div class="help-block">
        <ul>
        </ul>
      </div>

Puste elementy są złe semantycznie

<section id="main">

    
      
    

    
      <section id="top-column">

Sekcja w sekcji? Poza tym dlaczego sekcja ma id=main? Mamy od tego specjalny znacznik <main>

<img src="http://atelierfashion.pl/221-home_default/sukienka-szara-boho-kolorowe-fredzle.jpg" alt="" data-full-size-image-url="http://atelierfashion.pl/221-large_default/sukienka-szara-boho-kolorowe-fredzle.jpg">

Dlaczego alt jest pusty?

<footer class="page-footer">
          <!-- Footer content -->
        </footer>
      
    

  </section>


    
  </div>


        
        
      </section>

      <footer id="footer">

Dlaczego footer jest w sekcji? A ta sekcja w jeszcze innej sekcji? I w dodatku dlaczego footer jest pusty?

Poza tym kod jest rozwalony, taby nie wiadomo dlaczego, sekcje, artykuły nic nie wiadomo gdzie się zaczyna a gdzie kończy, brak wcięć, stylujesz po id a powinieneś po klasach no i masa kodu JS, który może zostać przeniesiony do osobnego pliku.