Na forum.stronghold.net.pl wykorzystujemy ciasteczka. Jeśli jeszcze nie masz dość tego typu komunikatów, więcej informacji znajdziesz w Polityce Cookies. zamknij
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ć.
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 .
Gra w: życie
Wiek: 29 Posty: 652 Skąd: Nowy Sącz Medale: 1 (Więcej...)
Wysłany: 25 Październik 10, 20:52
Dzięki za opinię i wytknięcie błędów
Siwy napisał/a:
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łęś )
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
Siwy napisał/a:
nie centruj długich tekstów, justuj ;]
Chodzi o dodanie w pliku styli dla body text-align: justify; ?
Jak tak robię to mój podpis nie jest na środku ;/
Siwy napisał/a:
wszystkie polecenia HTML pisz małymi literami ;]
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 )
Siwy napisał/a:
za niebiesko... dodaj jakiś kontrastujący kolor.
Kolorystykę też planuję troszkę zmienić, ale zbytnio czasu nie mam, może za kilka dni
Siwy napisał/a:
napisy w stopce strasznie nieczytelne (pomijając fakt, że "Dodaj do ulubionych", jak i wersja strony są bez sensu i głupio wyglądają)
Też poprawię, jak znajdę czas.
Dzięki jeszcze raz za czas poświęcony na sprawdzenie kodu
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;.
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 .
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.
Gra w: życie
Wiek: 29 Posty: 652 Skąd: Nowy Sącz Medale: 1 (Więcej...)
Wysłany: 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
Gra w: wszystko i nic
Wiek: 31 Posty: 324 Skąd: Stąd Medale: Brak
Wysłany: 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:
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).
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
Miło mi Takie było założenie - chciałem coś prostego, ale eleganckiego i na czasie.
Siwy napisał/a:
Jednak, opóźniony o parę sekund hover w menu do mnie jakoś nie przemawia.
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.
Siwy napisał/a:
Jedyne co mi sie nie podoba to troche bałagan w stopce . Te obrazki są jakoś tak chaotycznie ułożone.
Masz pomysł jak to lepiej rozwiązać? Testowałem kilka ustawień i cztery kolumny wypadły najlepiej. Mogę ewentualnie zmienić je miejscami.
Siwy napisał/a:
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
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.
Siwy napisał/a:
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".
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.
Siwy napisał/a:
Onclick - ajjjj nie, niepotrzebnie! ;D
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...
Siwy napisał/a:
Aaa.... Skrypt JS z head wywal do oddzielnego pliku
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
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.
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 .
Cytat:
Masz pomysł jak to lepiej rozwiązać? Testowałem kilka ustawień i cztery kolumny wypadły najlepiej. Mogę ewentualnie zmienić je miejscami.
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...
Cytat:
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.
Dlatego też napisałem pod koniec posta
Cytat:
(...) 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 (...)
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'.
_____________________________________________________________________
Cytat:
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...
Chodziło mi o onclick'i w stopce, które wywalają do nowej karty .
Nie możesz pisać nowych tematów Nie możesz odpowiadać w tematach Nie możesz zmieniać swoich postów Nie możesz usuwać swoich postów Nie możesz głosować w ankietach Nie możesz załączać plików na tym forum Możesz ściągać załączniki na tym forum