Forum PHP.pl > Portfolio do oceny

Pomoc - Szukaj - Użytkownicy - Kalendarz

neosatan

16.04.2011, 19:57:54

http://c-harsh.com/
Strona ma długą genezę. Była przebudowywana chyba z cztery razy, więc przy okazji trochę burdel się zrobił. Normalnie bym nie podał jej do oceny, lecz chciałbym ją jeszcze dopracować, więc przy okazji trochę krytyki się przyda.

IceManSpy

16.04.2011, 20:47:20

1. Jak wyświetlasz spis zdjęć, to muszą one wyświetlać się tak, aby scroll był tylko w górę/dół. Nie może być z lewa na prawą (zwłaszcza na dużym monitorze - 19 cali)
2. Literki w menu za małe i są mało widoczne przez patternowe tło
3. Jak przechodzę do kontaktu, to nie mam menu - mogę tylko wrócić do strony głównej - trochę dziwne rozwiązanie, bo nagle myślałem, że przeszedłem na inną stronę.
4. Strona jest pusta i trochę chaotyczna.

blooregard

16.04.2011, 21:09:38

Cytat

1. Jak wyświetlasz spis zdjęć, to muszą one wyświetlać się tak, aby scroll był tylko w górę/dół. Nie może być z lewa na prawą (zwłaszcza na dużym monitorze - 19 cali)

Zwłaszcza na MAŁYM monitorze (15,4 )

Reszta uwag j/w.

Na plus zaliczyłbym prostotę strony (w sensie skupienia się na jej głównym celu - przedstawienia portfolio fotografa) i brak zbędnych wodotrysków.

Turson

16.04.2011, 22:16:30

4. zdjęcie od lewej w górnym rzędzie jest za wąskie do rozmiaru ramki.
W 'legal' także zanika menu, a kontakt strasznie, ale to strasznie chaotyczny

.radex

17.04.2011, 10:05:38

Przenieś CSS do osobnego pliku. Unikaj style="". width="" i height="" w zdjęciach też przenieś do CSS-u.

Javascript najlepiej przenieś do osobnego pliku i dołączaj skrypty na końcu dokumentu. Tutaj nie ma to wielkiego znaczenia, ale jak strona się rozrasta i skrypty są coraz większe, to warto.

Wywal

[HTML] pobierz, plaintext 
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/DTD/strict.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
[HTML] pobierz, plaintext

I użyj

[HTML] pobierz, plaintext 
<!DOCTYPE html>
<meta charset="UTF-8">
[HTML] pobierz, plaintext

Nie ma powodu, aby nie używać doctype'a HTML5.

neosatan

17.04.2011, 13:04:03

html 4.01 jest użyty z powodu dziwnego zachowania przeglądarek. Przy używaniu html5 występowały jakieś abberacje pod różnymi przeglądarkami, natomiast pod html 4.01 jest wszystko ok pod każdą.

zdjęcia, które wydaje się być za wąskie jest efektem automatycznego cropa. Cropy ulegną zmianie.

Oczywiście dziękuję za wskazanie css i javascriptów..

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.