Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www]http://www.neatgroup.pl - Design Studio Poznań
Forum PHP.pl > Inne > Oceny
ciepolml
Witamy serdecznie, po prężnie przebytym roku nasz zespół wzbogacił sie o kolejna wersje strony www.
Zapraszamy do opinii / krytyki / sugestii.

Adres:
http://www.neatgroup.pl
l0ud
Wizualnie w porządku. Technicznie już za dobrze:

- Divitis. Istnieją też inne znaczniki niż <div>, a tych praktycznie brak.
- Niesemantyczne nazwy klas css, poza tym raz po polsku, raz po angielsku... ogólnie bałagan
- Puste atrybuty alt dla obrazków.
- Chwalicie się XHTML 1.0 Strict, a jest to Transitional
- Pełno jakichś &nbsp; w kodzie, br'y, niepotrzebne kombinacje z obrazkami, zamiast użycia listy
- Na siłę unikacie tabel, co to są da divy: komoreczka, real_wiersz, real_kolumna?
- Nieeleganckie puste tagi, jak np. <div id="home_spacer"></div>
- Ok, widzę pierwszą listę, a w niej pełno elementów z takimi samymi klasami: lista_tw - classitis. Można to było zrobić prościej w css.

Jeżeli to wygląda i jakoś działa to ok, ale technicznie słabiutko winksmiley.jpg
ShadowD
Na stronie są dostępne obrazki, po kliknięciu na nie ekran czarnieje i powiększa się zdjęcie, do tej poty wszystko ok jak na wielu innych stronach ale po pewnym czasie powiększania obrazek wychodzi poza stronę czyli widzę jego połowę próba przewinięcia strony kończy się fiaskiem...

I jak kolega napisał na stronie jest:
Cytat

W kodzie:
Cytat
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


Co najlepsze strona w Strict ma 3 błędy a w Transitional już żadnego rozumiem, że to taki chwyt marketingowy??
ciepolml
Dziękuje za konkretne wypowiedzi. Fakt z Strict nasz błąd i nie jest to chwyt marketingowy:)
Co do lightbox, pracujemy nad nim zeby byl widoczny suwak.

Za rozwinięcie potknięć technicznych, dziękuje - poprawimy.
kavka
W kod nie zaglądałam. Wizualnie bardzo mi się podoba. Jest jeden szczegół graficzny: każdy ważniejszy dział na stronie zaczyna się wypunktowaniem - jakieś takie zielone kółko. Niestety zlewa się z występującym po nim tytułem. Na pierwszy rzut oka wzięłam to za literę "O". Proponuję zmienić lekko odcień.
.radex
Korzystanie z <div>ów do danych tabelarycznych jest równie złe (albo jeszcze gorsze) niż korzystanie z tabelek do danych nietabelarycznych.

Najciekawsze kwiatki:

Kod
<div id="logo" onclick= "location.href='http://neatgroup.pl'">


Kod
<img src="img/kolko.gif" alt="" class="kolko_left" /><span class="naglowek">&nbsp;Parę słów o nas</span>


To powinno wyglądać tak:

Kod
<h3>Parę słów o nas</h3>


to img do zrobisz przez css, nbsp też tongue.gif

Kod
<img src="img/kolko.gif" alt="" class="kolko_left" /><span class="naglowek">&nbsp;Czym&nbsp;się&nbsp;zajmujemy</span>


Ciekawe... Ze spacją jest coś nie tak, aby używać zamiast niej na siłę nbsp?

Kod
<div id="menu">
<script type="text/javascript">
AC_FL_RunContent( 'type','application/x-shockwave-flash','data','flash/menu3.swf','width','483','height','129','id','swf','movie','flash/menu3' ); //end AC code
</script><noscript><object type="application/x-shockwave-flash" data="flash/menu3.swf" width="483" height="129">
<param name="movie" value="flash/menu3.swf" />
</object></noscript>
</div>


Menu we flashu? Szczyt wszystkiego biggrin.gif

Ogólnie: nie jest tragicznie, ale jeszcze dużo do poprawy winksmiley.jpg Strona kiepsko chodzi bez JS-a.

