Tay - 28 Lipiec 11, 15:11 Dodałbym tło pod ten górny napis, powiększyłbym ten taki "papier" na środku, wyśrodkowałbym też to menu w poziomie na tym tle z desek i przesunął trochę w dół. A przede wszystkim weź się za CSS 
Siwy - 28 Lipiec 11, 21:27 Generalnie kod do wyrzucenia (tabele i takie kwiatki jak font) + grafika też do renowacji  . Jednak pierwsze co to zrezygnuj z ramek. Jeśli chodzi o grafikę to postaraj się aby wszystko do siebie pasowało w miarę - póki co są to dwa tła walnięte, aby było... Kartka też fajnie jakby się powtarzała ;]. No i jak Tay słusznie zauważył - weź się za CSS.
Szeryf99 - 10 Wrzesień 11, 23:18 Oto moja najnowsza strona (a właściwie szablon) proszę o krytykę. Co poprawić a co nie i nie pisać nie rób na ramkach. www.minecraftfan.ugu.pl mroczny38 - 11 Wrzesień 11, 09:26 A ta twoja strona będzie także miała spis Texture Pack'ów? Siwy - 11 Wrzesień 11, 16:19
To się wyklucza . Naprawdę ramki to niewygodny przeżytek... To tak jakbyś nie instalował w domu kranu na rzecz studni... 
Pomijając, że te Twoje ramki sa źle wykonane - przerzuca nas w nowe okno/kartę, gdy otwieramy linka z menu. Sam kod to totalna zupa tagów: - <font> - <center> - twarda spacja ( ) nie służy do robienia odstępów - <i> jest zdeprecjonowane (uzyj font-style: italic;) - </font size> ot, kwiatek xd. - używaj list <ul> (do menu, jak i w np. artykułach) - używaj nagłówków hn (n < 6; n > 0) Generalnie nie widzę żadnej poprawy w stosunku do poprzedniej strony. Dalej kod jest do wyrzucenia. I nie oburzaj się tu na mnie - znam się na tym, uwierz mi. Co zaś się tyczy wyglądu... Hmm... No jest słabo, ale powiedzmy, że jaaaaakoś się tam kupy to trzyma, jednak nie wczytuje mi topu. Szeryf99 - 11 Wrzesień 11, 18:59
Poprawiłem.
Nie wiem. Podejrzewam, że tak, ponieważ robię ją dla kolegi, który KOCHA MINECRAFTA. Tay - 28 Październik 11, 22:48 Niedawno znów wziąłem się za pisanie stron. Wreszcie podszkoliłem się trochę w PHP (tu kieruję wielkie podziękowania w stronę Siwego, bez którego zapewne nadal nie wiedziałbym podstawowych rzeczy), oraz poprawiłem trochę swą znajomośc css. Efektem tego jest ta oto strona: http://krzysiekkosecki.boo.pl/ Nie jest ona co prawda ukończona w 100%, ale do oceny już się nadaje. Kod jest (chyba) w miarę dobrze napisany, choć jeszcze go nie optymalizowałem. Także w moim małym autorskim CMS'ie nie wszystko jest gotowe. Jednak zapraszam do komentowania
Siwy - 29 Październik 11, 11:46 No to jedziemy xd. Stronę już widziałem wcześniej, więc zacznę od kodu:... - id dla <h1> zbędne - jeśli uzywasz <h1> jako tytułu całej strony (a tak IMO powinno się robić), to wiadomo, że będzie ono zawsze tylko jedno. + menu na liście <ul> - zbędny div #MENU -> nadaj ten identyfikator <ul>'owi, który ma bezsensowną klasę
* - ten 'slogan' umieść także w <h1>, z tym, że np. w spanie; będzie to minimalnie efektowniejsze dla Ciebie z punktu widzenia SEO, bo ten 'slogan' to IMO także część tytułu strony - z tego co zauważyłem, na każdej podstronie masz jedno <h2> (i słusznie, bo to tytuł tej podstrony), więc .komunikat zbędna + podoba mi się, że dla id uzywasz wielkich liter, a dla class - małych - też systematyzuję oba te elementy, tylko minimalnie inaczej (identyfikator, klasa_, mojIdentyfikator, moja_klasa) * stopke dałbym jako akapit - dlaczego w "O mnie" Twoje zdjęcie to nie jest obrazek, tylko #JA z tłem? oO + Strict - unikaj nazw klas i identyfikatorów, które to już w nazwie opisują jego cechy - np. ".bez_ramki"; co jak potem zachcesz nadać mu ramkę, tylko, że inną od pól formularza, etc.? nadaj jej klasę '.wyslij' czy coś
+ walidator CSS * walidator HTML wywala jeden błąd, czepiając się wielkości liter w atrybucie method xd Ale ogólnie okej .
Tay - 29 Październik 11, 12:55 Dzięki za wszystkie rady
Zrobione
Niestety na razie ten div musi być, gdyż inaczej nie potrafię zrobić tego górnego obramowania, a nie chcę z tego rezygnować.
Zrobione
Zrobione
Zrobione, faktycznie tak jest lepiej.
A tak jakoś, później to zmienię
Za chwilkę to poprawię
Poprawione. Jeszcze raz dzięki za wszystkie wskazówki
Odyn - 29 Październik 11, 20:15 Po poprawkach zasugerowanych przez Siwego, nie mam praktycznie do czego się przyczepić, jeżeli chodzi o kod Trochę nie pasuje mi jednie stylizacja obrazka po id, zamiast ustawienia ogólnej klasy, ale to drobny szczegół
Ogólnie stronka podoba mi się kolorystycznie, ma prosty, ale niebanalny układ i jest spójna. Z biegiem czasu, jak dodasz więcej treści, całość będzie prezentowała się jeszcze lepiej. Czy menu na sztywno odsunięte od lewej krawędzi to celowy zabieg, czy po prostu zabrakło pomysłu na rozwiązanie tego inaczej? Przez moją przeglądarkę przewinęły się setki stron, ale pierwszy raz spotkałem się z takim rozwiązaniem, przy wyśrodkowanej treści
Tay - 29 Październik 11, 20:33
Już to poprawiłem, w rozdzielczości, której używam (1280x1024) nie raziło to specjalnie w oczy Teraz menu zaczyna się równi z divem #TRESC 
Odyn - 29 Październik 11, 20:56 Zastosowałeś dokładnie takie rozwiązanie, jakie chciałem Ci zaproponować 
Tay - 6 Maj 12, 14:10 ylyf.pl <--- efekt 3 miesięcy niespecjalnie wytężonej pracy Myślę, że jeszcze trochę i zacznę pisać obiektowo w PHP
[ Dodano: 6 Maj 12, 15:17 ] Napisałem też nową wersjęmojego portfolio: http://krzysiekkosecki.boo.pl/ |
