Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www]Inspiroo.net
Szczuras
post 21.03.2012, 18:33:17
Post #1





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 7.11.2009

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


Proszę o ocenę serwisu www.inspiroo.net

Na początek kilka uwag co do samego projektu, przez to że mamy mocno ograniczone
zasoby czasowe, dostosowanie działania portalu do wszystkich przeglądarek jeszcze trochę
kuleje (zwłaszcza w ie).

Z góry dziękuję za wszelkie konstruktywne wypowiedzi.
Go to the top of the page
+Quote Post
prowseed
post 21.03.2012, 19:04:37
Post #2





Grupa: Zarejestrowani
Postów: 433
Pomógł: 64
Dołączył: 29.01.2011
Skąd: Warszawa

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


Powiem tak, jedyne, co mi się nie podoba na całej stronie pod wzgledem graficznym, to wyszukiwarka, która nie pasuje do konceptu. Reszta przejrzysta, fajna, lekka. Niektore teksty ciezko sie czyta (elementy na siebie nachodza lub sa bardzo blisko siebie)- tu do poprawy estetyka strony. Wg mnie nie potrzebne tooltipy w stylu: Mam przycisk 'back to top', najezdzam i widze 'wróć na górę'. Po co to? Tym bardziej, ze najechalem przypadkowo i utrudnilo mi to klikniecie w strone w paginacji (tzn akurat mowie o 'more today', ale obydwa tooltipy denerwuja). Czesc tekstow po polsku, czesc po angielsku. Zawsze nieudany mix. Dopiski w sidebarze 'popular' tworzysz na tabelkach?

Ogolnie na plus


--------------------
Go to the top of the page
+Quote Post
IceManSpy
post 21.03.2012, 19:09:25
Post #3





Grupa: Zarejestrowani
Postów: 1 006
Pomógł: 111
Dołączył: 23.07.2010
Skąd: Kraków

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


A o czym jest ta strona? Dla mnie brak jasnego przesłania jej treści.


--------------------
Go to the top of the page
+Quote Post
Szczuras
post 21.03.2012, 19:28:27
Post #4





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 7.11.2009

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


Cytat(prowseed @ 21.03.2012, 19:04:37 ) *
Powiem tak, jedyne, co mi się nie podoba na całej stronie pod wzgledem graficznym, to wyszukiwarka, która nie pasuje do konceptu. Reszta przejrzysta, fajna, lekka. Niektore teksty ciezko sie czyta (elementy na siebie nachodza lub sa bardzo blisko siebie)- tu do poprawy estetyka strony. Wg mnie nie potrzebne tooltipy w stylu: Mam przycisk 'back to top', najezdzam i widze 'wróć na górę'. Po co to? Tym bardziej, ze najechalem przypadkowo i utrudnilo mi to klikniecie w strone w paginacji (tzn akurat mowie o 'more today', ale obydwa tooltipy denerwuja). Czesc tekstow po polsku, czesc po angielsku. Zawsze nieudany mix. Dopiski w sidebarze 'popular' tworzysz na tabelkach?

Ogolnie na plus


Dzięki za uwagi, generalnie jeśli chodzi o wszelkie wyskakujące komunikaty po najechaniu też uważam, że są zbędne, ale ostateczna decyzja nie jest moja smile.gif, pewnie w bliskiej przyszłości one znikną, co do pomięszania języków to też już pracujemy nad tym aby to ujednolicić i wprowadzić wersjonowanie językowe dla całego serwisu (tu po drodze koncepcje się zmieniały dlatego jest jak jest). Co do tabelek, to z tego co pamiętam nigdzie ich nie ma, poza popupami, bo już nie miałem czasu i dobrego pomysłu jak to inaczej rozwiązać aby się dopasowywały ładnie do zawartości.

Cytat(IceManSpy @ 21.03.2012, 19:09:25 ) *
A o czym jest ta strona? Dla mnie brak jasnego przesłania jej treści.