Pozdrawiam winksmiley.jpg
devnul
Cytat(.radex)
Menu we flashu? Szczyt wszystkiego

nie zgodzę się menu we flashu jest jak najbardziej ok pod jednym warunkiem - zapewniamy alternatywę. Po zatym, .radex, już kiedyś to pisałem, ale się powtórzę, czyjeś nawet najdrobniejsze błędy wytykasz i wyśmiewasz a na swoim podwórku posprzątać nie potrafisz. Twoja strona domowa (adres w Twojej stopce) aż woła o pomstę do niebios. Tyle błędów, niepodomykanych znaczników. Wymieniałbym dalej ale chyba wiesz co mam na myśli. Nie rób z siebie większego gooro niż jesteś bo to naprawdę fajne nie jest.

A co do ocenianej strony to od siebie dodam że o dostępności to najwyraźniej nikt u was nie słyszał. W szczegóły się wdawał nie będę (powim tylko że dostępność strony leży i kwiczy) - wystarczy pogooglać. Wizualnie mi to do gustu za bardzo nie przypada. Blade kolory, które zlewają się z tłem i pełno kropek walących po oczach. Straszne
ciepolml
Manu Flash? - długo sie zastanawiałem czy je dać czy zostawić normalne. Ale wydaje mi sie że narazie zostanie.
Co do nagłówków <h3> - prawda poprawimy, pomoże to z reszta w pozycjonowaniu strony.

A devnul, o co dokładniej chodzi z dostępnościa strony? Chcesz powiedzieć ze ma słabą pozycję w google czy ze jest powiązana z wątkami z którymi nie powinna jak wpiszesz neatgroup.pl w wyszukiwarce?
AjaxSrajax
Hej,

Ja nie mam flash i obrazków dziś haha.gif
U mnie stronka wygląda tak.
To też wyjaśnia konieczność alternatywy dla flash.
To też wyjaśnia ciut o dostępności.
Więcej o dostępności można przeczytać np. tu.
Kod faktycznie bardzo kiepściutki.
Powodzenia i owocnych projektów...
devnul
Cytat
A devnul, o co dokładniej chodzi z dostępnościa strony? Chcesz powiedzieć ze ma słabą pozycję w google czy ze jest powiązana z wątkami z którymi nie powinna jak wpiszesz neatgroup.pl w wyszukiwarce?

na stronie macie coś takiego w dziale oferta
Cytat
Spełniamy najnowsze standardy
i ja się pytam co to za standardy skoro na hasło dostępność robicie wielkie oczy. Co z was za agencja jak podstawowej terminologii nie znacie. Sugerowałbym zacząć od wikipedii bo tam jest wiedza ogólna. Dalej podążaj za dowiązaniami. Mam nadzieję że spać w nocy możecie bo ja bym tak kłamać klientom nie potrafił. A co do tych nagłówków to nie tylko o h3 chodzi. Tu chodzi o całą semantykę dokumentu
.radex
Cytat(devnul @ 15.08.2008, 23:25:59 ) *
nie zgodzę się menu we flashu jest jak najbardziej ok pod jednym warunkiem - zapewniamy alternatywę.


Święta prawda winksmiley.jpg

Cytat
Po zatym, .radex, już kiedyś to pisałem, ale się powtórzę, czyjeś nawet najdrobniejsze błędy wytykasz i wyśmiewasz a na swoim podwórku posprzątać nie potrafisz. Twoja strona domowa (adres w Twojej stopce) aż woła o pomstę do niebios. Tyle błędów, niepodomykanych znaczników. Wymieniałbym dalej ale chyba wiesz co mam na myśli.


Wytłumaczyłem, dlaczego tak jest (wiem, to nie jest usprawiedliwienie) i obiecałem, że to zostanie poprawione i dotrzymam obietnicty winksmiley.jpg

Cytat
Nie rób z siebie większego gooro niż jesteś bo to naprawdę fajne nie jest.


