Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Ocena strony
Forum PHP.pl > Inne > Oceny
Barcelona
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
r4xz
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)
Barcelona
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
r4xz
co dziwnego na ~60% strona wyświetla się poprawnie





PS zapomniałem dodać - rozdz. 1280x1024
poki289
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
thomson89
+ 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.

Substr
Przede wszystkim zmień login i hasło do panelu admina, bo jest banalne.
everth
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ć.
O$iek
Wchodzę w dane ogłoszenie i nic nie wpisuje w formularzu kontaktowym, klikam wyślij i:
Cytat
List został wysłany poprawnie
Barcelona
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
O$iek
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.
Barcelona
Wiem, pisałem że chwilowo walidacja jest podstawowa.
thomson89
Zaskakuje, zaskakuje - ale logicznie rzec biorąc nie pasuje i jest... nie logiczne smile.gif

-> Informator: Czcionka większa.
r4xz
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 )
everth
@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.
Spawnm
denerwuje mnie zmieniający się tekst w topie , porusza stroną na ff pod linuxem + złe kodowanie : pó�niejszych
Barcelona
@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?
everth
@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).
Barcelona
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
r4xz
  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. }
everth
Małe poprawki na CSSa:
  1. #menu {
  2. padding:0;
  3. margin:0;
  4. text-align: center; /* centruje menu */
  5. line-height:30px;
  6. }
  7. #menu li {
  8. display:inline;
  9. }
  10. #menu li.menu-item {
  11. background-image: obraz pod linkami w menu (faktura);
  12. background-repeat:repeat;
  13. }
  14. #menu-first,#menu-last {
  15. width:15px;
  16. height:30px;
  17. display:inline-block !important;
  18. background-repeat:no-repeat;
  19. vertical-align:top;
  20. }
Barcelona
Cytat(r4xz @ 2.08.2010, 15:23:22 ) *
  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. }



Teraz to mnie IE zaskoczyło smile.gif w MSIE centruje się na środku, tak jak miało być, a w Operze menu jest bardziej na prawo ;/

@everth
Dzięki, zastosuje się do Twoich poprawek
Magic Hereos
Cytat
O kontakt telefoniczny prosimy w godzinach pracy bióra, w późniejszych godzinach preferujemy kontakt elektroniczny.

Jak coś się robi, to bez błędów, szczególnie ortograficznych tongue.gif
Barcelona
Każdemu może się zdarzyć smile.gif Dzięki, już poprawiłem
Magic Hereos
Cytat(Barcelona @ 2.08.2010, 19:56:34 ) *
Każdemu może się zdarzyć smile.gif Dzięki, już poprawiłem

Wiem, ale lepiej to wyłapać na właśnie takim forum, niż żeby potencjalny klient zobaczył błąd smile.gif
Barcelona
Dlatego dałem do ocenienia smile.gif

Odnośnie kilku poprzednich postów związanych z problemem rozchodzącego się menu na różnych przeglądarkach, to chyba rozwiązałem ten problem w taki sposób:
1. Umieściłem menu w dodatkowym divie.
2. Na tego diva dałem taki styl

  1. .cent{
  2. position:absolute;
  3. width:999px;
  4. top:5px;
  5. right:-26px;
  6. }


Mam nadzieje że jest to zgodne z aktualnymi standardami smile.gif
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-2025 Invision Power Services, Inc.