Witam

Proszę o ocenę wyglądu, funkcjonalności oraz wszystkich innych rzeczy jakie znajdują się tutja:
http://postman.gorlice.pl/nowa/index.html


Z góry dziękuję za opinie i zaznaczam, że nie jest to żadna reklama.
Pozdrawiam

Powiem krótko: słabo. Po prostu.

Tzn?

Tzn. słabo.

Po co skan gazety w tle...?

tak mi się jakoś uwidziało

-strona długo mi się ładowała
-wypluwa mi się javascript w Archiwum
-wstawiasz w kodzie <br> po 20 razy
-strona nie przyciąga uwagi
-tło jakieś dziwne

ogólnie kiepsko

I właśnie w tym jest problem. Strona jest dla użytkowników (i właśnie o nich musisz na każdym kroku myśleć!), a nie dla Ciebie.
Niepotrzebnie wprowadziłeś takie tło graficzne. Na początku wydawało mi się, że jest to część strony, na równi z menu i treścią.
Logotyp jest za duży. Powinien być bardziej przesunięty na lewo i powinien przekierowywać na stronę główną. Każda podstrona nie może mieć innej kolorystyki. Użytkownik się gubi. Ustaw jeden kolor na wszystkich podstronach.
W stopce dodaj link (na twoim nick'u) z adresem e-mail webmastera (w tym wypadku Ciebie).
Wyróżnij pozycję w menu, w której się aktualnie znajduje użytkownik.

A teraz kod html:
- niepoprawna tabulacja,

[HTML] pobierz, plaintext 
<br><br><br><br><br><br>
[HTML] pobierz, plaintext 

- jest nagłówek h3, a nie ma h1 i h2,

[HTML] pobierz, plaintext 
iso-8859-1
[HTML] pobierz, plaintext 

Dla europy środkowej używa się iso-8859-2, ale i tak radzę przejść na UTF-8.

Co do html'a to nie ma za bardzo co oceniać, bo i strona jest prosta.

Podsumowując: strona nie jest piękna, a nawet ładna. Obejrzyj okiem trzeciej osoby strony o podobnej tematyce. Musisz zrozumieć, że aby zrobić dobrą stronę trzeba się uczyć latami. Nie jest źle, ale musisz cały czas pracować!

Pozdrawiam i życzę wielu sukcesów.

z tymi <br><br><br><br><br> to miałem problem który na poczekaniu rozwiązałem w ten sposób(brak pomysłu na ustawienie min. wysokości tak aby cała gazetka była widoczna). gazetka w tle to też pewnego typu celowy zabieg. wchodząc na stronę od razu widzisz z czym masz do czynienia(miała być bardziej zasłonięta ale za mało tekstów:P). zmieniające się tło stron myślałem, ze będzie fajnie(sugerowałem się joomla.pl) gdzie wchodząc w jakieś tam kolejne działy zmienia się wygląd strony.
Może jakies konstruktywne pomysły lecz nie typu wywal i zrób od początku.

pzdr

W tym rzecz, że nie do końca. Zanim załaduje się obrazek, mija u mnie co najmniej kilka sekund. Przez ten czas praktycznie nie wiadomo o co chodzi. Strona zajmuje około 1MB, czyli kilka razy za dużo :/

Moja porada to zrezygnować z tego tła, a dać jakieś ładniejsze logo. Bardziej wypełnij stronę treścią, ostatecznie nie musi być taka długa, jak aktualnie.

Witam

Poprawka do oceny.
http://postman.gorlice.pl/advert2/
Czekam na opinie.

Pozdrawiam i
Wesołych Świąt

Z zadowoleniem mogę powiedzieć - o to chodziło! Nie jest to jakieś cudo (na to trzeba lat pracy przy stronach), ale nie musisz się już wstydzić strony.

Moje zastrzeżenia:
- logo nie przenosi na stronę główną i przesunąłbym je bardziej na lwo,
- zwiększ line-height dla tekstu,
- brak stopki,
- nie przeczytałeś dokładnie moich rad (chodzi o kodowanie),
- niepoprawna tabulacja,
- brak nagłówków (h1, h2 itd.),
- zmień czcionkę na bezszeryfową,
- po co za każdym razem podajesz font-family?
- i kilka bezsensownych linijek kodu - m.in.:

[HTML] pobierz, plaintext 
<br>
    <br>
    <br>
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<strong><br>
 
</strong>
[HTML] pobierz, plaintext 

Dziękujemy i nawzajem.

Po najechaniu na link niech zmienia kolor tła na pomarańcz.
Daj jakieś marginy na logo, bo wygląda jakby je przycięło.
Przemyśl jeszcze menu po prawej i może jakiś inny kolor nagłówka, a nie wszedzie szary? ;P

Nie można tego w style wcisnąć?

[HTML] pobierz, plaintext 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
[HTML] pobierz, plaintext 

Co to za kodowanie?
http://www.w3.org/International/O-charset.pl.php
http://www.w3.org/International/questions/...encoding.pl.php

[HTML] pobierz, plaintext 
<ul>
      <li><a href="2005123.html">2005</a></li>
      <li><a href="20061.html">2006</a></li>
      <li><a href="">2007</a></li>
 
      <li><a href="">2008</a></li>
    </ul>
    <br>
    <br>
    <br>
[HTML] pobierz, plaintext 

Zwiększ padding-bottom dla tego diva, jak chcesz zostawić sobie więcej miejsca, a nie <br>.

Tak z moich odczuć to czerwone tło i biała czcionka daje po oczach. Wprowadź trochę ciepła do tej czerwonej barwy i trochę szarzyzny dla tej czcionki w menu.

Od strony estetycznej:
- nagłówek za wysoko... przesuń go o kilka pixeli w dół;
- oddziel jakoś logicznie menu od nagłówka (może trochę niżej i z jakimś lekkim tłem zbytnio nie wyróżniającym się ale jednak innym?);
- spójrz na nagłówek "Archiwum" a nagłówki treści menu -- nie są adekwatne i to trochę się kupy nie trzyma;
- te zawijasy symulujące kartki -- mi się nie widzą, przynajmniej nie w takiej postaci;
- div treści -- może warto zrobić stałą wysokość (lub minimalną);
- brak stopki.

Witam

Zastosowałem się do niektórych uwag, do niektórych zastosuje się potem.
http://postman.gorlice.pl/advert2/

Czekam na opinie

Pozdrawiam

No błędy są nadal w walidatorze w3.

A tak poza tym to czy muszą być dwa img pod sobą? Nie możesz normalnie napisać zamiast wstawiac obrazek z napisem?


A reszta ok.

Hmn, ale tego co wymaga najmniej pracy nie zmieniłeś, czyli kodowanie i padding. Najlepiej dokończ wszystko i dopiero pokaż.

Chyba grzecznie nie da się Tobie wytłumaczyć niektórych rzeczy. Po raz trzeci piszę: niepoprawna tabulacja, błędne kodowanie!
Czy na prawdę to zajmuje tak wiele czasu?

"Poprawiona wersja" jest chyba tylko odświeżona. Radzę zastosować się do rady @Cypherq-a.

[HTML] pobierz, plaintext 
<table width="446" border="0" align="center" cellpadding="0" cellspacing="4">
 
       <tr>
         <td width="438"><div align="center" class="czcionkaglowna">W budowie </div></td>
       </tr>
       <tr>
         <td>&nbsp;</td>
       </tr>
       <tr>
         <td>&nbsp;</td>
 
       </tr>
     </table>
[HTML] pobierz, plaintext 

Czy to na pewno prawidłowe przeznaczenie tabeli?

[HTML] pobierz, plaintext 
<div class="push"></div>
[HTML] pobierz, plaintext 

Staraj się nie pozostawiać pustych bloków. Jest tyle znaczników w HTML-u.

Po co Ci h3, skoro nie masz h2, ani h1?

[HTML] pobierz, plaintext 
<meta name="generator" content="HAPedit 3.1">
[HTML] pobierz, plaintext 

Wprowadzenie nazwy edytora w tym miejscu jest niepoprawne. Jeżeli nie korzystasz z żadnego CMS-a to usuń ten meta-tag.

Brak opisu strony i słów kluczowych.


Gdy następnym razem będziesz publikował poprawioną wersję postaraj się aby zawierała ona chociaż większość podanych wskazówek.

Zastosowanie tego pustego bloku jest celowe jest to "szablon " który przesuwa stopkę na dół. Chyba, że znacie inny sposób na przesunięcie stopki na spód w inny sposób niż taki.
Usunięte br-ki i inne zbędne części kodu, dodane paddingi, walidator też już nie krzyczy.
Jak macie jakieś pomysły co można yb tu jeszcze zmienić to czekam...

Pozdrawiam

Oo

No to padding-top tej stopki zwiększ...

Witam

Stronka wrzucona już na odpowiedni serwer.
http://advertbox.pl
wprowadziłem jeszcze niewielki zmiany, czekam na opinie.

Pozdrawiam
CeduNio

Mi się zupełnie niepodoba wizualnie, brak paddingów oddzielających treść od krawędzi, porozsuwane to jakieś wszystko... Szarobure...

A co powiecie na to?
adres ten sam

Pozdrawiam

No teraz jest kolosalna zmiana. Na plus oczywiście.
Co mi nie za bardzo się podoba to góra strony taka "pusta".
Pozatym nie wiem po co Ci to jest .
No i te archiwum strasznie szerokie.

Ale ogólnie spora poprawa.
Pozdrawiam

W 'specyfikacji' strona się rozlatuje.

specyfikacje jeszcze poprawiałem już działa,
archiwum no musi chyba być takie szerokie bardziej się go zwęszyć nie da:/