Warlike - 24 Październik 10, 09:52 Jedna uwaga - nie używaj na stronach czcionki Times New Roman, to wygląda strasznie nieprofesjonalnie, lepiej użyć Arial'a, Verdan'y czy Garamond'a. Wyjustuj tekst, będzie to dużo lepiej wyglądać. 
Tay - 24 Październik 10, 12:09 Dzięki za radę, jak będę mieć trochę czasu to tak zrobię
Siwy - 24 Październik 10, 19:46 W sumie jak na drugą stronę jest nieźle. Wygląd:  za niebiesko... dodaj jakiś kontrastujący kolor.
napisy w stopce strasznie nieczytelne (pomijając fakt, że "Dodaj do ulubionych", jak i wersja strony są bez sensu i głupio wyglądają)
zegarek - chyba każdy ma aktualny czas na "Pasku Start" (xD), albo na ręce, na ścianie, gdziekolwiek... zbędny "bajer"
nie centruj długich tekstów, justuj ;]
wiekszy padding w treści, bo prawie siedzi na bocznych kolumnach
profil ? to jakiś serwis społecznościowy ? ;P Daj jakieś zwykłe "O mnie", albo super pr0 po ang "About"
galerię umieść bezpośrednio na stronie, polecam lightbox 2 (albo zapromuję swój artykuł - http://filipmarkiewicz.boo.pl/galeriacss.php )
fajne tło ogólne . +
Kod: menu we Flash natychmiast wywal... dużo osób nie ma go zainstalowanego ze względu na sporo reklam w tej technologii - i jak będa poruszać się po Twojej stronie ? ;P. Tym bardziej, że tam tylko jakies strzałęczki się ruszają... Menu robi się na liście nieuporządkowanej <ul>, masz gotowce w kursie - kurshtml.boo.pl (chyba stamtąd sie uczyłęś )
wszystkie polecenia HTML pisz małymi literami ;]
nie, tylko nie <center> - natychmiast won!
target="_blank" też bym wywalił, niech uzytkownik sam zdecyduje, gdzie otworzy witrynę - jak spodoba mu sie Twoja strona to na pewno do niej wróci
na + nagłówek <h1> ;]
stosuj akapity <p>
W zasadzie jak na początek nie jest źle . Jak poprawisz te proste błędy bedzie całkiem przyzwoicie .
Tay - 25 Październik 10, 20:52 Dzięki za opinię i wytknięcie błędów
Mam menu we flashu, bo nie umiem zrobić takiej fajnej animacji z użyciem HTML i CSS jak ma na stronie np Odyn Tak, faktycznie uczyłem się z tamtego kursu 
Próbowałem zrobić kiedyś galerię lightboxem, ale coś mi nie wychodziło, sprawdzę ten Twój poradnik
Chodzi o dodanie w pliku styli dla body text-align: justify; ? Jak tak robię to mój podpis nie jest na środku ;/
Piszę małymi, jeśli gdzieś jest dużą, to pewnie wklejałem kod
Profil też zmienię na "o mnie" Wiem, głupie pytanie, ale możesz mi wytłumaszyć łopatologicznie co to jest "padding"? xD target="_blank" właśnie dlatego dałem żeby moja strona została otwarta, ale skoro radzisz wywalić, to nie ma sprawy
Jak zastąpić center? (jestem początkujący )
Kolorystykę też planuję troszkę zmienić, ale zbytnio czasu nie mam, może za kilka dni
Też poprawię, jak znajdę czas. Dzięki jeszcze raz za czas poświęcony na sprawdzenie kodu
Siwy - 26 Październik 10, 12:16 Mówię, że to zbędny bajer, a pchać do tego Flasha to grzech normalnie . Odyn ma to zrobione za pomocą JavaScript i to jest w formie dodatku - jak ktoś ma włączony JS to ma taki super bajer a jak nie to nie ma bejeru, ale widzi menu .
Tak, chodzi o text-align: justify; ;]. Daj justowanie tylko dla akapitow i wtedy podpis w inny element wsadzisz, gdzie będzie wycentrowany tekst za pomocą text-align: center; (pomijając to, że podpis zwykle umieszcza się po prawej stronie ;]). Padding to wewnetrzny margines po prostu (polecenie CSS) ;] Elementy liniowe centruje się nadając text-align: center; nadrzędnemu elementowi blokowemu (display: block;), a blokowe - szerokośc (width) i margin: 0 auto;. Tay - 26 Październik 10, 20:27 Jak dam padding dla diva zawartosc (załóżmy że 2 px) to cała strona mi się rozjeżdża ;( Siwy - 26 Październik 10, 20:55 Poczytaj - http://www.kurshtml.boo.p...,marginesy.html . padding + width/height + border = szerokość elementu, czyli, jeśli dodajesz po 10px paddingu po prawej i lewej stronie musisz odjąć 20px (10 + 10) od width .
Tay - 27 Październik 10, 14:38 A już kapuję bo odejmowałem przy padding 10px ; 10px(5+5)
[ Dodano: 16 Styczeń 11, 12:48 ] Odświeżam: zrobiłem całkowicie nowy szablon strony
Odyn - 21 Marzec 11, 12:35 W końcu udało mi się całkowicie zmienić wygląd mojej strony na coś bardziej na czasie Korzystając z okazji przeszedłem na UTF-8 i xHTML oraz zmieniłem wygląd odnośników.
Panowie, liczę na dobry feedback Przyda się zarówno ocena wyglądu, jak i wytknięcie ewentualnych błędów w kodzie.
Adres bez zmian - http://odyn.net.pl/ Tay - 21 Marzec 11, 14:52 Bardzo dobrze to wygląda. Tamten design też był dobry
Wygląd: 8/10 Menu bez fajerwerków, proste, ale dość dobrze się komponuje z resztą strony. Z wyglądu podobają mi się bardzo strzałki przy losowych zdjęciach itp. Co do kodu, to raczej nic Ci nie wytknę, bo nie jestem nawet w połowie taki dobry jak Ty
Odyn - 21 Marzec 11, 16:33 Menu nieco poprawiłem. Nadal nie ma fajerwerków, ale jest animowany prostokąt i biały cień tekstu (w FF)
Szeryf99 - 21 Marzec 11, 18:14 Mam pewien problem otóż na stronie robionej dla kumpla chciałem dodać graficzne odsyłacze, ale mi nie wczytuje nie wiem dlaczego. Wszystko robię normalnie. Oto kod: <a href="filmiki.html"><img src="c.jpg" border="0" alt="Filmiki" /></a> A oto błędny efekt: http://www.craftfan.ugu.pl/ A tak nawiasem nie patrzcie proszę na brak polskich znaków itp. i nie utworzenia podstron. Jak już to możecie zaproponować jakiś inny wygląd strony. Siwy - 21 Marzec 11, 19:18 Po pierwsze to nie jest temat do rozwiązywania błędów... Po drugie... ramki, Matko Boska... Ogółem kod dobry,l aby pokazać "jak nie pisać stron", ale trudno. Co do odnośników - zgaduję, że rozszerzenie jest wielkimi literami i dlatego nie działa na serwerze, a na kompie pewnie jest ok. No i sprawdź, czy nazwy się na pewno zgadzają. Domniemywać mogę jeszcze, że obrazki są np. w katalogu img czy images czy o jeszcze innej nazwie, a w kodzie nie podałeś takiej ścieżki. Co do wyglądu też lipa. Odyn! Strona moim zdaniem dużo, dużo ładniejsza od poprzedniej (co nie znaczy, że poprzednia była zła ;]). Ogółem do wyglądu nie mam zastrzeżeń - powiem więcej, jest bardzo fajny, taki... lekki i przejrzysty . Jednak, opóźniony o parę sekund hover w menu do mnie jakoś nie przemawia. Jedyne co mi sie nie podoba to troche bałagan w stopce  . Te obrazki są jakoś tak chaotycznie ułożone.
Kod generalnie w miarę okej, jednak nie jest idealny, zawsze idzie się do czegoś przyczepić . Po pierwsze - wykorzystaj <html> i to jemu nadaj te kratkowane tło . Body proponuję dać jako tło cały top, od granatowego paska do tego szarego paska dolnego (nie tego w stopce, tylko tego w topie). Dzięki temu wywalisz dwa zbędne divy - #naglowek i .pasek . Bardzo fajnie, że wreszcie zacząłeś uzywać nagłówków hx, ale data to chyba nie nagłowek IMO - wydaje mi się, że lepiej dać to jako paragraf, tym bardziej, że h3 jest nad h2, co jest "błędem".
Stopka - to samo co top - wszystko włąduj jako jedno tło dla jednego diva, i potem w drugiego daj to tło, które ma ten czarny promienisty gradient - zaoszczędzisz diva .pasek . Onclick - ajjjj nie, niepotrzebnie! ;D
Aaa.... Skrypt JS z head wywal do oddzielnego pliku . Podsumowując jest IMO bardzo dobrze - layout przejrzysty i ładny, kod także w porządku (jednak w miare możliwości popraw te wymienione przeze mnie błędy - ofc nie dam głowy, że wszystko tak pieknie łądnie można zrobić inaczej, acz wydaje mi się, że te zmiany są jak najbardziej możliwe).
Odyn - 21 Marzec 11, 20:20
Miło mi Takie było założenie - chciałem coś prostego, ale eleganckiego i na czasie.
Tutaj przesadzasz... Opóźnienie wynosiło 100ms, a teraz zmieniłem na 80. No chyba, że Twój komputer ma problemy z przemieleniem tej animacji... Mój laptop czasami przycina
Celowo użyłem skryptu hoverIntent, żeby nie animowało każdego najechania myszką, a tylko te zamierzone.
Masz pomysł jak to lepiej rozwiązać? Testowałem kilka ustawień i cztery kolumny wypadły najlepiej. Mogę ewentualnie zmienić je miejscami.
Ja nie czuję potrzeby tak mocnego ograniczania liczby elementów Wolę użyć kilku dodatkowych pojemników, zamiast całkowicie przestrzegać reguł semantycznej struktury kodu (co jest i tak głównie sztuką dla sztuki...).
Połączenie tła nagłówka z kratkowanym tłem body/html w jedną grafikę nie będzie proste ze względu na różne układy powtarzalności. Poza tym rozmiar nowego pliku będzie większy niż suma rozmiarów obu teł. To samo dotyczy łączenia paska z tłem stopki.
Data jako nagłówek to faktycznie nie najlepszy pomysł. Wprowadziłem to już prawie rok temu i jakoś przestałem zwracać na to uwagę. W najbliższym czasie poprawię to.
Co złego widzisz w onclick'ach dla galerii zdjęć, która i tak jest w JS? Jak ktoś ma wyłączony JS, to i tak nie zobaczy efektu...
Próbowałem to zrobić kilka miesięcy temu, ale miałem problemy z działaniem Highslide po takiej operacji  Teraz poszło bez problemów i nareszcie ten kod nie będzie mi zawalał źródła strony
Podsumowując, dzięki za wyczerpującą recenzję. Część poprawek na pewno wprowadzę, ale do pozostałych raczej mnie nie przekonasz
Siwy - 22 Marzec 11, 16:44
Nie mówię, że powinieneś to zmienić, tylko (być może niepotrzebnie) napisałem, że mi się to jakoś nie podoba, ale to kwestia gustu jest .
Pomyślałbym nad równymi odstępami między tymi "grupami" linków. Chodzi mi o to, że są tak porozrzucane troche, jedno po prawej, potem dwa po lewej...
Dlatego też napisałem pod koniec posta
Fakt, być może troche za bardzo się czasem czepiam, jednak tak jakoś zawsze pisząc "recenzję" strony wypisuję wszystko co tylko można zrobić 'lepiej'. _____________________________________________________________________
Chodziło mi o onclick'i w stopce, które wywalają do nowej karty .
|
