Witam,

To moja pierwsza strona strona, którą zrobiłem od zera w PHP.
PHP się dopiero uczę, działam na Symfony2, póki co działa tylko skrypt od newsów i galerii

Proszę o jej ocenę Krytyczna ocenę!

http://dieantwoord.cba.pl/

http://dieantwoord.cba.pl/post/87656


Grafika zbyt prosta, wielkie boxy menu z zaokrągleniami wieją amatorszczyzną. Zbyt dużo pustej przestrzeni na dole. Zbyt dużo szarości. Strona odstrasza.

Dodatkowo po za jednym linkiem z newsów, nic innego nie działa...

Jak na pierwszą stronę nie jest źle. Radzę tylko skorzystać z jakiegoś dobrego serwera, na której nie będzie nachalnych reklam.

Jak wszedłem to rozbolała mnie głowa.

Stopka conajmniej odraza. Nie zebym cos mial do penisow, ale to nie powazne moim zdaniem. Buduje zly wizerunek i zniecheca ludzi.

To jest evil boy, maskotka tego zespołu

Wielkie dzięki za wszystkie opinie.

Wiedziałem, że z tym chłopkiem z penisem będzie problem, ale to jeden z najbardziej popularnych symboli tego zespołu

Co do użycia tych kontrastowych kolorów, to biały i czarny również są czymś charakterystycznym dla nich.

@Spawnm - dzięki za opinie, poprawię te rzeczy które wypisałeś

Zrób czarne tło

I biały tekst na czarnym tle? Lepiej się czyta czarne litery na białym tle, ale zrobię również taki projekt i wtedy porównam

Poszukaj w necie jak to robią inni.
Przykład: http://www.wpulti.net/best-black-wordpress-themes-templates/

Zmienilem design trochę, może teraz będzie lepiej

1) Masz link od wyświetlania newsów : "http://dieantwoord.cba.pl/post/6".
Jak wpiszemy jakąkolwiek liczbę(wykraczającą poza zakres id newsów) albo ciąg znaków w miejsce '6', wyskakuje nam błąd:

Fatal error: Call to a member function getTitle() on a non-object in /virtual/dieantwoord.cba.pl/src/Acme/IndexBundle/Resources/views/Index/post.html.php on line 3

Tak być nie powinno.
Podejrzewam, że masz oddzielną akcję w kontrolerze od wyświetlania newsów.

[PHP] pobierz, plaintext 
 
/**
     * @Route("/single_news/{news_id}/", requirements={"news_id" = "\d+"})
     * @Template()
     */
    public function singleNewsAction($news_id)
{
 
}
[PHP] pobierz, plaintext 

Powinno to wyglądać mniej więcej tak jak w kodzie powyżej. Zamiast takich adnotacji w komentarzu, można użyć konfiguracji w pliku yaml. Tu np. masz więcej o adnotacjach. http://symfony.com/doc/current/bundles/Sen...ns/routing.html

Jeśli chodzi o proste tłumaczenie to w requirements za pomocą wyrażeń regularnych określasz, że spodziewasz się tylko liczb. W samej akcji powinieneś także sprawdzić czy news o podanym id faktycznie istnieje czy nie. Jeśli nie to powinieneś wyrzucić jakiś błąd o tym.

2) Jeśli chodzi o logowanie i zarządzanie użytkownikiem, polecam: https://github.com/FriendsOfSymfony/FOSUserBundle
Na początku chciałem sam wszystko ogarnąć, ale po męczarniach z ustawieniem symfony, darowałem sobie. Prościej modyfikuje się tego bundla.


A tak ogólnie jak na początek mi się całkiem podoba, tylko dół strony się posypał.

Jak dla mnie graficznie zdecydowanie lepiej

@BigPig - no właśnie dopiero od niedawna używam symfony, więc na razie uczę się tych głównych rzeczy, a później skupię się na wywalanych błędach.

Dzięki, na pewno też skorzystam z tego bundla, bo systemu logowania etc. boję się najbardziej

Jak to się posypało, ja na lapku otwieram i wszystko jest na miejscu?

Cały footer ci się posypał.
Białe tło nie pasuje do strony o tym zespole, serio

Kaj się posypał? W mojej przeglądarce jest wszystko git

No przecież już nie ma białego tła

@Sprawnm, wyłącz adblocka.

Jak pierwszy raz wszedłem na stronie z adblockiem, to miałem białe tło i posypany footer. Po wyłączeniu wszystko się ułożyło. Spróbowałem jeszcze raz włączyć adblocka i tym razem także z adblockiem wszystko dobrze chodziło.

Tło całkiem całkiem

Po f5 widzę to nowe. Ładne
Tylko pod ff/ubuntu inputy wylatują po za box logowania.
Błąd z Fatal error: Call to a member function getTitle() nadal istnieje.

W formularzu logowania zamiast placeholder użyłeś value przez co jak się kliknie to zawartość nie znika i trzeba samemu usuwać. Zmień to.

A generalnie disajn na plus. Może footer trochę za mało się odznacza, więc na pierwszy rzut oka nie widać gdzie się zaczyna.

