[www] Portfolio Opcje

[owi]

Czuję ze mnie zjecie, ale coz. Warto spróbować.


jest to pierwsza strona w której nie musze patrzec na to co klient powiedział, na to co komus sie bedzie podobac, a co zonie pana Janka.
jak wiadomo najtrudniej sie robi dla siebie. Miliony pomysłów, tysiace poprawek. Dno , zle , slabo - nie wystarczy.

Niestety musielismy siegnac po jakiś kompromis i w koncu ustalić pewien poziom funkcjonalnosci ktorego bedziemy sie trzymac, gdyż w przeciwnym wypadku strona nigdy by się nie ukazała online. Zawsze byloby cos do rozbudowania...
Nad dopieszczaniem detali spedzilismy mase godzin, cale portfolio wraz z tekstami zjadlo kilkaset. Kiedys musi byc STOP

Jest to wersja powiedzmy release candidate, ale jeszcze mam troszkę poprawek i moga zdarzac sie bledy. Niestety aby zyc musze wykonywac projekty na zlecenie a nie patrzeć czy aby na pewno na kazdej stronie wszystko się waliduje i czy w ie 5.0 wyswietla sie jakies gowno. Jednak z czasem na bank zostaną załatane, zas funckjonalnosci rozbudowane.


Po przydlugim wstepie zapraszam
http://lowgravity.pl




p.s Badzcie mili (IMG:style_emoticons/default/winksmiley.jpg) Przeczytac najpierw wstep prosze (IMG:style_emoticons/default/winksmiley.jpg)

Idac z duchem czasu mozecie nawet nas LAJK na fejsbuku (IMG:style_emoticons/default/smile.gif)
http://www.facebook.com/LowGravitypl

Ten post edytował owi 10.08.2010, 22:14:58

[erix]

Hmm, fajny projekt, dawno tutaj w takim stylu widziałem.

Ale że jestem starym zrzędą, zawsze coś znajdę (czytaj: nie bierz tego tak mocno do siebie). (IMG:style_emoticons/default/tongue.gif)

• div.clear IMHO jest zbędne; generated content załatwiłoby sprawę (tym bardziej, że zleliście stare IE)
• JS powinien być w pliku zewnętrznym
• w paru miejscach są style liniowe, ale domniemam że to z powodu użycia jakiegoś WYSIWYG w panelu administracyjnym (co nie zmienia faktu, że można dopiąć tam własne style jak w procesorach tekstów (IMG:style_emoticons/default/winksmiley.jpg) )
• brakuje mi trochę elementów AJAX i DHTML (zwłaszcza *boksów przy powiększaniu obrazków, etc, walidacji formularzy po stronie klienta)
• trochę to ciężkie (choć wizualnie ciekawe, ale IMHO można by popracować nad optymalizacją)

Końcowe: 8.5/10

[Pawel_W]

patrząc w kod - na pierwszy rzut oka widać, że w ogóle nie stosujesz paragrafu (<p>) do tekstu, za to widać nadmiar div'ów (IMG:style_emoticons/default/smile.gif)

validator nie szaleje, brakuje kilku rzeczy, ale ok (IMG:style_emoticons/default/smile.gif)

poza tym, widać trochę styli lokalnych, które powinny być w arkuszu (IMG:style_emoticons/default/smile.gif)

co do designu - ogólnie ok, w kilku miejscach nie podoba mi się czcionka (m.in. widgety po prawej - facebook itp.)

podsumowując bardzo ładna strona (IMG:style_emoticons/default/smile.gif)

