Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

2 Stron V   1 2 >  
Reply to this topicStart new topic
> Ocena strony, Przy okazji test
Barcelona
post 1.08.2010, 01:31:22
Post #1





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


Witam, już prawie skończyłem stronkę i prosiłbym o waszą ocenę. Przy okazji może zauważycie jakieś błędy na stronie itd.

link: Vikor

Pozdrawiam
Go to the top of the page
+Quote Post
r4xz
post 1.08.2010, 07:59:05
Post #2





Grupa: Zarejestrowani
Postów: 673
Pomógł: 106
Dołączył: 31.12.2008

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


Opera:
Version
10.60 Internal

Platform
Linux


całość się sypie ino huczy...
a poza tym stopka mi tutaj nie pasuje (tzn. forma - obrazek jak i styl)


--------------------
Go to the top of the page
+Quote Post
Barcelona
post 1.08.2010, 08:56:22
Post #3





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


A mógłbyś dać screena? Chciałbym wiedzieć co poprawić.
Dziwne, że opera wywaliła strone, bo właśnie pod nią robiłem, tylko że nie na Linuxie. Myślałem, że IE tylko robi problemy a tu taka niespodzianka.

Dzięki i pozdrawiam
Go to the top of the page
+Quote Post
r4xz
post 1.08.2010, 09:32:35
Post #4





Grupa: Zarejestrowani
Postów: 673
Pomógł: 106
Dołączył: 31.12.2008

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


co dziwnego na ~60% strona wyświetla się poprawnie





PS zapomniałem dodać - rozdz. 1280x1024

Ten post edytował r4xz 1.08.2010, 09:43:18


--------------------
Go to the top of the page
+Quote Post
poki289
post 1.08.2010, 11:20:03
Post #5





Grupa: Zarejestrowani
Postów: 26
Pomógł: 2
Dołączył: 1.01.2010

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


U mnie na nieco starszej Operze 9.64 i rozdzielczości 1024x768 wszystko jest spoczko, do testowania strony pod różne przeglądarki polecam browsershots.
Ogólnie layout mi się podoba


--------------------
PHP-Fusion Polska | Metin2 | Forum Metin2
Go to the top of the page
+Quote Post
thomson89
post 1.08.2010, 13:34:39
Post #6





Grupa: Zarejestrowani
Postów: 1 178
Pomógł: 51
Dołączył: 7.01.2009
Skąd: Gdańsk

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


+ Nagłówek: Ładna tekstura
+ Kolorystyka poprawnie dobrana
+ Podobają mi się kolory

- "Ostatnio dodane": Zbyt ściśnięte napisy (w ramce)
- Wyszukiwarka: Pole text, powinno być wysokie na wysokość przycisku i trochę krótsze
- "O nas": Typografia leży
- "Wybrane ogłoszenia": Jak kliknę, na numerek niżej, to oczekuję że zdjęcie pojedzie niżej lub wyżej a nie na bok. Dodaj wartość do tinycarousel: axis: 'y'

? "Wybrane ogłoszenia": O co chodzi z tym remont?

-> Co do kolegów, Opera 10:60, roz. 1440x900 nie rozjeżdża się strona.



Ten post edytował thomson89 1.08.2010, 13:35:44


--------------------
Sklep 70%
Go to the top of the page
+Quote Post
Substr
post 1.08.2010, 14:01:15
Post #7





Grupa: Zarejestrowani
Postów: 58
Pomógł: 8
Dołączył: 22.06.2009

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


Przede wszystkim zmień login i hasło do panelu admina, bo jest banalne.
Go to the top of the page
+Quote Post
everth
post 1.08.2010, 15:14:35
Post #8





Grupa: Zarejestrowani
Postów: 782
Pomógł: 153
Dołączył: 21.07.2010

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


