• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Moja strona proszę o opinie i wskazówki.

VMware Cloud PRO - przenieś swoją infrastrukturę IT do chmury
+7 głosów
985 wizyt
pytanie zadane 5 kwietnia 2015 w HTML i CSS przez Dany Obywatel (1,370 p.)
przywrócone 8 kwietnia 2015 przez Dany
Witam

Jestem fanem gry World of Tanks i gdy się nudziłem postanowiłem zrobić coś pożytecznego i stworzyć stronę która miała by pomagać graczom. Strona http://wot-informacje.pl a tak ma wyglądać mniej więcej podstrona do każdego czołu http://wot-informacje.pl/nacje/zsrr/t-54light.php . I teraz moja prośba do graczy jeśli są co o niej sądzą od strony gracza czy strona pomaga czy też nie, a dla osób które zajmują się robieniem stron co poprawić co zmienić a co usunąć .

Z góry dziękuję za każda wskazówkę, uwagę

Pozdrawiam

11 odpowiedzi

+6 głosów
odpowiedź 5 kwietnia 2015 przez nefil1m Stary wyjadacz (10,690 p.)
wybrane 10 maja 2015 przez Dany
 
Najlepsza

Cześć, przęglądnąłem Twój stronkę na szybko, zapowiada się ciekawie i mam dla Ciebie kilka rad:

1. Jak masz spis czołgów dla danego kraju to albo wyrównaj wszystko do lewej albo zrób z tego tabelkę. Teraz całość "pływa" i wygląda trochę nieczytelnie. (chyba najelpsza byłaby tabela)

2. Tutaj coś Ci się "rozjechało", na oko jakaś literówka w kodzie. Oprócz tego masz list-itemy poza jakąkolwiek listą.

<p> V </p> 
<selektor> 
<a href="brytania/ChurchillI.php"> Churchill I</a>, 
<a href="brytania/Excelsior.php"> Excelsior </a>  
</<selektor>
<li></li>

Poza tym zamiast robić <p>V</p> powinieneś sobie zrobić <ol> ze stylem list-style-type: upper-roman.

Powinno wyglądać tak: (oczywiście style najlepiej dać osobno do pliku)

<h2>Średnie czołgi</h2>
<ol style="list-style-type: upper-roman">
  <li><a href="brytania/VickersMediumMk.I.php"> Vickers Medium Mk. I </a></li>
</ol>


4. Zastąp <selector> np. <span>. Semantycznie bardziej poprawne. (tak na prawdę oba są zbędne w tym wypadku)

5. Staraj się unikać takich rzeczy w kodzie

<ul>
	<p></p>
</ul>

6. Atrybut 's' ? Pusty alt="". Style powinny być osobno. Możesz im nadać jedną klasę np .flaga

<img src="images/Niemcy.png" width="25" height="15" alt="" s >

7.

<meta http-equiv="Content-Type" content="text/html; charset=utf-8">

Zamień na

<meta charset="utf-8">

8. Nie ma <!doctype html>

9. Masz <script> poza tagiem <html></html>

10. Nie łatwiej dać #stopka text-align: center? Po co ten dodatkowy div?
 

<div id="stopka">  
  <div style="text-align: center">Polityka prywatności	::  &copy; wot-informacje 2014-2015 :: <a href="kontakt.php">Zgłoś błąd lub sugestię</a></div>
</div>

Teraz trochę o CSS
1. Nie styluj id, styluj klasy. ID zostaw dla js.

2. scalaj takie wpisy w jeden

#menu_lewe li {
	padding: 2px 0;	
	list-style-type: none;
}

#menu_lewe li {
	vertical-align: middle
}

Dodatkowo polecam wyrobić sobie nawyk stawiania ; na końcu każdej linii

3. Nadajesz listy-style-type tagom <li> co jest błędne. Powinieneś dawać takie style <ul> albo <ol> i zmieniać całą listę na raz.

4. To nic nie zmienia:

#menu_lewe li {
    vertical-align: middle
}

I to chyba tyle. Większość z rzeczy które napisałem to raczej kosmetyczne zmiany ale uczynią Twój kod ładniejszym i bardziej poprawnym. Oprócz tego (ale to już kwestia preferencji) polecałbym używać angielskich nazw i camelCase (łatwiej się piszę). Jakby coś co napisałem nie było dla Ciebie zrozumiałe to śmiało pytaj.

komentarz 5 kwietnia 2015 przez Dany Obywatel (1,370 p.)
Dziękuje bardzo za twój czas postaram się dziś na tym posiedzieć i zrobić jak mówisz jeszcze raz dzięki za twoja analizę
komentarz 5 kwietnia 2015 przez nefil1m Stary wyjadacz (10,690 p.)
Polecam się na przyszłość :)
+3 głosów
odpowiedź 5 kwietnia 2015 przez krecik1334 Maniak (58,390 p.)
Jeżeli chodzi o layout to przydałoby się jakieś tło np ze strony subtsubtlepatterns.com. Oprócz tego menu boczne mogloby byc większą czcionką. Oceniam z poziomu smartfona. Pozdrawiam.
+3 głosów
odpowiedź 5 kwietnia 2015 przez Sebastian Fojcik Nałogowiec (43,040 p.)
Robiąc stronę o czołgach będziesz musiał robić własne projekty z hit-pointami. Strasznie dużo roboty, biorąc pod uwagę to, że będziesz chciał wzbogacić stronę o ładne grafiki. Kopiowanie z innych stron nie wchodzi w grę. Gratuluję podjęcia się takiego wyzwania.
komentarz 5 kwietnia 2015 przez Dany Obywatel (1,370 p.)
Tak wiem dzięki
+2 głosów
odpowiedź 5 kwietnia 2015 przez fobos Obywatel (1,400 p.)
Co usunąć? Komunikat - Uważasz że strona może być przydatna i będziesz z niej korzystał ?
+2 głosów
odpowiedź 5 kwietnia 2015 przez GrzesiekB Gaduła (3,720 p.)
1. Wywal ankietę z topu strony - to jest dokładnie to o czym mówił Mirosław w swoim podcaście - daj ją w formie boxa z boku albo pod tekstem, ostyluj ją porządnie CSS tak any zachęcić usera do kliknięcia. Wywal przycisk głosuj - samo kliknięcie powinno już zapamiętac wybór.

