Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Site dla firmy zajmującej się głównie inst. sieci LAN
Forum PHP.pl > Inne > Oceny
nitek
Witam,

przedstawiam kolejną stronkę do oceny. Proszę o konstruktywne opinie.
Tym razem dtd już trzyma się stricta (nie loose jak poprzednio winksmiley.jpg). Na stronce żadnych konkretnych tekstów jeszcze nie ma.
Testowane na FF, IE i Chrome pod win.
Jedyny bug jaki wyłapałem i nie poprawiłem jeszcze, to strzałka prowadząca na górę strony nie wyświetla się w ogóle pod IE 6 - w innych przeglądarkach jest ok.

roboczy adres stronki: http://id2it.pl/zero/lantek/index.php

pozdrawiam
nexis
Grafika bardzo przyjemna, a jeśli chodzi o kod to jest dużo pustych wierszy, a poza tym użyłbym " w znacznikach zamiast '.
l0ud
Dobre doctype, ale robione trochę na siłę. Co jak co, ale alty do obrazków w menu mogły by już być... Jeżeli chodzi o grafikę to ok, ale u mnie, na wyższej rozdzielczości całość przylega do lewej krawędzi przeglądarki, co nie wygląda za ciekawie winksmiley.jpg. Layout powinien być w takim wypadku wycentrowany.
nitek
Co do znaczników używam z przyzwyczajenia ', ponieważ wiele moich stron jest mocno obudowanych php i w przypadku echo zwyczajnie przyzwyczaiłem się do instrukcji typu echo "<img src='obrazek.jpg' ..."; winksmiley.jpg
Layout na życzenie klienta do lewej - szczerze też mi trochę nie leży, ale cóż smile.gif alty będą - choćby ze względu na pozycjonowanie, to jest dopiero wersja robocza..
*l0ud - czemu "robione trochę na silę"? smile.gif
l0ud
Chodziło głównie o te puste alty winksmiley.jpg Ale mogę też się przyczepić chociażby do braku nagłówków i innych specyficznych znaczników, które można by było wykorzystać.
nitek
Cytat(l0ud @ 7.11.2008, 21:12:54 ) *
Chodziło głównie o te puste alty winksmiley.jpg Ale mogę też się przyczepić chociażby do braku nagłówków i innych specyficznych znaczników, które można by było wykorzystać.


Alty i wszystkie tagi z head'a będą - obiecuje! smile.gif
no i puste linie między wierszami też pousuwam w najbliższy poniedziałek winksmiley.jpg
nexis
Cytat(nitek @ 7.11.2008, 20:58:09 ) *
Co do znaczników używam z przyzwyczajenia ', ponieważ wiele moich stron jest mocno obudowanych php i w przypadku echo zwyczajnie przyzwyczaiłem się do instrukcji typu echo "<img src='obrazek.jpg' ..."; winksmiley.jpg


Więc robisz podwójny błąd, ponieważ echo" title="Zobacz w manualu PHP" target="_manual powinno się używać tak:
  1. <?php
  2. echo '<img src="' . $filepath . '" alt="' . $alt . '" />';
  3. ?>

ponieważ między cudzysłowem " są wyszukiwane zmienne, a między ' już nie, co może w nieznacznym, ale jakimś stopniu przyśpiesza parsowanie.
nitek
Cytat(nexis @ 7.11.2008, 22:34:16 ) *
Więc robisz podwójny błąd,
ponieważ między cudzysłowem " są wyszukiwane zmienne, a między ' już nie, co może w nieznacznym, ale jakimś stopniu przyśpiesza parsowanie.


Ok, potwierdzam co napisałeś i przyznaje, że wcześniej tego nie doczytałem nigdzie (strona manuala wcale o tym nie mówi), ale potwierdziłem sobie to m.in. tutaj: http://binboy.sphere.pl/index.php?show=22

//edit - cudzysłowy poprawione na stronie, sekcja head również już uzupełniona, puste linie w większości pokasowane, no i alt'y już również dodane. będę wdzięczny za kolejne sugestie, opinie smile.gif
nasty
Cytat(nexis @ 7.11.2008, 22:34:16 ) *
Więc robisz podwójny błąd, ponieważ echo" title="Zobacz w manualu PHP" target="_manual powinno się używać tak: (...) ponieważ między cudzysłowem " są wyszukiwane zmienne, a między ' już nie, co może w nieznacznym, ale jakimś stopniu przyśpiesza parsowanie.

Nie zgodze sie z Toba smile.gif poniewaz tu doczepiajac kolejne elementy redeklarujesz ten string w pamieci niszczac kazda poprzednia czesc i tworzac tymczasowe zmienne z usunietym stringiem. to sie dzieje w srodku, w silniku PHP. Zobacz to.

Pozdrawiam
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.