EDIT:
no i jeszcze jakiś hover w menu na górze by się przydał, tak z przyzwyczajenia do tego typu "bajerów" (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował Pawel_W 10.08.2010, 22:28:40

[owi]

- cleary moglem zalatwic chociazby afterem, wiem ze semantycznie brzydkie - ale po prostu to zlałem - wolę tak - w koncowym rozrachunku zerowe znaczenie (no prawie oprocz tych paru bajtów)
- style inline tylko i wylacznie ze wzgledu an wysiwiga (IMG:style_emoticons/default/smile.gif) cala reszta tylko i wylacznie w zewnetrznym - juz o takie przedszkole proszę nei oskarzac. Wysiwyg rowniez odpowiedzialny glwonie za bledy w walidatorze o ile gdzies są
- zewnetrzny JS - tutaj polemizowalbym czy za pare linijek warto dodatkowego requesta sobei tworzyc.
- odnosnie braku powiekszaczy - nie ma i nie bedzie w najblizszej przyszlosci -swiadomy wybor
- p nie ma z prostych wzgledow - podobnie jak w przypadku inline css - wszystkie teksty (prawie) na stornie to wysiwyg. Moi dwaj koledzy nie znaja na tyle htmla, cssa aby optymalizowac kazdy tekst ktory wprowadzą.


generalnie czeka na mnie pare prac optymalizacyjnych. PYTANIE KIEDY? optymalizacja wysiwyga nie wyzywi mi rodziny w zadnym stopniu (IMG:style_emoticons/default/smile.gif)



Dziekuje ze przychylne opinie

Ten post edytował owi 10.08.2010, 22:31:43

[erix]

- cleary moglem zalatwic chociazby afterem, wiem ze semantycznie brzydkie - ale po prostu to zlałem - wolę tak - w koncowym rozrachunku zerowe znaczenie (no prawie oprocz tych paru bajtów)

Czemu brzydkie semantycznie? Właśnie jest to dobre rozwiązanie, a to z tego powodu, że nie uzależniasz funkcji warstwy od wyglądu. after wykorzystałbyś w celu zmiany wyglądu przez CSS i dodawałbyś przez CSS.

- zewnetrzny JS - tutaj polemizowalbym czy za pare linijek warto dodatkowego requesta sobei tworzyc.

Owszem, warto, bo przeglądarka posiada coś takiego, jak cache, z którego wykorzysta ten sam plik na wszystkich podstronach Twojego serwisu. (IMG:style_emoticons/default/tongue.gif) Oszczędność transferu wzrasta wprost proporcjonalnie do liczby odwiedzonych.

- odnosnie braku powiekszaczy - nie ma i nie bedzie w najblizszej przyszlosci -swiadomy wybor

Szkoda, bo są dość wygodne.

optymalizacja wysiwyga nie wyzywi mi rodziny w zadnym stopniu

No nie, ale dałeś stronę do oceny, to oceniamy. (IMG:style_emoticons/default/winksmiley.jpg) A kod/wygląd przechodzi przez standardową procedurę oceniania, czyli czepianie się wszelkich możliwych szczegółów. (IMG:style_emoticons/default/winksmiley.jpg)

[owi]

Tutaj ja mowie o moim rozwiazaniu jako brzydkim semantycznie - czyli puste cleary. Po prostu w tym wypadku zlalem te clearfixy bo tak mi bylo wygodniej (IMG:style_emoticons/default/biggrin.gif)

Dzieki wam poprawilem pare najprostszych bledow w walidacji ktore powstaly calkiem niedawno przy dolepianiu paru rzeczy (IMG:style_emoticons/default/winksmiley.jpg)

Owszem, warto, bo przeglądarka posiada coś takiego, jak cache, z którego wykorzysta ten sam plik na wszystkich podstronach Twojego serwisu. tongue.gif Oszczędność transferu wzrasta wprost proporcjonalnie do liczby odwiedzonych.

Tak oczywiscie zgoda, ale nadal jest to swiadomy wybor gdy portfolio jest w rozkroku pomiedzy faza developerska a dzialaniem w pelni produkcyjnym. Zwlaszcza ze jak na razie szalu odwiedzin nie ma. Na miliony zadnych wiedzy juzerow trzeba poczekac (IMG:style_emoticons/default/smile.gif)

Szkoda, bo są dość wygodne.

Ano wygodne, ale nie kazdy projekt moze posiadac powiekszane zdjecie, na niektorych klient nie zgada ssie na wykorzystanie loga i tak wyszlismy z zalozenia ze na razie mamy tlyko thumbnaile...

No nie, ale dałeś stronę do oceny, to oceniamy. winksmiley.jpg A kod/wygląd przechodzi przez standardową procedurę oceniania, czyli czepianie się wszelkich możliwych szczegółów. winksmiley.jpg

No i licze na to ze dobrze wypatrzycie wszystko

Bycie jedynym developerem w firmie i majac tyle na glowie latwo doprawdy cos przegapić badz machnąć reką



P.S.

no i jeszcze jakiś hover w menu na górze by się przydał, tak z przyzwyczajenia do tego typu "bajerów" winksmiley.jpg

Wiesz ile takich bajerow czeka nieskonczonych? (IMG:style_emoticons/default/biggrin.gif) Jest i hover, jest i colorbox chociazby (vide powiekszacze). Duzo z nich nie przeszlo procesu wydziwiania w naszym malym kregu. Duzo czeka na implementacje - jest to 5 min, ale pogodzic gusta 3 osob, zaprojektowac to schodzi, a potem i tak trzeba zajac sie czyms innym i wypada z glowy

Ten post edytował owi 10.08.2010, 22:44:06

[Twist]

Wg. mnie - straszny chaos, nie wiem na czym ma uwagę skupić, świeci się jak choinka. Jednak widać ręke profesjonalisty (IMG:style_emoticons/default/winksmiley.jpg) Poza "przesyceniem" podoba mi się styl, dobór kolorów i ta czcionka "Lowgravity sets you with HIGH standards for your web NEEDS" - podzielisz się nazwą? (IMG:style_emoticons/default/smile.gif)

Gdyby nie przesyt byłoby 10/10 a tak to tylko 9.5/10 (IMG:style_emoticons/default/biggrin.gif)

[SHiP]

To tak w kilku punktach:

• Plus za nie wspieranie IE6 (IMG:style_emoticons/default/winksmiley.jpg)
• Semantyka całkiem całkiem ale puste divy z clearem gryzą
• Zmieńcie grafika albo kupcie mu książkę na temat typografii. Nie wiem czy za granicą, tam gdzie pracujecie, się takie rzeczy ludziom podobają ale ja trochę montuję na rynek zagraniczny i imho wygląda to niemodnie(zresztą sporo z waszych innych produkcji również). Ale to moje widzimisię
• Czemu Transitional? Używacie czegoś, co jest niedozwolone w Strict?
• Nieoptymalny kod CSS np.
  
  Kod
      background-image:url(../images/background/footer.jpg);
      background-repeat:repeat-x;
      background-position:center 0;

@erix: ale, żeby sprawdzić czy cache jest aktualny trzeba odpytać serwer o informacje dot. pliku. Nie zawsze jest to robione ale jednak. Nie zmienia to jednak faktu, że ja również ten kod wrzuciłbym do oddzielnego pliku JS ;]


