Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Overwatch24.pl - potrzebny feedback
Forum PHP.pl > Inne > Oceny
Forti
Cześć!

Po blisko 8 miesięcznej przerwie w developingu postanowiłem wrócić do projektu.

Pozostaje kilka rzeczy do ogarnięcia, ale po krótce opisze założenia:

Portal od fanów dla fanów gry Overwatch. Obecnie mamy logowanie z użyciem kont battle.net. Gdy tylko pojawi się API gry (wg. ekipy dev.battle.net ma to nastąpić zaraz po premierze gry) mam zamiar rozbudować portal o rankingi, profile itp. - liczę, że będzie to jeden z najpopularniejszych i "najprzydatniejszych" tego typu stron dla Polskich graczy. OBecnie bez ów API jest to po prostu kolejna tego typu strona w necie, niczym specjalnym nie wyróżniająca się na tle innych, już online od pewnego czasu.

Adres: https://overwatch24.pl
obecnie chronione htaccess, login/haslo: forti/dude


Grafikiem nie jestem, więc pewnie do tego będzie najwięcej uwag. Kilka rzeczy wymaga poprawy (np. "baner" pobiera się fullHD zamiast odpowiednio przycięty i skompresowany.

Jest również wersja mobilna, która wymaga kilka drobnych poprawek.

Macie jakieś konkretne uwagi, pomysły itp.? Chętnie posłucham smile.gif


Jeżeli ktoś chce dostęp do panelu admina to piszcie wasze zalogowane battle.tag - nadam dostęp.
Comandeer
https://overwatch24.pl/assets/js/search.js -> 404

Kod
<meta name="keywords" content="">

Ostatnio jakaś plaga z tymi pustymi meta.

CSS można by z powodzeniem zminifikować.

[aria-labelledby="myModalLabel"] -> trochę głupio bawić się dostępnością i zapomnieć wstawić elementu #myModalLabel wink.gif

Jak już przy dostępności jesteśmy, krótki cytat ze specki HTML5:
Cytat("http://www.w3.org/TR/html5/forms.html#the-placeholder-attribute")
The placeholder attribute should not be used as a replacement for a label. [...] Use of the placeholder attribute as a replacement for a label can reduce the accessibility and usability of the control for a range of users including older users and users with cognitive, mobility, fine motor skill or vision impairments.


Hierarchia nagłówków jest zwalona równo (polecam zobaczyć jak validator.w3.org/nu generuje outline dla tej strony).

Czemu nie ma HTTP/2? wink.gif
Forti
Okej, o http/2 nawet nie myślałem. Puste meta są gdy jest noindex, (nie powinno ich również tam być) być może gdzieś jakiś bug jest.

Co do nagłówków - hierarchia od h1 do h6, z czego h1 tylko jeden na widoczny HTML może być, prawda? Więc w edytować tekstu np. Z drugiej szacunek artykuł trzeba o to odpowiednio zadbać. Muszę to ogarnąć konkretniej.

Gdzie jest użyty place Holder zamiast label? Na pewno w wyszukiwarce (która nie jest zakodowana jeszcze po stronie kontrolera). Pytanie - w czym to wadzi?
Comandeer
Co do nagłówków: tak, 1 h1 to najsensowniejsza jego ilość.

Co do [placeholder] - wadzi to w tym, że placeholder niekoniecznie jest obsługiwany przez czytniki ekranowe i znacząco obniża dostępność formularzy.
memory
3/10 błąd za błędem. Przed wejściem nic nie wiedziałem o grze i po wejściu na stronę to samo. Żadnych treści,artykułów,wideo,opisów.

Grafika ok, tylko stopka jakoś nie pasuje
Forti
Cytat(memory @ 26.12.2015, 10:39:01 ) *
3/10 błąd za błędem. Przed wejściem nic nie wiedziałem o grze i po wejściu na stronę to samo. Żadnych treści,artykułów,wideo,opisów.

Grafika ok, tylko stopka jakoś nie pasuje



No, tutaj nie masz oceniać treści O_o przeczytałeś wogóle co napisałem w pierwszym poście? Treść przyjdzie jak strona już będzie opublikowana.

Opisz proszę co znaczy "błąd za błędem" wink.gif
memory
- przycisk kolejna strona
- wyszukiwarka nie działa, jak wpadłeś na to by dać przycisk po lewej stronie
- fullhd baner
- galeria napis loading ... ledwo widoczny.

Wypadało by wypełnić chociaż testowymi danymi abym mógł sprawdzić zachowanie tagów,wyszukiwarki itd czy działają
Forti
Cytat
- przycisk kolejna strona


działą dobrze. Będą artykuły to będzie gitara. Teraz zwraca dobrze, że brak kolejnych wyników.

Cytat
- wyszukiwarka nie działa, jak wpadłeś na to by dać przycisk po lewej stronie


Nie czytasz tego co się piszę w temacie a odpowiadasz

Cytat
- fullhd baner


zgaduje - napisałem o tym, ale nie przeczytałeś smile.gif

Cytat
- galeria napis loading ... ledwo widoczny.


Dzięki, zajmę się tym smile.gif
Spawnm
https://overwatch24.pl/login

górne menu mogło by się zmniejszać przy scrollu, obecnie zbyt wielkie.

Jeśli mam być szczery to obecnie nie widzę tu zbyt wiele do oceny. Całość wygląda jak podstawowy szablon bootstrapa lub wp z banerem i zbyt wielkim top menu.

"2015 Artur 'Forti' Schroeder" <- serio tyle przestrzeni na taki napis?

Nie da się przetestować jak wygląda/działa strona z paginacjami oraz contentem chociaż w stylu loren ipsum, możliwe że masz źle poustawiane paddingi itd ale tego nie da się stwierdzić bo obecnie na tej stronie nic nie ma

Forti
Zaraz zrobie mały updated i wypełnie danymi. Dajcie mi 5-10 minut.

O którym menu mówisz? O tym malutkim "szarym"?
Spawnm
Chodzi mi o to czarne z logiem, można by dodać bajer z shrinkiem http://www.bootply.com/109943 i zmniejszyć na dzień dobry. Obecnie całość przez te wielkie footery i menu wygląda jak by robił to gimnazjalista na zaliczenie
Forti
No nie wiem. Z założenia to "menu", header nie ma działać w ten sposób. To ma być "Statyczny" element na górze strony. Chociaż.. można by to przemyśleć.

Dodałem kilka map i postaci do wikipedi, widzę że tagi troche nie funkcjonują tak jak powinny.
Spawnm
Jeśli nie ma działać w ten sposób to i tak nie powinien być tak wielki wink.gif

https://overwatch24.pl/uzytkownik/profil/1# <- klikam wyślij wiadomość i nic się nie dzieje.
Szukajka zwraca error 500
Forti
W wyszukiwarke pewnie wpisujesz coś nie z listy. Jeszcze do końca tego nie ogarnąłem. Małe kroki tongue.gif
Spawnm
Cytat(Forti @ 26.12.2015, 14:01:52 ) *
W wyszukiwarke pewnie wpisujesz coś nie z listy. Jeszcze do końca tego nie ogarnąłem. Małe kroki tongue.gif

overwatch i battle.net wink.gif
Jak działa twoja wyszukiwarka? Przy battle.net powinno zwrócić artykuł
Forti
Obecnie Typeahead po tytule.

Bez wybrania z podpowiedzi będzie wyszukiwac po tagach. Jeszcze do zakodowania.
Pyton_000
Nie oceniam bo mało jest co wink.gif
Forti
Wywaliłem ten baner chwilowo całkowicie. Później wstawie tam slider z top artykułami czy coś i tylko na stronie głównej.

Artykuły liste zrobiłem po dwa kafelki. Zastanawiam się czy i tak nie są za duże. Tak jak było wcześniej to w sumie brzydkie było biggrin.gif

Kurde, 90% czasu schodzi mi na front. Słaby w to jestem.
Spawnm
https://overwatch24.pl/artykul/battle-net-lorem-ipsum-3

Chyba coś zepsułeś.
Szukajka dalej 500.

Skończ projekt i wróć jak będzie działać. Obecnie zaczynasz zaśmiecać forum mini zmianami i fakeupami.
Forti
Nie wiem kiedy szukają tobie nie działa. Znaczy się wiem, ale to szczegół chwilowo. Ten artykuł działa, tylko nie ma treści.

Okej, na prawiłem kilka mniejszych bugów i dodałem ten "shrink" do menu.

Co sądzisz teraz, spawn? wink.gif
Spawnm
Usuwasz logo z menu? biggrin.gif
Szukajka dalej Opss... coś poszło źle... Spróbuj później. czyli dla mnie jako casuala co cię odwiedził przypadkowo dalej wygląda na zepsutą.

http://i.imgur.com/pI1r3aY.png artykuły serio tak mają wyglądać?
https://overwatch24.pl/uzytkownik/profil/1# klikanie wyślij wiadomość dalej nic nie robi

Co tak właściwie się zmieniło po za hidden na logo przy scrollu?

Zmień aby przy szukajce nie wyskakiwał input na tle blokującym stronę tylko obok lupki wysuwał się input.
Forti
Nie wiem co robisz w szukajce. Zablokuje button i odblokuje na event wyboru tytułu z listy z podpowiedziami. To nie ma działać tak jak myślisz, że ma działać - że pwisujesz byle słowo i pokazuje wszystkie możliwe treści wink.gif Ten input jako "popup" zamiast modala no nawet dobry pomysł.

Artykuły tak, tutaj ciągle walcze. Cieńki jestem z frontu biggrin.gif

Logo lepiej nie usuwać, zapisane.


Mam problemy z odpowiednim wyglądem tego, nie mam takiego hm... jakby to ująć - zmysłu artystycznego - robię co mogę smile.gif

-------------------
okej, dość sporo rzeczy poprawiłem, również na mobile. Jest znacznie lepiej, technicznie wszystko działa jak należy, wizualnie jeszcze pewnie coś można usprawnić.

HTML jest troche do poprawy wg. w3c. W przyszłym tygodniu się tym zajmę.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2018 Invision Power Services, Inc.