Trochę czepiactwa:
  • strona się rozjeżdża bo wpisy z menu wychodzą poza wysokość - dodawanie overflow:hidden do menu omija w jakiś sposób problem (tylko wpis menu staje się niewidoczny), można to obejść inaczej (przesuwając div 'm' do tagu body i centrując)
  • główna, oferta, kontakt - da się je zrobić normalnym tekstem a nie obrazkiem
  • zawartość divu 'right' wychodzi poza jego szerokość - normalnie nie widać, ale przy skalowaniu staje się widoczne
  • przejście tekstu w nagłówku - można za pomocą fadeIn, fadeOut, zrobić ładniejsze, płynne
  • subiektywne - zamiast białego tła można mu nadać jakąś delikatną fakturę, wrażenie pustki byłoby mniejsze

Chyba tyle, mechaniki nie chciało mi się sprawdzać.


--------------------
Już mi się ani wiedzieć, ani tym bardziej myśleć nie chce.
[Think different]!
Go to the top of the page
+Quote Post
O$iek
post 1.08.2010, 16:41:35
Post #9





Grupa: Nieautoryzowani
Postów: 45
Pomógł: 16
Dołączył: 28.02.2009

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


Wchodzę w dane ogłoszenie i nic nie wpisuje w formularzu kontaktowym, klikam wyślij i:
Cytat
List został wysłany poprawnie
Go to the top of the page
+Quote Post
Barcelona
post 1.08.2010, 16:54:30
Post #10





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


Dzięki za zainteresowanie tematem, na pewno będę się sugerował waszymi uwagami.
Widzę, że są wysyłane maila ze strony. Wiem że nie ma walidacji i będzie ona dodane później, bo to szczegół.


Odpowiedzi do @everth
Cytat
główna, oferta, kontakt - da się je zrobić normalnym tekstem a nie obrazkiem


Można, ale po co jak tak też może być.

Cytat
można to obejść inaczej (przesuwając div 'm' do tagu body i centrując)


Mógłbyś mi to zaprezentować bo nie wiem jak to zrobić

Cytat
zawartość divu 'right' wychodzi poza jego szerokość - normalnie nie widać, ale przy skalowaniu staje się widoczne


Wiem o tym, aktualnie nad tym walczę smile.gif

Cytat
subiektywne - zamiast białego tła można mu nadać jakąś delikatną fakturę, wrażenie pustki byłoby mniejsze


Szczerze nawet o tym nie pomyślałem, i nawet nie głupi pomysł - dzięki smile.gif

Odpowiedź do @Substr

No dane do logowania są banalne, ale nie miałem po co zmieniać bo na razie to tylko wersja robocza. No ale zmienię bo zaraz wszyscy zaczną kombinować

Odpowiedzi do @thomson89

Cytat
Wyszukiwarka: Pole text, powinno być wysokie na wysokość przycisku i trochę krótsze


Właśnie z tym walcze.

Cytat
"O nas": Typografia leży


Pisane wczoraj na szybko.

Cytat
"Wybrane ogłoszenia": Jak kliknę, na numerek niżej, to oczekuję że zdjęcie pojedzie niżej lub wyżej a nie na bok. Dodaj wartość do tinycarousel: axis: 'y'


Właśnie dlatego wydaje mi się że jest to ciekawe rozwiązanie - zaskakuje smile.gif

Cytat
? "Wybrane ogłoszenia": O co chodzi z tym remont?


Pisane też na szybko, miało być że w trakcie prac smile.gif

Odpowiedź do @r4xz

A mógłbyś zobaczyć inne podstrony bo podejrzewam że właśnie ta strona główna psuje.

Odpowiedź do @O$iek

Wiem, że nie ma walidacji, ale już dodałem taką prostą bo już ponad 100 pustych maili dostałem. Jak kliknie się wyślij to nie wyśle ale nie pokaże też co jest puste, jest to chwilowe rozwiązanie.


Pozdrawiam

Ten post edytował Barcelona 1.08.2010, 16:59:08
Go to the top of the page
+Quote Post
O$iek
post 1.08.2010, 16:58:35
Post #11





Grupa: Nieautoryzowani
Postów: 45
Pomógł: 16
Dołączył: 28.02.2009

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


