Witam

Proszę o ocenę serwisu, wersja testowa jest pod adresem:

Kulturystyka

Pozdrawiam

Pierwszy, podstawowy błąd. Strona zaczyna się na środku wysokości. Strona zaczyna się dopiero od połowy ekranu. Pierwsze 50% ro reklamy. Bez przesady. Najpierw nie mogłem znaleźć loga, a to rzecz podstawowa.

Jak dla mnie to rozmiar czcionki w tekstach jest za mały.
Za duży odstęp treści od nagłówka.

Gdzie są zastrzeżone prawa autorskie?

Wszędzie te reklamy, nie przesadzaj z tym.

Jak dla mnie to logotyp jest za mały. Według mnie powinien być ok. dwa razy większy niż jest teraz.


Przejdźmy do kodu:
Wiesz co to jest przejście do nowej linii? Rób odstępy od niektórych elementów (np. oddziel kolejne skrypty javascript).
Dlaczego w tytule masz słowa kluczowe? Nie rozumiem tego.
h1 powinien zawierać tytuł strony na każdej podstronie. Np. <h1><span style="display: none;">tytuł</span></h1>.

Ogólnie kod jest bardzo ładny, ale jak dla mnie trochę nieczytelny.


echo 3maj_się();

Reklamy z góry strony już usunąłem, ale zrobiło się jakoś pusto chyba przydałby się jakiś top.

Własnie miałem to powiedzieć, brakuje loga, a to co jest nad menu może być jedynie navbarem Zawsze na logu możesz dać reklamę

Strona jest jakaś taka mdła. Nie ma na niej ani jednego elementu, który skupiałby uwagę i powodował, że od niego zaczynamu oglądanie/czytanie. Dużo naćkanego tekstu. Brak wyraźnych ram. To taki mój pierwszy odbiór.

Druga sprawa, niegdy nie pisz kudu w taki sposób:
<h1><span style="display: none;">tytuł</span></h1>

Jeśli już chcesz ukryć nagłówek to zrób to po ludzku, chociażby tak:
<h1 style="display: none;">tytuł</h1>

Kod jest dość dziwny. Czytający go robot indeksujący najpierw znajdzie h3 a potem h1 na końcu h2. Trochę to dziwne, nie?

@Joshima

Druga sprawa, niegdy nie pisz kudu w taki sposób:
<h1><span style="display: none;">tytuł</span></h1>

Jeśli już chcesz ukryć nagłówek to zrób to po ludzku, chociażby tak:
<h1 style="display: none;">tytuł</h1>

Bzdura, pisz tak, ale tylko wtedy, gdy chcesz dać background graficzny dla h1, a co za tym idzie chcesz się pozbyć zawartości tego znacznika, co by nie przesłaniał grafiki.
Display:none; dla h1 spowoduje w ogóle nie wyświetlenie elementu, a nie o to nam chodzi
Alternatywa to img i alt.

W dziale koksy nie masz najbardziej dotkliwego dla mężczyzny skutku ubocznego. Chyba nie muszę mówić jakiego

To ciekawe o co i jaki sens ma wyświetlanie pustego elementu <h1> jeśli miałoby to tylko na celu wyświetlenie grafiki to dla mnie jest to zupełnie niesemantyczny a więc zbędny wygibas.

Nie chcę się wtrącać, ale podam tylko jeden adres: http://wiadomosci.wp.pl I już chyba powinno być wszystko jasne.

I niby jak to się ma?

A właśnie, że jest bardzo semantyczny, dlatego, że roboty nie analizują CSS. Dla robotów jest to dalej element h1 o zawartości takiej jak w span. Wyłączasz CSS i masz h1. Włączasz i masz np. jakiś tam web2.0 odbity i co tam jeszcze h1. Poza tym znasz jakiś lepszy sposób na sprawienie, aby np. element h1 był wyświetlony jako grafika? Bo ja nie, jedynie moim zdaniem mniej semantyczny img z altem wewnątrz np. h1. Dlaczego mniej semantyczny? Bo wkładamy grafę w HTML, a w przypadku poprzednim zwalamy ją na CSS zachowując semantykę HTML.

Dla robotów to jest właśnie <h1 style="display: none;">tytuł</h1> a wyświetlanie h1 jako grafiki jest niesemantyczne.