Ten post edytował SHiP 10.08.2010, 23:05:25

[erix]

@erix: ale, żeby sprawdzić czy cache jest aktualny trzeba odpytać serwer o informacje dot. pliku.

Z tego, co pamiętam, to można wysłać takie nagłówki, które uśpią to sprawdzanie. (IMG:style_emoticons/default/winksmiley.jpg)

[owi]

Usiadlem z rana troszkę wczesniej na pol godzinki i zrobilem parę zmian
- wyciagnalem JSa na zewnatrz
- wrzucilem clearfixa

Zmieńcie grafika albo kupcie mu książkę na temat typografii. Nie wiem czy za granicą, tam gdzie pracujecie, się takie rzeczy ludziom podobają ale ja trochę montuję na rynek zagraniczny i imho wygląda to niemodnie(zresztą sporo z waszych innych produkcji również). Ale to moje widzimisię

prosiłbym bez takich komentarzy. Nawet jezeli gdzies typografia kuleje (gdzie się w paru miejscach zgodzę), co oczywiscie nie jest dobre to nadal nie oznacza to "zmiany" grafika gdyż to zaledwie ułamek z jego zadań. Nikt nie jest dobry we wszystkich aspektach. Genialne umiejetnosci Darka w wielu dziedzinach grafiki przewazają jakiekolwiek wady.

Czemu Transitional? Używacie czegoś, co jest niedozwolone w Strict?

Tak oczywiscie - targety w linkach chociazby..
Zaraz jednak sie ozywi dyskusja od tych co tych wyzej wspomnianych książek czytaja więcej i będą elaboraty na temat jak to użytkownikowi nie wolno narzucać tego gdzie okno ma się otwierać.

Boję się poza tym tego co czasem może wyjść z WYSIWYGA.

Nieoptymalny kod CSS np.

Sztuka dla sztuki. Jako ze to jest portfolio wiec wiecznie bedzie w fazie developement - wolę miec przejrzystość tak abym ja sie w tym połapał. Zaoszczędzenie 1KB na optymalizacji cssa nie da ŻADNYCH wymiernych efektów.

Ten post edytował owi 11.08.2010, 08:23:12

[wookieb]

Kod

A PHP Error was encountered

Severity: Notice

Message: Array to string conversion

Gdzie? To już wasza broszka (IMG:style_emoticons/default/smile.gif)
Strona ładna ale nie bardzo ładna. (monita Imaca jest tak cholernie dziwny, że aż boli)