Znowu w zakładce "Kontakty" nie ma żadnej informacji, czy wiadomość została wysłana, czy nie?
#edit:
Jeżeli nie wypełnimy nie zwraca informacji.

Ten post edytował O$iek 1.08.2010, 16:59:33
Go to the top of the page
+Quote Post
Barcelona
post 1.08.2010, 17:06:59
Post #12





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


Wiem, pisałem że chwilowo walidacja jest podstawowa.
Go to the top of the page
+Quote Post
thomson89
post 1.08.2010, 18:57:38
Post #13





Grupa: Zarejestrowani
Postów: 1 178
Pomógł: 51
Dołączył: 7.01.2009
Skąd: Gdańsk

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


Zaskakuje, zaskakuje - ale logicznie rzec biorąc nie pasuje i jest... nie logiczne smile.gif

-> Informator: Czcionka większa.

Ten post edytował thomson89 1.08.2010, 18:58:53


--------------------
Sklep 70%
Go to the top of the page
+Quote Post
r4xz
post 1.08.2010, 19:02:08
Post #14





Grupa: Zarejestrowani
Postów: 673
Pomógł: 106
Dołączył: 31.12.2008

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


na każdej tak samo, ponieważ (ktoś już to nawet napisał) link "Przewóz osób" wychodzi ze swojej belki. (na linuxie nie ma identycznych czcionek jak na windowsie. trzeba to brać pod uwagę przy projektowaniu stron winksmiley.jpg )


--------------------
Go to the top of the page
+Quote Post
everth
post 1.08.2010, 20:18:59
Post #15





Grupa: Zarejestrowani
Postów: 782
Pomógł: 153
Dołączył: 21.07.2010

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


@Barcelona: wymagałoby to trochę większej kombinacji przy układzie strony. Ja widzę to tak:
  • likwidacja kontenera centrującego
  • dodanie:
    1. body > * {margin:0 auto;}
  • menu masz jako listę - fajnie, wywal otaczający ją div i nadaj id='menu' znacznikowi ul
  • elementom li zawierającym pozycję menu nadaj klasę 'menu-item'
  • stwórz krawędzie belki menu - te wygibasy na krawędziach ,będą one w statycznym elemencie li - po prostu w szablonie umieścisz li o id='first' i 'last'
  • stwórz background pod 'menu-item', powinien być to plik z tym niebieskiem wzorem co masz na górze

Przykładowy html
  1. <div id='header'>
  2. <ul id='menu'>
  3. <li id="menu-first" />
  4. <li class='menu-item'><a>costem</a></li>
  5. (...)
  6. <li id='menu-last' />
  7. </ul>
  8. <div id='pageContent' />
  9. <div id='footer' />
  10. </body>

CSS:
  1. body {
  2. min-width: 1024px; /* zabezpiecza przed niskimi rozdzielczościami */
  3. }
  4.  
  5. body > * {
  6. margin: 0 auto; /* centruje wszystkie elementy w body */
  7. }
  8.  
  9. #menu {
  10. text-align: center; /* centruje menu */
  11. }
  12. #menu li {
  13. line-height:30px;
  14. }
  15. #menu li.menu-item {
  16. background-image: obraz pod linkami w menu (faktura);
  17. background-repeat:repeat;
  18. }
  19. #menu-first,#menu-last {
  20. width:15px;
  21. background-repeat:no-repeat;
  22. }
  23. #menu-first {
  24. background-image:lewa krawędź;
  25. }
  26. #menu-last {
  27. background-image:prawa krawędź;
  28. }

Wydaje mi się że przy takim ułożeniu gdyby elementy menu wychodziły poza krawędź to po prostu włączałby się pasek przewijania na dole. Ale to nie testowany układ. Ponadto ograniczaj się z floatami, w miarę możliwości zastępuj je display:inline lub display:inline-block (+poprawka na IE, niestety) - prawdopodobnie dlatego masz problem z wychodzącą zawartością bo w którymś divie który zawiera elementy float, nie dałeś height:auto i overflow:hidden.


