Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

2 Stron V   1 2 >  
Reply to this topicStart new topic
> Strona o zespole Die Antwoord
folt3k
post
Post #1





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


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

Proszę o jej ocenę (IMG:style_emoticons/default/smile.gif) Krytyczna ocenę!

http://dieantwoord.cba.pl/
Go to the top of the page
+Quote Post
Spawnm
post
Post #2





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




http://dieantwoord.cba.pl/post/87656
Cytat
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


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...
Go to the top of the page
+Quote Post
DG1
post
Post #3





Grupa: Zarejestrowani
Postów: 4
Pomógł: 0
Dołączył: 22.11.2011

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


Jak na pierwszą stronę nie jest źle. Radzę tylko skorzystać z jakiegoś dobrego serwera, na której nie będzie nachalnych reklam.
Go to the top of the page
+Quote Post
Pyton_000
post
Post #4





Grupa: Zarejestrowani
Postów: 8 068
Pomógł: 1414
Dołączył: 26.10.2005

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


Jak wszedłem to rozbolała mnie głowa.
Go to the top of the page
+Quote Post
tzm
post
Post #5





Grupa: Zarejestrowani
Postów: 675
Pomógł: 58
Dołączył: 17.12.2013

Ostrzeżenie: (10%)
X----


Stopka conajmniej odraza. Nie zebym cos mial do penisow, ale to nie powazne moim zdaniem. Buduje zly wizerunek i zniecheca ludzi.
Go to the top of the page
+Quote Post
Spawnm
post
Post #6





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




Cytat(tzm @ 24.08.2014, 19:57:49 ) *
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 (IMG:style_emoticons/default/wink.gif)
Go to the top of the page
+Quote Post
folt3k
post
Post #7





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


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 (IMG:style_emoticons/default/biggrin.gif)

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ś (IMG:style_emoticons/default/smile.gif)

Go to the top of the page
+Quote Post
Spawnm
post
Post #8





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




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

Zrób czarne tło (IMG:style_emoticons/default/wink.gif)
Go to the top of the page
+Quote Post
folt3k
post
Post #9





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


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 (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Spawnm
post
Post #10





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




Poszukaj w necie jak to robią inni.
Przykład: http://www.wpulti.net/best-black-wordpress-themes-templates/
Go to the top of the page
+Quote Post
folt3k
post
Post #11





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


Zmienilem design trochę, może teraz będzie lepiej (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
BigPig
post
Post #12





Grupa: Zarejestrowani
Postów: 61
Pomógł: 9
Dołączył: 18.06.2013
Skąd: Białystok

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


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.
  1.  
  2. /**
  3.   * @Route("/single_news/{news_id}/", requirements={"news_id" = "\d+"})
  4.   * @Template()
  5.   */
  6. public function singleNewsAction($news_id)
  7. {
  8.  
  9. }


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ł.

Ten post edytował BigPig 26.08.2014, 18:01:13
Go to the top of the page
+Quote Post
Pyton_000
post
Post #13





Grupa: Zarejestrowani
Postów: 8 068
Pomógł: 1414
Dołączył: 26.10.2005

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


Jak dla mnie graficznie zdecydowanie lepiej
Go to the top of the page
+Quote Post
folt3k
post
Post #14





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


@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 (IMG:style_emoticons/default/biggrin.gif)

Jak to się posypało, ja na lapku otwieram i wszystko jest na miejscu?
Go to the top of the page
+Quote Post
Spawnm
post
Post #15





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




Cały footer ci się posypał.
Białe tło nie pasuje do strony o tym zespole, serio (IMG:style_emoticons/default/wink.gif)
Go to the top of the page
+Quote Post
folt3k
post
Post #16





Grupa: Zarejestrowani
Postów: 31
Pomógł: 0
Dołączył: 18.02.2014

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


Cytat(Spawnm @ 26.08.2014, 21:07:10 ) *
Cały footer ci się posypał.
Białe tło nie pasuje do strony o tym zespole, serio (IMG:style_emoticons/default/wink.gif)

Kaj się posypał? W mojej przeglądarce jest wszystko git (IMG:style_emoticons/default/biggrin.gif)

No przecież już nie ma białego tła (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
BigPig
post
Post #17





Grupa: Zarejestrowani
Postów: 61
Pomógł: 9
Dołączył: 18.06.2013
Skąd: Białystok

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


@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 (IMG:style_emoticons/default/wink.gif)

Ten post edytował BigPig 26.08.2014, 21:24:43
Go to the top of the page
+Quote Post
Spawnm
post
Post #18





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




Po f5 widzę to nowe. Ładne (IMG:style_emoticons/default/smile.gif)
Tylko pod ff/ubuntu inputy wylatują po za box logowania.
Błąd z Fatal error: Call to a member function getTitle() nadal istnieje.
Go to the top of the page
+Quote Post
timon27
post
Post #19





Grupa: Zarejestrowani
Postów: 578
Pomógł: 69
Dołączył: 15.04.2007
Skąd: Wrocław

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


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.

Ten post edytował timon27 27.08.2014, 22:49:12
Go to the top of the page
+Quote Post
ZaXaZ
post
Post #20





Grupa: Zarejestrowani
Postów: 285
Pomógł: 18
Dołączył: 30.01.2014
Skąd: <?=$_GET['city']?>

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


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ąć.
Go to the top of the page
+Quote Post

2 Stron V   1 2 >
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: 14.09.2025 - 22:44