![]() |
![]() ![]() |
![]() |
![]()
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/ |
|
|
![]()
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... |
|
|
![]()
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.
|
|
|
![]()
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.
|
|
|
![]()
Post
#5
|
|
Grupa: Zarejestrowani Postów: 675 Pomógł: 58 Dołączył: 17.12.2013 Ostrzeżenie: (10%) ![]() ![]() |
Stopka conajmniej odraza. Nie zebym cos mial do penisow, ale to nie powazne moim zdaniem. Buduje zly wizerunek i zniecheca ludzi.
|
|
|
![]()
Post
#6
|
|
Grupa: Moderatorzy Postów: 4 069 Pomógł: 497 Dołączył: 11.05.2007 Skąd: Warszawa ![]() |
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) |
|
|
![]()
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) |
|
|
![]()
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) |
|
|
![]()
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)
|
|
|
![]()
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/ |
|
|
![]()
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)
|
|
|
![]()
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.
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 |
|
|
![]()
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
|
|
|
![]()
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? |
|
|
![]()
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) |
|
|
![]()
Post
#16
|
|
Grupa: Zarejestrowani Postów: 31 Pomógł: 0 Dołączył: 18.02.2014 Ostrzeżenie: (0%) ![]() ![]() |
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) |
|
|
![]()
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 |
|
|
![]()
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. |
|
|
![]()
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 |
|
|
![]()
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ąć. |
|
|
![]() ![]() |
![]() |
Aktualny czas: 14.09.2025 - 22:44 |