Tej opinii trochę nie rozumiem :/
ciepol
No więc może i ja się wypowiem jako osoba odpowiedzialna za kod stronki. W sumie to mój pierwszy taki projekt po kilku latach przerwy w "zabawie w stronki".
Wszystkim dziękuję za wszelkie uwagi. Większość z sugerowanych rzeczy już poprawiliśmy.

Kilka tylko uwag:
Cytat
- Niesemantyczne nazwy klas css, poza tym raz po polsku, raz po angielsku... ogólnie bałagan

Hmm, stosuję taki miks żeby używać jak najkrótszych nazw (najczęściej). Poza tym - po co ci znać nazwy moich klas, którymi ja się będę posługiwał i które dla mnie są logiczne i wydajne itp itp questionmark.gif


Cytat
- Nieeleganckie puste tagi, jak np. <div id="home_spacer"></div>

Niekiedy spacery są potrzebne aby zlikwidować float. Inne przydają się na etapie projektowania stronki a niekiedy, gdy chcę umieścić dodatkowe tło, które będzie się powtarzać wiele razy.


Cytat
- Chwalicie się XHTML 1.0 Strict, a jest to Transitional

Tzw. czeski błąd można powiedzieć. Poprawione.


Cytat
Na stronie są dostępne obrazki, po kliknięciu na nie ekran czarnieje i powiększa się zdjęcie, do tej poty wszystko ok jak na wielu innych stronach ale po pewnym czasie powiększania obrazek wychodzi poza stronę czyli widzę jego połowę próba przewinięcia strony kończy się fiaskiem...

Już poprawione, choć z efektu nie jestem do końca zadowolony. Wynika to z dość nietypowej, chyba, struktury strony.


Cytat
Menu we flashu? Szczyt wszystkiego biggrin.gif

Pole do dyskusji. Coraz więcej jest stron tylko i wyłącznie we flash. Niestety jest on już powoli standardem. Jeżeli ktoś nie ma flasha, no to przepraszam bardzo, ale pół internetu ten człowiek nie zobaczył. Póki co zostaje więc.


Reszta kwiatków też poprawiona. Ale tak to jest - czasami stronkę tworzyliśmy, że tak powiem dynamicznie winksmiley.jpg Więc o pewnych rzeczach potem się zapomina. Dlatego m. in. daliśmy stronkę do fachowej oceny.
ciepolml
Apropos menu Flash.
Cytat z oferty:

"Nasz zespół najczęściej wykorzystuje Flash tylko w elementach strony, gdyż uważamy ze tylko wtedy spełnia w pełni swoja funkcje."

Mi sie osobiscie wydaje, ze menu flash jest OK. A co do alternatywy, fakt mogłoby sie pojawic tekstowe menu w warstwie pod Menu ale jest mapa strony i menu dolne.
devnul
Cytat
Niekiedy spacery są potrzebne aby zlikwidować float. Inne przydają się na etapie projektowania stronki a niekiedy, gdy chcę umieścić dodatkowe tło, które będzie się powtarzać wiele razy.
LOL
większej bzdury nie słyszałem dawno. Każdy, ale to KAŻDY efekt występujący na waszej stronie mógłbym z miejsca zakodować na co najmniej 5 różnych sposobów nie używając ŻADNEGO spacera/pustego elementu i tym podobnych zbędnych kwiatków

Cytat
Hmm, stosuję taki miks żeby używać jak najkrótszych nazw (najczęściej). Poza tym - po co ci znać nazwy moich klas, którymi ja się będę posługiwał i które dla mnie są logiczne i wydajne itp itp
Nie no - pogooglałbyś trochę żeby się nie wygłupić. Generalnie też uważam że bawienie się w semantykę klas to trochę już przeginanie pałki w drugą stronę ale z zagadnieniem warto się zaznajomić.

A nagłówki h3 to nie wszystko h3 jest nagłówkiem potomnym h2 a ten h1. Dwóch ostatnich na waszej stronie nie widać. Cienko chłopaki. A tłumaczenia o kilkuletniej przerwie są jakby śmieszne. Standardy nie zmieniły się aż tak bardzo przez te kilka lat. Może bardziej upowszechniły ale na pewno nie zmieniły.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2024 Invision Power Services, Inc.