Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Portfolio
kamil_lk
post
Post #1





Grupa: Zarejestrowani
Postów: 325
Pomógł: 33
Dołączył: 31.05.2008

Ostrzeżenie: (0%)
-----


Witam.
Proszę o jakieś słowo na temat mojego portfolio.
Odwiedź moją stronę

Ten post edytował kamil_lk 8.08.2011, 01:42:46
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
klocu
post
Post #2





Grupa: Zarejestrowani
Postów: 291
Pomógł: 45
Dołączył: 21.08.2007

Ostrzeżenie: (0%)
-----


Cukierkowo, ale fajnie - pierwsze wrażenie.

Strona główna - radziłbym przebudować troszkę koncepcję tego bloku z treścią. Szczególnie jeśli chodzi o ten czarny blok - pusty i nie bardzo wiadomo co się z nim / w nim dzieje i jakie jest jego zadanie w tej części strony.
Portfolio - przekazujesz nazwę projektu jako parametr $_GET - a ja wpiszę sobie cokolwiek i dostaje szablon taki, jakby projekt o wymyślonej przeze mnie nazwie istniał
Kontakt - jak nic nie wpiszę do formularza - to dlaczego komunikat o błędzie pojawia mi się w polu "Tytuł wiadomości"?
Dalej - nie sprawdzasz czy mail jest poprawny pod względem składni (przechodzi "." - kropeczka)
Po podaniu jednej błędnej danej wszystko to co wpiszę idzie w cholerę, ta kwestia była już wielokrotnie przerabiana na forum przy okazji różnych projektów.

To samo co przy portfolio. Wpisanie wymyślonej nazwy podstrony pokazuje szablon jakby strona istniała, tyle że bez treści.

--
  1. <script type="text/javascript">
  2. // <![CDATA[
  3. new Image().src = 'graf/kontent-2.png';
  4. new Image().src = 'graf/kontent-3.png';
  5. new Image().src = 'graf/logo.png';
  6. new Image().src = 'graf/tlo.jpg';
  7. new Image().src = 'graf/menu-1.png';
  8. new Image().src = 'graf/menu-2.png';
  9. new Image().src = 'graf/menu-3.png';
  10. new Image().src = 'graf/menu-1-hover.png';
  11. new Image().src = 'graf/menu-2-hover.png';
  12. new Image().src = 'graf/menu-3-hover.png';
  13. new Image().src = 'graf/przycisk-wyslij.png';
  14. new Image().src = 'graf/przycisk-wyslij-hover.png';
  15. new Image().src = 'slider/skins/tango/przycisk-przewijania-lewy.png';
  16. new Image().src = 'slider/skins/tango/przycisk-przewijania-lewy-hover.png';
  17. new Image().src = 'slider/skins/tango/przycisk-przewijania-prawy.png';
  18. new Image().src = 'slider/skins/tango/przycisk-przewijania-prawy-hover.png';
  19. // ]]>


Czekaj, czekaj - a co to za cudak? Taki śmieszny preloader grafik, żeby były wczytane od razu ze wszystkim?
Z własnych obserwacji nie zauważyłem znaczących opóźnień jeśli wczytuję je tylko poprzez CSS.
Go to the top of the page
+Quote Post
Damonsson
post
Post #3





Grupa: Zarejestrowani
Postów: 2 355
Pomógł: 533
Dołączył: 15.01.2010
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----


  1. <script type="text/javascript">
  2. // <![CDATA[
  3. new Image().src = 'graf/kontent-2.png';
  4. new Image().src = 'graf/kontent-3.png';
  5. new Image().src = 'graf/logo.png';
  6. new Image().src = 'graf/tlo.jpg';
  7. new Image().src = 'graf/menu-1.png';
  8. new Image().src = 'graf/menu-2.png';
  9. new Image().src = 'graf/menu-3.png';
  10. new Image().src = 'graf/menu-1-hover.png';
  11. new Image().src = 'graf/menu-2-hover.png';
  12. new Image().src = 'graf/menu-3-hover.png';
  13. new Image().src = 'graf/przycisk-wyslij.png';
  14. new Image().src = 'graf/przycisk-wyslij-hover.png';
  15. new Image().src = 'slider/skins/tango/przycisk-przewijania-lewy.png';
  16. new Image().src = 'slider/skins/tango/przycisk-przewijania-lewy-hover.png';
  17. new Image().src = 'slider/skins/tango/przycisk-przewijania-prawy.png';
  18. new Image().src = 'slider/skins/tango/przycisk-przewijania-prawy-hover.png';
  19. // ]]>

Cytat
Czekaj, czekaj - a co to za cudak? Taki śmieszny preloader grafik, żeby były wczytane od razu ze wszystkim?
Z własnych obserwacji nie zauważyłem znaczących opóźnień jeśli wczytuję je tylko poprzez CSS.

A to żaden cudak, dlaczego śmieszny? I dlaczego się tego czepiasz? To dość trendy sposób (IMG:style_emoticons/default/smile.gif)


Co do oceny strony, graficznie tylko, jest jakiś za duży kontrast, tło takie blade, a ten niebieski daje po oczach jak naga Carmen Electra (IMG:style_emoticons/default/wink.gif) i ogólnie jest brzydko. Wygląda jakbyś top i stopkę wziął z jakiejś fajnej strony, a od siebie zrobił content. Bo top i stopka mi się podoba, ale kompletnie nie współgra ze środkiem.
Chciałem sprawdzić podatność na XSS w formularzu i wysłało mi formularz z bzdurnymi danymi, waliduj chociaż e-mail. A odpowiedź "Wiadomość została wysłana" czy jak to tam brzmi, daj gdzieś poza formularzem, a nie w inpucie.
Używasz tylko jCarousel, by wklejać całego jquery. W sumie nawet nie widzę, gdzie z tego jCarousel korzystasz.
Typografia leży... "Swoją pasję do tworzenia stron internetowych kreuję od lat gimnazjalnych, kiedy to wykonałem swoją pierwszą stronę wykorzystującą język html." do tego czcionka jest jakaś brzydka.
No i jak już to będziesz miał w wersji finalnej, to wrzuć na jakiś szybszy serwer, bo ten nie ogarnia.

Ten post edytował Damonsson 8.08.2011, 11:26:37
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 12.10.2025 - 17:14