Nie oceniam html i js bo mi się nie chce (IMG:style_emoticons/default/smile.gif) Aczkolwiek jest to zdecydowanie praca z top5 w tym dziale więc ocena 8.5/10

[owi]

eee podpowiedzialbys (IMG:style_emoticons/default/winksmiley.jpg)

[wookieb]

Tak to jest jak korzysta się z gotowców i myśli sie, że jest "zajebiście".
Formularz -> zmiana nazwy pola na nazwę tablicową (firebug), ale dzięki temu że pokazujesz błędy wiem z czego korzystasz (pomijając stopkę) co umożliwia mi lepsze szukanie luk.

[owi]

Jak pisalem we wstępie - jest to wersja RC do finalnego wylapwyania bledow - dlatego jest pozostawiony error_reporting zeby jak najwiecej ich wylapac w tym stadium.


Odnosnie tych "gotowców" - chociaz takiego okreslenia w stosunku do CI bym nie uzyl. Jakie to ma znaczenie, że za pomocą firebuga oszukasz / wykryjesz lukę w zwykłym form_validation w formularzu kontaktowym? Powaznie sie teraz pytam - jakie to niesie za sobą konsekwncje? Oprócz wyswietlenia błędu wraz ze sciezką etc (dlaczego błędy są włączone pisałem wyżej). Nie jest to zadne odparcie ataku tylko realne pytanie.

Ten post edytował owi 11.08.2010, 09:28:57

[Pawel_W]

po zmianie z tym clear fixem pierwszy box z portfolio mi się rozwala, cała treść z obrazkiem jest po prawej (IMG:style_emoticons/default/smile.gif)

[wookieb]

Jest to jeden z elementów oszukania walidacji. Może się okazać, że w pewnym momencie w aplikacji chcesz wykorzystać tablicę (którą Ci już podałem) a ty jej nie walidujesz. Możliwości jest wiele w zależności od kodu.

[owi]

po zmianie z tym clear fixem pierwszy box z portfolio mi się rozwala, cała treść z obrazkiem jest po prawej (IMG:style_emoticons/default/smile.gif)

moglbys powiedziec na jakiej przegladarce? Zaraz sie temu przyjrze jako ze przyznam szczerze po zmianie nie sprawdzilem wszystkiego na kazdej platformie. Dzieki

Jest to jeden z elementów oszukania walidacji. Może się okazać, że w pewnym momencie w aplikacji chcesz wykorzystać tablicę (którą Ci już podałem) a ty jej nie walidujesz. Możliwości jest wiele w zależności od kodu.

oczywisce, tylko w przypadku gdy ten formularz nigdy nie ma zadnej stycznosci z bazą ani innymi newralgicznymi rzeczami (ba nic po stronie uzytkownika nie ma takiej stycznosci) w tym konkretnym wypadku nie ma to najmniejszego znaczenia (wg mnie). Ja rozumiem sam fakt oszukania walidacji jako niedobry, ale przypomina mi się szczerze scena z Misia "a co by bylo gdyby tu przechodziła wasza matka"...

W kazdym badz razie - rozbuduję walidację - dzieki.

[Pawel_W]

najnowszy Chrome (IMG:style_emoticons/default/smile.gif)

(IMG:http://i34.tinypic.com/fu181u.jpg)

[owi]

dziwne no za cholere na chromie tego u mnie nie ma, sprawdzalem i rano i teraz... Jeszcze poszukam jak nie to naloze height na ourservices. Dzieki (IMG:style_emoticons/default/winksmiley.jpg)



edit:

No sprawidzlem na paru kompach na chrome i nic. Moze u Ciebie cos z kaszy podlapalo?

Ten post edytował owi 11.08.2010, 09:59:09

[Pawel_W]

rzeczywiście, po usunięciu pamięci jest ok (IMG:style_emoticons/default/smile.gif)

[owi]

Ok super (IMG:style_emoticons/default/winksmiley.jpg)




@wookieb

obudowałem walidacje, zmienilem poziomy raportowania bledow - powinno byc ok (o ile dobrze zreplikowałem Twe działania) - mógłbyś sprawdzić?. Dzieki za uwagi

Ten post edytował owi 11.08.2010, 10:49:37

[gigzorr]

teraz formularz juz jest ok "moim zdaniem" , a co do strony to ladna praca (IMG:style_emoticons/default/smile.gif)

Ten post edytował gigzorr 11.08.2010, 14:00:41