myślę, że odpowiedź na to pytanie można doszukać się tu , lepiej bym tego nie ują smile.gif
Go to the top of the page
+Quote Post
prowseed
post 21.03.2012, 19:37:55
Post #5





Grupa: Zarejestrowani
Postów: 433
Pomógł: 64
Dołączył: 29.01.2011
Skąd: Warszawa

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


Cytat(Szczuras @ 21.03.2012, 19:28:27 ) *
Co do tabelek, to z tego co pamiętam nigdzie ich nie ma, poza popupami


nie kłam ; )

Cytat
  1. <div class="descPopular">
  2. <table><tr><td style="height: 33px;vertical-align: middle;">
  3. <a href="/articles/details/id/1598" title="Photo by Annette Pehrsson">Photo by Annette...</a>
  4. </td></tr></table>
  5. </div>


--------------------
Go to the top of the page
+Quote Post
Szczuras
post 21.03.2012, 19:42:03
Post #6





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 7.11.2009

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


Cytat(prowseed @ 21.03.2012, 19:37:55 ) *
nie kłam ; )


sory, nie przylukałem tego dobrze smile.gif
Go to the top of the page
+Quote Post
slammer
post 21.03.2012, 21:45:16
Post #7





Grupa: Zarejestrowani
Postów: 187
Pomógł: 6
Dołączył: 31.08.2005
Skąd: Bielsko-Biała

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


http://www.inspiroo.net/asd

Jak robicie na zendzie, to może warto pomyśleć o routerze? W każdym razie zróbcie coś z błędnymi urlami.
W nicku podałem apostrofy i nie mogę teraz dodawać komentarzy, bo wywala błąd JS.

http://www.inspiroo.net/articles/details/id/test jest sql injection.
http://www.inspiroo.net/ajax/likearticle POST podatny na sql injection
http://www.inspiroo.net/img/photo/w/400/h/110/?p=index.php fatal error
http://www.inspiroo.net/ajax/sharelist/id/%23 fatal error

Graficznie wg. mnie jest ok, ale przeglądnijcie to pod kątem bezpieczeństwa, bo póki co, to katastrofa tongue.gif

Ten post edytował slammer 21.03.2012, 21:47:25
Go to the top of the page
+Quote Post
Szczuras
post 22.03.2012, 08:07:13
Post #8





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 7.11.2009

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


Cytat(slammer @ 21.03.2012, 21:45:16 ) *
http://www.inspiroo.net/asd

Jak robicie na zendzie, to może warto pomyśleć o routerze? W każdym razie zróbcie coś z błędnymi urlami.
W nicku podałem apostrofy i nie mogę teraz dodawać komentarzy, bo wywala błąd JS.

http://www.inspiroo.net/articles/details/id/test jest sql injection.
http://www.inspiroo.net/ajax/likearticle POST podatny na sql injection
http://www.inspiroo.net/img/photo/w/400/h/110/?p=index.php fatal error
http://www.inspiroo.net/ajax/sharelist/id/%23 fatal error

Graficznie wg. mnie jest ok, ale przeglądnijcie to pod kątem bezpieczeństwa, bo póki co, to katastrofa tongue.gif


dzięki za uwagi, jeśli chodzi o błędne urle to jest to oczywiście zrobione, z testów zostały jeszcze wyłączone handlery błędów i nie wyłapałem tego,
wszelkie sprawy typu "...details/id/test" gdzie są podawane błędne dane do zapytań, mają generować błąd (docelowo będą tam oczywiście odpowiednie walidatory) i być wyłapane przez kontroler błędów,
na pewno jest jeszcze wiele do zrobienia w sprawie bezpieczeństwa, ale i tak myślę że nie jest źle smile.gif
Go to the top of the page
+Quote Post

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

 



RSS Wersja Lo-Fi Aktualny czas: 22.06.2025 - 17:58