hejka

Community feedback

• 

  Nitr0Skay•100

  @Nitr0Skay

  Posted over 3 years ago

  Witam, panie Chory12. Albo może raczej powinienem odpisać tak jak pan "hejka" :D Mam nadzieję, iż nie obrazi się pan, że napiszę do Pana w naszym ojczystym języku, jeżeli jednak to problem to nie będzie problemu kontynuować naszą konwersację po Angielsku, niestety inne języki nie wchodzą w grę :D

  Do rzeczy. Co do styli to nie wiem co konkretnie mógłbym napisać, każdy korzysta z nich wedle swojego uznania i mówi się, że ile ludzi, tyle opinii co do tego. Jeżeli ma Pan jakieś pytania lub potrzebuje Pan z tym jakieś porady, proszę zapytać, postaram się pomóc.

  Ja natomiast postanowiłem nieco dopomóc Panu z Semantyką. Po pierwsze: dobrze byłoby zachować pewną konsekwencję przy nazewnictwu klas. U Pana występuje mieszanka klas nazywanych raz po Polsku (i po tym stwierdziłem też, że obaj jesteśmy Polakami) i po Angielsku. Dobrze jak wszystkie klasy nazywane są w jednym języku, a najlepiej wszystkie po Angielsku, gdyż składnia zarówno HTML jak i CSS to wszystko Angielski :)

  Po drugie: "<section class="main">" - na co to komu ? Wszak Standard HTML oferuje nam semantyczny element main: https://www.w3.org/TR/html-main-element/

  Po trzecie: "<img src="illustration-hero.svg" alt="">" - Text alternatywny (czyli atrybut alt) nie powinien być pusty. Jako iż ta ilustracja była typowym elementem designu (przynajmniej według mnie), dlatego postanowiłem to wstawić poprzez CSS jako background-image: https://developer.mozilla.org/en-US/docs/Web/CSS/background-image Ta opcja pozwala dodać ten obrazek bez konieczności martwienia się o tekst alternatywny i o to, co tam wpisać :)

  Po czwarte: "<p class="order">Order Summary</p>" - To mi szczerze wygląda na nagłówek sekcji, a nie jak Akapit. Od tego mamy też takie elementy jak h1 do h6. Trzeba tylko z nich korzystać rozważnie ;)

  Po piąte: Pan pozwoli, że przytoczę tutaj pewien wycinek Pańskiego kodu:

  <section class="tekst"> <p class="you">You can now listen to millions of songs,</p> <p class="audio">audiobooks, and podcasts on any device</p> <p class="like">anywhere you like!</p> </section>

  Ogólnie wydaje mnie się, że cały powyższy text można by potraktować, jako jeden spójny akapit i nikt by się raczej nie doczepił. Proszę także zauważyć, że powyższa wypowiedź to jedno zdanie ;)

  Sekcja z klasą "tekst" - po co to w ogóle powstało, skoro element "p" jest rozumiany jako jeden akapit, akapit tekstu ? ;p https://www.w3.org/MarkUp/HTMLPlus/htmlplus_11.html

  Ogólnie jeszcze sporo rzeczy można przytoczyć. Ale chcąc oszczędzić Panu natłoku informacji, na chwilę obecną zostawię Pana z powyższymi poradami. Jeżeli będzie Pan ciekawy, możemy kontynuować :D

  Co do Ligi, to co Pan mainuje i jaką ma pan dywizję ? Tak z ciekawości pytam ;p Życzę Panu miłego dnia :D

  00

  Marked as helpful
• 

  chory12•40

  @chory12

  Posted over 3 years ago

  Dziękuje za odpowiedź. Nie spodziewałem się że ktoś napisze aż tak dużo porad. To był mój pierwszy taki projekt. Co do ligi to mam platyne 4 ,mainuje Diane oraz Lee sina. Jeszcze raz dziękuje za wszystkie porady.

  00