--------------------
Już mi się ani wiedzieć, ani tym bardziej myśleć nie chce.
[Think different]!
Go to the top of the page
+Quote Post
Spawnm
post 1.08.2010, 20:24:30
Post #16





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




denerwuje mnie zmieniający się tekst w topie , porusza stroną na ff pod linuxem + złe kodowanie : pó�niejszych
Go to the top of the page
+Quote Post
Barcelona
post 1.08.2010, 23:33:08
Post #17





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


@everth, wielkie dzięki, bardzo dobry pomysł z tym menu. Pokombinuje z tym, tylko że na localu żeby tam nic nie nabroić.

Słuchaj, w css miałem już w body margin:0 tyle że bez auto
  1. body{ margin:0; padding:0;


Dodałem teraz poprostu auto do marginesu

  1. margin: 0 auto; padding:0;


I jeszcze dla pewności dodałem do każdego float to co napisałeś, czyli "height:auto i overflow:hidden"

@Spawnm - właśnie to kodowanie jest moją zmorą. Żałuję, że nie zrobiłem UTF od początku i teraz właśnie skaczą takie krzaki ;/ Zabierając się za ten projekt moja wiedza z programowania była bliska zeru więc dopiero w trakcie prac dostrzegłem minusy w iso. Teraz żeby przerobić to na utf to bedę musiał od nowa zmieniać kodowanie całego serwisu i całej bazy (baza to nie problem na szczęście). Na pewno przejde na utf, ale to przy następnej odsłonie.

A o tym że ten tekst porusza całą stroną to nawet nie pomyślałem, dziwne rzeczy się dzieją na linuxie. Masz może jakąś radę na ten tekst?
Go to the top of the page
+Quote Post
everth
post 2.08.2010, 00:00:20
Post #18





Grupa: Zarejestrowani
Postów: 782
Pomógł: 153
Dołączył: 21.07.2010

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


@Barcelona:
  1. body > * {margin:0 auto}

Nie nadajesz go tagowi body tylko jego dzieciom pierwszego poziomu. Poza tym on służy do centrowania w poziomie divów które mają stałą szerokość. "height:auto..." nie nadajesz obiektom mającym atrybut float tylko tym divom które zawierają divy z atrybutem float (kontenerom).


--------------------
Już mi się ani wiedzieć, ani tym bardziej myśleć nie chce.
[Think different]!
Go to the top of the page
+Quote Post
Barcelona
post 2.08.2010, 14:01:03
Post #19





Grupa: Zarejestrowani
Postów: 322
Pomógł: 15
Dołączył: 29.01.2010

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


OK, rozumiem. Dodatkowo zawęziłem obszar z menu, który się właśnie rozjeźdzał. Zaoszczędzieł na tym pare cm na różnicę czcionek pomiędzy linuxem i windowsem.
Tylko teraz mam pytanie, jak wycentrować to menu żeby było na środku a nie żeby zaczynało się od lewej krawedzi ? Center nie działa smile.gif

Pozdrawiam
Go to the top of the page
+Quote Post
r4xz
post 2.08.2010, 14:23:22
Post #20





Grupa: Zarejestrowani
Postów: 673
Pomógł: 106
Dołączył: 31.12.2008

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


  1. <div id="center">
  2. <ul>
  3. <li>1...</li>
  4. <li>2...</li>
  5. <li>3...</li>
  6. <li>4...</li>
  7. </ul>
  8. </div>


  1. #center {
  2. width: 100%;
  3. overflow: hidden;
  4. position: relative;
  5. }
  6.  
  7. #center ul {
  8. float: left;
  9. text-align: center;
  10. position: relative;
  11. left: 50%;
  12. }
  13.  
  14. #center ul li {
  15. float: left;
  16. display: block;
  17. position: relative;
  18. right: 50%;
  19. }


--------------------
Go to the top of the page
+Quote Post

2 Stron V   1 2 >
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: 19.06.2024 - 14:34