Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Site dla firmy zajmującej się głównie inst. sieci LAN
nitek
post
Post #1





Grupa: Zarejestrowani
Postów: 209
Pomógł: 37
Dołączył: 27.02.2008
Skąd: Kwidzyn

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


Witam,

przedstawiam kolejną stronkę do oceny. Proszę o konstruktywne opinie.
Tym razem dtd już trzyma się stricta (nie loose jak poprzednio (IMG:http://forum.php.pl/style_emoticons/default/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

Ten post edytował nitek 7.11.2008, 16:29:24
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 8)
nexis
post
Post #2





Grupa: Zarejestrowani
Postów: 1 012
Pomógł: 109
Dołączył: 26.09.2003
Skąd: nexis.pl

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


Grafika bardzo przyjemna, a jeśli chodzi o kod to jest dużo pustych wierszy, a poza tym użyłbym " w znacznikach zamiast '.
Go to the top of the page
+Quote Post
l0ud
post
Post #3





Grupa: Zarejestrowani
Postów: 1 387
Pomógł: 273
Dołączył: 18.02.2008

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


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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Layout powinien być w takim wypadku wycentrowany.
Go to the top of the page
+Quote Post
nitek
post
Post #4





Grupa: Zarejestrowani
Postów: 209
Pomógł: 37
Dołączył: 27.02.2008
Skąd: Kwidzyn

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


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' ..."; (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Layout na życzenie klienta do lewej - szczerze też mi trochę nie leży, ale cóż (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) alty będą - choćby ze względu na pozycjonowanie, to jest dopiero wersja robocza..
*l0ud - czemu "robione trochę na silę"? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował nitek 7.11.2008, 22:10:07
Go to the top of the page
+Quote Post
l0ud
post
Post #5





Grupa: Zarejestrowani
Postów: 1 387
Pomógł: 273
Dołączył: 18.02.2008

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


Chodziło głównie o te puste alty (IMG:http://forum.php.pl/style_emoticons/default/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ć.
Go to the top of the page
+Quote Post
nitek
post
Post #6





Grupa: Zarejestrowani
Postów: 209
Pomógł: 37
Dołączył: 27.02.2008
Skąd: Kwidzyn

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


Cytat(l0ud @ 7.11.2008, 21:12:54 ) *
Chodziło głównie o te puste alty (IMG:http://forum.php.pl/style_emoticons/default/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! (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
no i puste linie między wierszami też pousuwam w najbliższy poniedziałek (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował nitek 7.11.2008, 22:10:29
Go to the top of the page
+Quote Post
nexis
post
Post #7





Grupa: Zarejestrowani
Postów: 1 012
Pomógł: 109
Dołączył: 26.09.2003
Skąd: nexis.pl

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


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' ..."; (IMG:http://forum.php.pl/style_emoticons/default/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.
Go to the top of the page
+Quote Post
nitek
post
Post #8





Grupa: Zarejestrowani
Postów: 209
Pomógł: 37
Dołączył: 27.02.2008
Skąd: Kwidzyn

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


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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował nitek 8.11.2008, 07:53:11
Go to the top of the page
+Quote Post
nasty
post
Post #9





Grupa: Zarejestrowani
Postów: 634
Pomógł: 14
Dołączył: 27.05.2006
Skąd: Berlin

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


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 (IMG:http://forum.php.pl/style_emoticons/default/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

Ten post edytował nasty 8.11.2008, 22:46:09
Go to the top of the page
+Quote Post

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: 24.08.2025 - 19:54