Skocz do zawartości
  • Przeglądający   0 użytkowników

    Brak zarejestrowanych użytkowników, przeglądających tę stronę.

Promowane odpowiedzi

SpartaN    27
1 minutę temu, Mativve napisał:

U mnie na kompie te zdjęcia w collections wyjeżdżają poza ekran:

 

  Ukryj zawartość

screenshot-spartancode_ct8.pl-2017-09-01-13-03-01.thumb.jpeg.bdf63be27a6eef7722d23383760b603f.jpeg

 

 

Hmm jaka rozdiałka ?

Udostępnij tego posta


Odnośnik do posta
Udostępnij na innych stronach
Lord Fenrir    612

Fajnie, że ćwiczysz, ale to nie jest coś co można udostępniać. Błędy na każdej płaszczyźnie od designu po kodowanie. Już samo to, że w przypadku takiego projektu brakuje RWD świadczy o tym, że nie powinno się go udostępniać bo użyteczność zerowa. A jakby ktoś miałby sobie do tego dopisać, to chyba mniej siły kosztowało by napisanie od nowa.

 

- przydałby się jakiś 'lang' dla htmla, co nie?

- jakbyś dołączał bibliotekę jQuery asynchronicznie to nie byłoby problemu, ale tego nie robisz i wskazane jest abyś zamieszczał ją przed znacznikiem zamykającym 'body' a nie w 'head'.

- warto do każdego znacznika przypisać klase gdyż struktura CSS może łatwo ulec zmianie, a wtedy będzie jeszcze więcej poprawek do naniesienia.

- osobiście nie wykorzystałbym znacznika 'main' tylko dla sekcji głównej.

- 'h1' może występować raz w danym dokumencie strony, Ty masz ich kilka.

- brakuje 'diva' zamykającego:

<div class="container-2">
   <div class="box1">
   <img src="img/elegance.jpeg" alt="Kolekcja Elegancka">
</div>

- czy ten odnośnik w zamyśle miał prowadzić do zewnętrznej strony czy służyć jako przycisk? Jeżeli to drugie to można by nadać 'role=button' albo w ogóle zamienić to na 'button'.

<a href="#" class="btn btn-blue">SHOP NOW</a>

- szkoda, że nie ma stałego kontenera.

- troche nie rozumem tej konstrukcji:

<div class="container">
   <a href="#"><img src="img/winter.jpeg" alt="Winter collection"></a>
   <a href="#"><img src="img/summer.jpeg" alt="Summer Collection Letnia"></a>
   <a href="#"><img src="img/hipster.jpg" alt="Kolekcja Retro"></a>
</div>

- znowu brakuje 'diva' zamykającego:

<div class="opinion">
   <div class="opinion-info">
      <span class="opinion-author">John Doe</span>
      <span class="opinion-percent">100%</span>
      <span class="opinion-id">#1</span>
</div>

- zamiast 'span' należy użyć 'p':

<span>Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exe
          rcitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit es
          se cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.</span>

- załączasz FontAwesome, ale go nie używasz, to po co on tam?

- puste grafiki, które nie informują o zawartości, która kryje się pod odnośnikiem

- sekcja z newsami wydaje mi się zupełnie zbędna

- obrazki nachodzą na siebie podczas powiększania

- nie uważasz, że te obrazki na samym dole są trochę za duże?

 

Nie wiem co o tym myśleć. Nie jest to rewelacyjny projekt, a można napisać to o wiele lepiej. Mam za to jedno pytanie, Ty używasz jakiegoś specjalnego edytora czy zwykłego notatnika, bo nie wiem jakim cudem nie zauważasz niedomkniętych 'div' i 'section'...

Udostępnij tego posta


Odnośnik do posta
Udostępnij na innych stronach
SpartaN    27
9 minut temu, rogue napisał:

Nie wiem co o tym myśleć. Nie jest to rewelacyjny projekt, a można napisać to o wiele lepiej. Mam za to jedno pytanie, Ty używasz jakiegoś specjalnego edytora czy zwykłego notatnika, bo nie wiem jakim cudem nie zauważasz niedomkniętych 'div' i 'section'...

 

Korzystam z Atoma i teraz przejrzałem jeszcze raz i wszystko jest domknięte ( ? ) 

 

Udostępnij tego posta


Odnośnik do posta
Udostępnij na innych stronach
Lord Fenrir    612

Ach, cofam to. Jeszcze raz spojrzałem na te dziwne taby i faktycznie zgadza się. Wszystko jest domknięte, ale to nie wiele zmienia :)

Udostępnij tego posta


Odnośnik do posta
Udostępnij na innych stronach
Perry    15

 Musisz zrobić ładną szatę graficzną, bo nawet jakbyś pisał świetny kod (chociaż jeszcze do tego daleko ) to i tak nikt tego nie kupi bo jest brzydkie.

U mnie teksty się nachodzą , a grafika wychodzi na boki dlatego musisz popracować jeszcze trochę , ale co raz lepiej ;) 

  • Nie lubię tego! 2

Udostępnij tego posta


Odnośnik do posta
Udostępnij na innych stronach

Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto

Tylko zarejestrowani użytkownicy mogą komentować zawartość tej strony.


×