Tak się składa, że specyfikacja jest przeciwnego zdania i bardziej semantyczny jest img, który służy do wyświetlania grafiki niż pakowanie grafiki do h1 z ukrytym tekstem. Wrzucanie grafiki do css ma sens i jest zgodne ze standardami wyłącznie wtedy gdy chodzi o tło na którym ma być jakiś tekst lub inny element. W innych sytuacjach to zwykła rzeźba jest.

Cóż wychodzi na to, że faktycznie masz rację co do display:none;
Zwłaszcza Twoje ostatnie zdanie jest warte przemyślenia. Skoro tak jest faktycznie, że display:none; powoduję tę rzeźbę, bo tekstu nie ma, to co by było gdyby dać dla spana text-indent:- jakaś duża liczba, albo position:absolute; - jakaś duża liczba. Tekst by chyba był nadal "widoczny", czy się mylę?

Nie bardzo rozumiem jaką dokładnie konfigurację masz na myśli. Zapisz to po prostu.

można też zrobić tak:

<img src="logo" alt="(...)">
<h1>(...)</h1>

h1{display:none}

Ale wtedy tytuł pojawia się dwa razy (w alcie i w h1). Rozwiązaniem jest zostawienie pustego argumentu alt

Choć prawdę mówiąc wolę takie rozwiązanie:

<h1><span>tytuł</span></h1>

h1{background-color:url(..)}
h1 span{display:none}

<h1><span>gdfldfnlhndfg</span></h1>

a)

h1 {
position:relative;
}

h1 span{
position:absolute;
left:-12000px;
}



h1 span{
text-indent:-12000px;
}

Poza tym coś mi jednak nie daje spokoju w tym całym temacie. Zaczynam chyba wierzyć w to, że CSS ma wpływ na semantykę dokumentu. Tego przyznam się nie wiedziałem. Czy nie jest to trochę zaprzeczenie idei CSS i XHTML, oddzielenie treści od wyglądu?

Uwzględniłem Wasze uwagi i dokonałem wielu zmian, aktualna wersja

Kulturystyka

Wasze uwagi bardzo mi pomagają, dlatego proszę jeszcze o ocenę strony

Moim zdaniem to nie stoi w sprzeczności. Jeśli używasz css po to, żeby wykonywać HTMLm jakieś niesemantyczne fikołki to można powiedzieć, że css ma wpływ na semantykę kodu a jednocześnie jest to rozdzielenie treści od wyglądu. Treść jest niesemantycznie skonstruowana a CSS to jakoś tam "naprawia" i powoduje, że wizualnie jest OK.

Cóż naprawdę nigdzie nie mogę znaleźć informacji na temat wpływu CSS na semantykę dokumentu. Więc śmiem twierdzić, że IMHO się mylisz, ale jako człek open minded chętnie zmienię zdanie i nauczę się czegoś nowego, więc proszę Cię o poparcie swoich teorii jakimiś sensownymi linkami.
Poza tym to, że jakoś to wygląda nijak się ma do semantyki dokumentu i doskonale sobie z tego zdajesz sprawę. Nie chcę kwestionować Twojej wiedzy bo podejrzewam, że nie jest mała, więc jedynie mogę pokusić się o stwierdzenie, że mamy różne pojęcie semantyki. Poza tym ja też jestem fanatykiem standardów, jednak nawet mnie span w h1 nie razi aż tak strasznie, choć przyznaję, wolałbym, żeby go nie było. I nie wiem co Ciebie bardziej razi, czy span, czy display:none;? Bo jeśli coś jest niesemantyczne to IMHO span. Ale nie popadajmy w paranoję.
Podsumowując:

Nie zrozumiałeś. Chodzi o to, że możesz sobie tworzyć niesemantyczny kod HTML i żonglować CSSem tak, że będzie sie prezentował dobrze. Trochę tak jak z tworzeniem stron na tabelkach. Kompletnie niesemantyczne, ale wyglądały OK.


Więc uściślijmy. Za niesemantyczne uważam wykorzystanie znacznika HTML niezgodnie z jego przeznaczeniem opisanym w dokumentacji (W3C). Tak więc uzycie <hx/> do wyświetlania grafiki jest niesemantyczne bo do tego według specyfikacji języka HTML służy znacznik <img/>. Oczywiście da się to zrobić i jeszcze połączyć z CSS tak, żeby przy okazji ukryć tytuł, czy słowa kluczowe w <hx/> ale jest to niesemantyczny kod i już. W tym właśnie sensie napisałam, że CSS wspomaga kodera w tworzeniu niesemantycznych rzeźb.