2. Wywal treść o trwających pracach - to nie zachęca.

3. Data aktualizacji? Ok, ale na dole strony - to nie jest najważniejsze info na stronie.

4. Zrób jakiś zachęcający nagłówek, grafikę, fajnego screena z gry - tego brakuje...

 

O źródle strony HTML/CSS się nie wypowiadam - na tablecie tego podejrzeć nie mogę...
+1 głos
odpowiedź 5 kwietnia 2015 przez Bartosz Płóciennik Dyskutant (8,510 p.)
Ja zachęcałbym abyś od razu próbował iść ścieżką responsive design i np. wykorzystać framework boostrap. Nie zajmie to dużo czasu a efekty bedą od razu zauważalne zwłaszcza na telefonach :)

Do tego również przydałaby się szata graficzna w tle np. jakieś zdjęcie czołgu czy coś takiego co będzie wyglądało ciekawie i będzie dodatkiem do treści, która oczywiście jest najważniejsza.

Formularz kontaktowy nie wygląda zbyt ciekawie znaczy chodzi mi o input zaraz po tekście np. adres-email, temat - sformatuj to jakoś a efekt będzie o wiele lepszy :)

Favicona jest bardzo ładna xD

Menu również dobrze zrobione i oparte na ul i li - to jest dobra praktyka. Stronka jest prosta ale ogólnie schemat(DOM) dobrze rozplanowałeś. Kod wygląda dobrze.

Nie mnie życzę aby się Tobie udało stworzyć coś co pomożę społeczności WoT.
+1 głos
odpowiedź 5 kwietnia 2015 przez Szymon Giza Obywatel (1,580 p.)

Według mnie strona meeega przydatna !

Z pewnością polecę ją kuzynowi który jest "uzależniony' ot tej gry ! cheeky

Ale wygląd troche w CSS popraw i bedzi yes idealnie 

+1 głos
odpowiedź 5 kwietnia 2015 przez Robert Saternus Stary wyjadacz (12,180 p.)
Ogólnie, jak piszesz pod HTML5 to nie powinieneś się opierać na divach. Wiem, że Pan Mirek tak uczył, ale myślę, że pokazał to tylko dla lepszego zrozumienia tematu (według standardów jest to błąd).

Poczytaj sobie o takich znacznikach jak <header>, <section>, <article>, <aside>, <footer>, <figure> oraz koniecznie o outline strony jak mylisz oczywicie o bardziej poważnej działalności :)
+1 głos
odpowiedź 6 kwietnia 2015 przez Mizukage Pasjonat (21,750 p.)
Troche za mała ta strona, przynajmniej na full hd res
+1 głos
odpowiedź 6 kwietnia 2015 przez krecik1334 Maniak (58,390 p.)

Wcześniej oceniałem z poziomu smartfona, teraz laptopa(1366x768). Co mi się rzuca w oczy? Divy nie mają parametrów w procentach, przez to strona staje się zbyt mała na większe rozdzielczości np. 1920x1080. Polecam zamiast takiego np. kodu:

#kontener {
    width: 900px;
    margin: 5px auto;
    text-align: left;
    background: url("images/tlo.png") repeat-y scroll 0% 0% #EBDFC5;
    border: 1px solid #B59E6B;
    list-style-type: none;
}

Dać taki kod:

#kontener {

    width: 100%;
    text-align: left;
    background: url("images/tlo.png") repeat-y scroll 0% 0% #EBDFC5;
    border: 1px solid #B59E6B;
    list-style-type: none;
}

 

 

Potem poustawiać marginesy i inne divy też na wartość % np. menu boczne width 20%. Polecam, stronka będzie wtedy na fullscreenie u każdego, nawet w rozdzielczości 4K czy QHD. Przykładowy efekt widoczny u mnie:

http://niekrytykretek.cba.pl/

 

komentarz 6 kwietnia 2015 przez nefil1m Stary wyjadacz (10,690 p.)

100% na kontener to moim zdaniem za dużo. Full Width Page źle się czyta na np 1920x1200 24". Zrobiłbym raczej (dla desktopów)

min-width: 900px;
width: 100%;
max-width: 1350px //przykładowo, możliwe, że można dać trochę więcej



 

 

komentarz 6 kwietnia 2015 przez Dany Obywatel (1,370 p.)
Dzięki za celna uwagę postaram się to zmienić

Pozdrawiam
+1 głos
odpowiedź 6 kwietnia 2015 przez ReksetoDev Gaduła (4,530 p.)
zrób drzewko czolgów dla każdego kraju :)

Podobne pytania

+7 głosów
2 odpowiedzi 391 wizyt
0 głosów
0 odpowiedzi 227 wizyt
+1 głos
6 odpowiedzi 529 wizyt
pytanie zadane 29 grudnia 2015 w Nasze projekty przez kacpx123 Początkujący (410 p.)

93,434 zapytań

142,429 odpowiedzi

322,662 komentarzy

62,798 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

...