![]() |
![]() ![]() |
![]() |
![]()
Post
#1
|
|
![]() Grupa: Zarejestrowani Postów: 260 Pomógł: 26 Dołączył: 26.01.2012 Skąd: Ci to przyszło do głowy? Ostrzeżenie: (0%) ![]() ![]() |
Cześć, ostatnio robiłem projekt dla pewnej wsi, możecie ocenić i wytknąć mi błędy? ;-)
tutaj strona Pozdrawiam ;-) -------------------- |
|
|
![]()
Post
#2
|
|
![]() Grupa: Zarejestrowani Postów: 8 068 Pomógł: 1414 Dołączył: 26.10.2005 Ostrzeżenie: (0%) ![]() ![]() |
Zacznij od początku bo to najzwyczajniej jest brzydka
|
|
|
![]()
Post
#3
|
|
![]() Grupa: Zarejestrowani Postów: 650 Pomógł: 16 Dołączył: 5.07.2010 Skąd: Ściśle Tajne Ostrzeżenie: (0%) ![]() ![]() |
nie chcę być nie grzeczny, ale ta strona jest wieśniacka
![]() -------------------- Jak coś jest dobre, to nie znaczy że nie może być to lepsze - Ideały nie istnieją ;D
Strony internetowe Świnoujście |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 717 Pomógł: 120 Dołączył: 18.04.2009 Ostrzeżenie: (0%) ![]() ![]() |
Estetyka leży. Strona wygląda jak tworzona na kolanie. Popracuj nad:
- kolorystyką - ogólnym rozłożeniem elementów na ekranie - przemyśleniem interakcji użytkownika ze stroną (to menu jest efekciarskie ale całkowicie BEZSENSOWNE pod kątem używania) czyli generalnie nad wszystkim. konkretne błędy: - obrazki są po lewej stronie, a po prawej pustka zupełna (spróbuj `float: left` ustawić na obrazkach) - to szare tło jest kompletnie brzydkie - chciałeś zrobić animowane menu? Okej. Ale animacje są za wolne. Jak klikam menu to nie chcę oglądać animacji, tylko chcę zobaczyć menu. - po menu się okropnie nawiguje. Niepotrzebnie zrobiłeś to w kształcie akordeona, bo teraz trzeba się naklikać po menu solidnie, żeby znaleźć cokolwiek. Menu powinno służyć uzytkownikowi, a ma wrażenie, że posłużyło jedynie tobie (wygląda tak jakbyś się chciał nauczyć robić animacje w jQuery robiąc to menu). Czyli: menu jest strasznie nieczytelne, nie wiadomo jak się w nim poruszać (lepiej byłoby, gdyby podkategorie były od razu widoczne). - poza tym hamburger na stronie desktopowej? Ok, czasem to się sprawdza, ale tutaj nie ma sensu zbytnio. Jeśli jest miejsce na ekranie, to można równie dobrze wyświetlić od razu całe menu. Hamburger niech sobei będzie jak się wejdzie z mobilki... (w zasadzie nie użyłeś ikonki hamburgera tylko koła zębatego, ale na jedno wychodzi) - pojawiająca się znikąd niebieska płachta z loaderem przy każdym kliknięciu pozycji menu? Jak dla mnie do wywalenia. Niczemu to nie służy a jedynie może spowodować epilepsję. Ogólne wrażenie - przygnębiające. Chociaż nie chcę zniechęcać, dużo osób zaczyna od rzeczy słabych, a potem poprawiają swoje skille i to co robią. Rozwój to praca nad sobą i umiejętność poprawy/modyfikacji tego co nie wyszło. Ten post edytował PrinceOfPersia 27.09.2015, 20:16:42 -------------------- |
|
|
![]()
Post
#5
|
|
Grupa: Zarejestrowani Postów: 3 Pomógł: 0 Dołączył: 3.06.2015 Ostrzeżenie: (0%) ![]() ![]() |
Musisz dużo więcej pracować.
Zanim zaczniesz robić stronę, obejrzyj sobie kilka tematycznie powiązanych. Korzystaj z kuler.adobe.com - pomoże Ci kolorystycznie coś wybrać. Nie kombinuj z takim menu. Grunt to funkcjonalność strony, a nie napakowanie jej efektami. Dosłownie, miedzy menu, a resztą strony jest przepaść. Nie ma kompletnie żadnego powiązania. Skąd ta szarość? ![]() No i te niebieskie coś przy przełączaniu zakładek, doprowadza do frustracji... Nie zniechęcaj się, najważniejsza jest praca. |
|
|
![]()
Post
#6
|
|
![]() Grupa: Zarejestrowani Postów: 260 Pomógł: 26 Dołączył: 26.01.2012 Skąd: Ci to przyszło do głowy? Ostrzeżenie: (0%) ![]() ![]() |
Oki, a więc usunąłem "niebieskie coś", wyłączyłem ukrywanie menu, przyspieszyłem wszystkie animacje, dla contentu strony przydzieliłem więcej miejsca, no i usunąłem tę szarość. Teraz prezentuje się chociaż troszkę lepiej?
![]() No niestety specem od designu nie jestem, bo wolę kodowanie zaplecza, stąd moja prośba o Waszą ocenę, na przyszłość to zawsze jakaś rada ![]() Ten post edytował Beniooo 28.09.2015, 19:55:05 -------------------- |
|
|
![]()
Post
#7
|
|
![]() Grupa: Zarejestrowani Postów: 3 034 Pomógł: 366 Dołączył: 24.05.2012 Ostrzeżenie: (0%) ![]() ![]() |
kup jakiś szablon z themeforest bo nadal źle, strona powinna zachęcać do skorzystania a nie odstraszać...
a na przyszłość, skoro w back lepiej sie czujesz to Ty rób back-end a ktoś inny niech robi front-end ![]() |
|
|
![]()
Post
#8
|
|
![]() Grupa: Zarejestrowani Postów: 94 Pomógł: 9 Dołączył: 11.10.2010 Skąd: Kraków Ostrzeżenie: (0%) ![]() ![]() |
Dla mnie to (wybacz) ale nadal wygląda fatalnie.
-Pierwsze wrażenie po wejściu to takie, że strona jest w budowie. -Brakuje jakiegoś stałego nagłówka, stopka która nie jest stopką bo jest u samej góry psuje kompozycję. -Kolorystyka leży. Nie wiedzieć czemu napis 'Menu' jest wyróżniony oczojebnym niebieskim. Spokojnie - domyślam się, że to jest menu. Nie musisz tego podpisywać. -Forsujesz tego Bootstrapa, a strona responsywna nie jest prawie w ogóle. Myślę, że powinieneś pooglądać jakieś strony gmin i miast i starać się zrobić/poszukać czegoś w odpowiedniej konwencji. Powodzenia ![]() -------------------- Podchodź pozytywnie do swojego kodu :)
|
|
|
![]()
Post
#9
|
|
![]() Grupa: Zarejestrowani Postów: 2 707 Pomógł: 290 Dołączył: 16.12.2008 Skąd: Śląsk Ostrzeżenie: (0%) ![]() ![]() |
Mnie się nie włącza.
Ciągle dostaje komunikat: Cytat DO WYWIETLENIA STRONY WYMAGANA JEST WŁĽCZONA OBSŁUGA JAVASCRIPT!!! (pomijam literówki w tym zdaniu i te niepotrzebne wykrzykniki).edit: Już wiem, po prostu bez JS jest przekierowanie na podstronę noJavaScript.html - szkoda, że jak go już włączę i odświeżę stronę to nadal ta strona się mi wyświetla zamiast w najgorszym wypadku chociaż przekierować na główną. Ten post edytował markonix 28.09.2015, 21:50:05 -------------------- |
|
|
![]()
Post
#10
|
|
Grupa: Zarejestrowani Postów: 717 Pomógł: 120 Dołączył: 18.04.2009 Ostrzeżenie: (0%) ![]() ![]() |
Cytat Teraz prezentuje się chociaż troszkę lepiej? lepiej to pojęcie relatywne. W porównaniu do tamtego lepiej (bo nie trzeba klikać buttona, żeby pokazać menu + nie ma przeszkadzającej niebieskiej płachty). Ale dalej jest to strasznie słabe. Należałoby raczej całkowity redesign zrobć. No i zamiana szarości na półprzezroczyste tło może wygląda ładniej, ale to spowodowało, że tekst jest nieczytelny. Co do JavaScriptu, to ta strona równie dobrze mogłaby działać bez niego. Do czego uzywasz JS? Chyba tylko do animacji submenu (Wywalić animacje, nie bedzie potrzebny JS). Co prawda w przeciwieństwie do markonixa, nie widzę nic złego w tym, żeby strona się nie uruchamiała bez włączonego JavaScriptu, pod warunkiem jednak, że jest ku temu jakis większy powód. Tutaj tego powodu nie ma. Jest to strona statyczna, gdzie JS jeśli jest do czegos uzywane to tylko chyba do odpalania animacji submenu -------------------- |
|
|
![]()
Post
#11
|
|
![]() Grupa: Zarejestrowani Postów: 3 034 Pomógł: 366 Dołączył: 24.05.2012 Ostrzeżenie: (0%) ![]() ![]() |
Cytat poza tym hamburger na stronie desktopowej? Ok, czasem to się sprawdza, ale tutaj nie ma sensu zbytnio. Jeśli jest miejsce na ekranie, to można równie dobrze wyświetlić od razu całe menu. Hamburger niech sobei będzie jak się wejdzie z mobilki... (w zasadzie nie użyłeś ikonki hamburgera tylko koła zębatego, ale na jedno wychodzi) hamburger & ux tak bardzo, jak można to nie używaj go wcale ![]() Ten post edytował com 29.09.2015, 15:11:52 |
|
|
![]()
Post
#12
|
|
![]() Grupa: Zarejestrowani Postów: 2 707 Pomógł: 290 Dołączył: 16.12.2008 Skąd: Śląsk Ostrzeżenie: (0%) ![]() ![]() |
w przeciwieństwie do markonixa, nie widzę nic złego w tym, żeby strona się nie uruchamiała bez włączonego JavaScriptu A czy ja napisałem, że to mi przeszkadza? Mam po prostu włączonego no-script'a i jak wyskoczy mi, że muszę włączyć JS to go włączę (świadomie). Chodzi o fakt, że go włączyłem a "komunikat" WŁĄCZ JS ciągle mi nie pozwala otworzyć strony (bo się okazuje, że to tylko statyczna strona z komunikatem). A to, że JS jest wymagany bezwzględnie na stronie typu wizytówka to jest inna kwestia, wymóg bezwzględnego JSa powinien być czymś umotywowany. Ten post edytował markonix 29.09.2015, 15:20:19 -------------------- |
|
|
![]()
Post
#13
|
|
![]() Grupa: Zarejestrowani Postów: 260 Pomógł: 26 Dołączył: 26.01.2012 Skąd: Ci to przyszło do głowy? Ostrzeżenie: (0%) ![]() ![]() |
Zrobiłem w sumie pobieranie odpowiednich podstron przez ajaxa, bo okazało mi się to najlepsze, bo strona nie ma zbyt rozbudowanego contentu, więc cały szkielet nie musi się ładować za każdym razem
![]() Dodaj do statycznej strony proszącej o włączenie JS, żeby się odświeżała ![]() -------------------- |
|
|
![]()
Post
#14
|
|
![]() Grupa: Zarejestrowani Postów: 2 707 Pomógł: 290 Dołączył: 16.12.2008 Skąd: Śląsk Ostrzeżenie: (0%) ![]() ![]() |
Zrobiłem w sumie pobieranie odpowiednich podstron przez ajaxa, bo okazało mi się to najlepsze, bo strona nie ma zbyt rozbudowanego contentu, więc cały szkielet nie musi się ładować za każdym razem ![]() Dodaj do statycznej strony proszącej o włączenie JS, żeby się odświeżała ![]() A co da jej samo odświeżanie. A co do tego "ajaxa" to niepotrzebna komplikacja, która i tak działa beznadziejnie. Nie zauważyłeś, że nie działa Ci wstecz/dalej? Ładowanie kontentu w ten sposób musi być przemyślane i także czymś podyktowane. Tutaj to tylko niepotrzebna komplikacja. Nie wiadomo też jak to zadziała na seo. -------------------- |
|
|
![]()
Post
#15
|
|
![]() Grupa: Zarejestrowani Postów: 1 268 Pomógł: 254 Dołączył: 11.06.2009 Skąd: Świętochłowice Ostrzeżenie: (0%) ![]() ![]() |
Cytat więc cały szkielet nie musi się ładować za każdym razem Równie dobrze może to rozwiązać kontroler po stronie serwera zwracając cały szablon przy braku nagłówka X-Requested-With/flagi, że Ajaks oraz jedynie fragmentu strony gdy taki nagłówek/flaga jest. Zwlaszcza, że używasz History API i gdzieś ten cały szablon i tak musisz mieć. Obecny system ładowania strony powoduje, że jeśli wejdzie się bezpośrednio na adres typu http://sciegnytomy.pl/noclegi/agroturystyka to widać FOUC. Jest to spowodowane tym, że cały CSS zasysasz asyncem. Taka technika ma sens tylko wówczas jeśli krytyczne style (czyli above the fold) są zinline'owane, a asyncem dociągasz resztę mniej potrzebną. Jeśli nie wierzysz, przepuść stronę przez http://www.webpagetest.org/ i sam zobacz na w trybie filmstrip (interwał 0.1s) Co do systemu linków - dziwacznie to rozwiązałeś. Atrybut [file] jest całkowicie niepotrzebny (BTW [data-file] byłoby przynajmniej zgodne z wymogami HTML5 ![]() Dodatkowo źle używasz History API. Wejdź sobie bezpośrednio na jakąś podstronę i zobacz ile razy została dodana do historii. Jeśli po otwarciu nowej karty, wklejeniu adresu i pacnięciu "Enter" mam działający przycisk "Wstecz", to znaczy, że coś jest skopsane. Nie wiem czy nie łatwiej byłoby Ci użyć https://github.com/defunkt/jquery-pjax (który nota bene poleca sposób ładowania treści, o jakim już wspominałem: https://github.com/defunkt/jquery-pjax#server-side ) Co do osobnej podstrony no-js: szedłbym raczej w kierunku http://www.paulirish.com/2009/avoiding-the-fouc-v3/ → jak już masz zamiar wypiąć się na userów, zrób to przynajmniej ładnie ![]() -------------------- ★Mój blog || Okiem krytyka★
|
|
|
![]()
Post
#16
|
|
![]() Grupa: Zarejestrowani Postów: 148 Pomógł: 14 Dołączył: 23.02.2013 Ostrzeżenie: (0%) ![]() ![]() |
Szkoda że nawet taka mała stronka wymaga ode mnie włączenia JavaScriptu bo inaczej nic nie zobaczę (poza literówką w informacji że JS jest wymagany)
![]() -------------------- |
|
|
![]()
Post
#17
|
|
![]() Grupa: Zarejestrowani Postów: 60 Pomógł: 8 Dołączył: 7.01.2014 Ostrzeżenie: (0%) ![]() ![]() |
Co ja uważam: Że w ogóle nie ma potrzeby robienia tam animowanego menu - bo to nie jest strona na której trzeba zawrzeć 300 pozycji nawigacji i kombinować z miejscem dla nich.
Przykładowo, masz tam wymienione 3 miejsca z noclegami, a w menu są na to 2 osobne pozycje. Daj to wszystko na jedną i tą samą podstronę - klient zainteresowany noclegami sam doczyta, czy chcąc przyjechać na 2 dni do waszej wsi powinien wybrać agroturystykę czy turystykę wiejską, zjeść nabiał czy może lepiej twaróg, wykąpać się w wodzie czy w jeziorze ![]() W pozostałych działach również polecam to samo - bo nie nadrobisz braku faktycznej treści zawiłością menu, sądzę że aż takich durni to w żadnej wsi nie znajdziesz ![]() Skoro w newsach masz 'co było' i 'co będzie' - nazwij to 'wydarzenia'. Przeniesienie się na nie najpiękniejszą stronę parafii po kliknięciu w menu to niezła terapia szokowa. Lepiej wrzucić tam artykuł opisujący samą architekturę i historię obiektu, i z niego dać link do oficjalnej strony parafii. Zresztą, jak próbuję wrócić zwykłym "wstecz" to zamiast menu wyskakuje jakaś buka strasząca info o wczytywaniu się menu - to też zbędne. Pewnie nie tylko tam. Po prostu widać że menu jest na siłę rozciągnięte jak kupa przez morze za przeproszeniem - zamiast uprościć, zawoalowałeś. Menu przy okazji w ogóle nie pozwala otworzyć sobie czegoś w nowym oknie - a to przydatna funkcja. Facebook ukryty, możliwość polubienia fanpaga po kliknięciu w to menu - więc pewnie mniej osób się pofatyguje. Masz tam tak mało treści że bez problemu ładnie wkomponujesz okienko z fb, w sposób niekrzykliwy a widoczny. Zresztą, powodu do zastosowania JS w ogóle tam nie widzę. Takie samo menu można uzyskać bez JS, nie obciążając użytkownika frustracjami. Nie patrz na to że "duże strony coś tam mają to i twoja musi mieć" - bo dzisiaj strony są tak przeładowane zbędnym kodem że się płakać chce i można od razu wskazać kto na oślep klikał po programach do automatycznego tworzenia stron. Zresztą, co tam niby ma się doczytywać tym JS'em? Artykuł na 12 linii? Jeśli chcesz mieć system oceny artykułów, wystarczy ci jeden maleńki skrypt zapobiegający przeładowaniu strony przy odświeżaniu. Ktoś tu już mówił, ja potwierdzę. Zrób od początku - bo tam nie ma 40 tys artykułów do przepisywania by było to trudne ![]() I nie zastanawiaj się co by tam jeszcze "dowciskać" w stronkę. Jak nie wiesz co, znaczy że nic. Zresztą, nocne zwiechy, i wstrzymywanie się strony na napisie "ładowanie" też nie wpłynie pozytywnie. Zobacz, ile osób ci tutaj doradza, pewnie w ciebie wierzą. Nie zawiedź nas ![]() -------------------- -------------------------Nie naprawiaj tego, co działa---------------------------
|
|
|
![]() ![]() |
![]() |
Wersja Lo-Fi | Aktualny czas: 24.06.2025 - 07:53 |