Nie rejestrowałem się,
http://dieantwoord.cba.pl/?user=ZaXaZ&...aloguj+si%C4%99
Czemu dajesz haslo w adresie? O.o, ponoć takie adresy da się przejąć.

No tak, i w sumie jedyna bo jest tylko główna i to treścią przykładową wypełniona.

- Formularz bez akcji i metody (temu idzie GET zamiast POST)
- brak placecholder
- brak strzałek w galerii po otworzeniu obrazka

Na stronie nic nie działa, ponieważ chciałem jedynie design do oceny

Z PHP działa jedynie system newsów i ta galeria

Sorry, nie mogłem się powtrzymać.


Co do strony. Na pierwszy rzut oka widać lekki semantyczny bełkot. Wydaje mi się, że pierwsze kontenery oznaczone jako header mogłyby być elementem header. Poza tym boczna kolumna powinna być raczej elemntem aside.
Widać pewną nie konsekwencję w konwencji nazewniczej, formatowaniu kodu. W dalszej perspektywie warto byłoby wypracować sobie czytelny i transparentny kod. Nie zagłębiałem się dokładnie kod ale wydaje mi się, że masz divitis i classitis; przeraża ilość id.

Górna nawigacja wydaje się "lekko" zbyt oczojebna. Może warto byłoby poprawić typografię? Większość elementów nie posiada zaokrągleń, więc może warto ten formularz i przyciski zmienić? Dodatkowo skoro cała strona ma taką mroczną i zuuaa tonację to taki formularz powoduje taki emocjonalny dysonans. Okrągłe kształy podobno wywołują raczej łagodne i spokojne wrażenie. Natomiast ostre i nie regularne, zajebiście coś innego.

Popracowałbym też nad hierarchią wizualną. Po otwarciu strony, zupełnie nie wiedziałem gdzie mam się patrzeć. Jeśli nie reprezentujesz kultu turpizmu, to może warto byłoby dodać inne kolory niż tylko prawilną biel i czerń?
Informacje pod tytułem bym lekko "przygasił", a czytaj więcej wzmocnił.

Witam,

Wrzucam po raz drugi do oceny moją stronę o Die Antwoord. Wiem, że kod HTML jest tragiczny, przepiszę go później.
Proszę o waszą ocenę przede wszystkim designu strony oraz ewentualnie uwagi odnośnie samego funkcjonowania strony.

Nie jestem programistą, zajmuję się tym zajawkowo w wolnych chwilach, stąd rozumiem, że pewnie niektóre rozwiązania jakie zastosowałem są błędne.

Treść postów jest przykładowa, nie zwracajcie na to uwagi

DIEANTWOORD.COM.PL

Proszę tylko o szczere opinie !

Mam wrażenie, że na tematy w tym dziale nie ma co patrzeć. Kolejna brzydka strona, a autor na pewno będzie chciał usłyszeć "Pięknie zrobione milordzie.". Nie jest ładnie, jest brzydko. Tematyka też nieciekawa (nie darzę tego zespołu sympatią).

Apel do ludzi:

Ludzie, jeżeli chcecie się czymś pochwalić, to chwalcie się czymś ładnym, bo aż niedobrze mi się robi, jak co lepszy miszcz wrzuca kupa-dzieło.

No chyba że są turpistami

Co do samej strony:
Czcionka w menu jest dla mnie nieczytelna, rushofy top mnie odstrasza, lupka przy wyszukiwarce chyba jest trochę za wysoko i stopka odstrasza mnie tak samo jak top.

Po co tyle chamstwa milordzie?

--

A jak myślicie, gdyby napisy były białe a reszta czarna, to byłoby to bardziej zjadliwe?

Zależy jakbyś sobie przyszalał z tym czarnym i białym
Ogólnie to to co jest straszy według mnie przede wszystkim tym, że to połączenie różowego i niebieskiego jest kiepskie, dodatkowo 2 użyte czcionki do siebie nie pasują i jeszcze zauważyłem, że link 'Zobacz wszystkie posty >>' na głównej, jest prawie niewidoczny. Ogólnie to wiesz... są gusta i guściki nam się nie musi podobać, ale prosiłeś o ocenę.

Dla mnie sam motyw tła na plus, niestety kolory trochę odrzucają i mam spore wątpliwości co do tego czy oddają klimat zefside, turkus z różowym kojarzy mi się z wesołymi emo zespołami dla dziewcząt. No i szkoda ze zrezygnowałeś z evilboya w stopce, Die Antwoord jest odważne i taka też powinna być strona o nich.

Zmieniłem kolorystykę. Lepiej czy gorzej?

Eee, ostatnio rok temu widziałem tą stronę i nie widze zeby cos sie zmienilo.
Chyba rejestracja/logowanie bylo i juz nie ma.

1. http://dieantwoord.com.pl/img/post/uglyboy.png
Linkujesz do tego w <img> a tam 404....
2. W wyszukiwarce, a już kompletnie to w ankiecie nie czytelny kolor czcionki.
3. Tlo bys mogl w jakims mini stopniu poszarzyć.
Ogolem fb sie wbija w oczy przy tak ciemnej i nie pasuje a pewnie da sie zmienic.
(Swoja droga ciekawe ze ta strona jedyne polubienie na fb ma z Hiszpanii bodajże).