No i tu się z Tobą zgadzam, bo <h1><span style="display:none">gdfldfnlhndfg</span></h1> Jest niesemantyczne bo:
1. Występuje nadużycie znaczników, gdyż do zamaskowania tytułu stworzonego ewidentnie dla robotów wystarczy <h1 style="display:none">gdfldfnlhndfg</h1>
2. Zapis <h1><span style="display:none">gdfldfnlhndfg</span></h1> został stworzony celowo, żeby wykorzystać <h1/> niezgodnie z przeznaczeniem czyli do wyświetlenia grafiki w połączeniu z CSS h1{background:url(..)}

Oczywiście ja też mam takie samo pojęcie semantyki. Twierdzisz: użycie znacznika <hx> do wyświetlenia grafiki. Tylko zauważ, że gdy jest wyłączony CSS nie jest wyświetlana żadna grafika! To nie jest użycie znacznika hx do wyświetlenia grafiki, to jest użycie CSS do wystylizowania semantycznego nagłówka h1, tak , aby wyglądał jak grafika. Gdybyś miała rację, jedyną opcją umieszczenia grafiki na stronie byłby znacznik img. I tak jest, jeśli chodzi o HTML (przepchnęli notabene znacznik img, bo ludzie go używali, miał być object). Natomiast CSS to inna bajka. CSS jest jak ubranie. Nie ważne w co się ubierzesz, to nadal jesteś Ty. Tak samo semantyczna...

Pozdrawiam

Jest grafika i grafika. Grafika jako tło faktycznie jest i powinna być wyświetlana za pomoca CSS. Grafika jako objekt, element strony powinna być wyświetlana przez <img/> i to jest semantyczne i jest to jedyna opcja wyświetlania grafiki, która nie jest tłem (logo, fotka itp.) . Wyświetlanie jej przez <hx/> nie jest semantyczne i już jest nadużyciem, które wynika z niezrozumienia idei oddzielenia warstwy treści od warstwy prezentacji. Tło jest prezentacją, logo jest treścią mimo, że jest graficzne.


Ale jak jestem niesemantyczna ale ubiorę sie odpowiednio to tego nie widać

Z tym się nie zgadzam. Też nie podoba mi się, jak ktoś korzysta tagów niezgodnie z przeznaczeniem, ale to jest IMO ok. Bo to w końcu nadal jest np. tytuł strony, a więc główny nagłówek. Co z tego, że w formie graficznej? To nadal jest główny nagłówek. CSS służy ustaleniu sposobu prezentacji, więc właśnie ja chcę, aby <h1> był w formie graficznej. Czemu mam dodawać dodatkowy <img>owy obrazek? Przecież to cały czas jest ten sam główny nagłówek, ale w formie graficznej, tylko o zmienionym sposobie prezentacji za pomocą CSS. Jak na moje to nie jest niesemantyczne.

Jak wikipedia mówi (tak, wiem, wikipedia kłamie, ale to akurat jest fakt):


Więc ja będę sobie za pomocą CSS opisywał formę prezentacji (wyświetlenia) <h1>. W postaci grafiki.


PS. Dobra, skończmy tą debatę o tym co jest bardziej semantyczne, bo to forum Oceny jest...

Tak, ale czy tak czy siak koszulka z napisem display:none; na przedzie niczego nie zmienia

[HTML] pobierz, plaintext 
h1{background:#xxx url(img) no-repeat;text-indent:-10000px}
[HTML] pobierz, plaintext 

widok normalnej strony z css - nagłówek graficzny
wyłączamy css - zwykły nagłówek tekstowy

bardziej sematycznie się nie da

sam kod - deklarowane xhtml wysyłane jako text/html, używanie marquee, z jednej strony ramki bezpośrednio w kodzie z innej osadzane przez JS - czyli lekka sieczka

ale widziałem dużo gorsze, więc nie ma się czym zbytnio przejmować

bardzo mało czytelna strona, do tego tekst z newsów zachodzi na menu po prawej...