Skip to content

Front create game#310

Open
PEXEL2002 wants to merge 72 commits intomainfrom
front-create-game
Open

Front create game#310
PEXEL2002 wants to merge 72 commits intomainfrom
front-create-game

Conversation

@PEXEL2002
Copy link
Collaborator

@PEXEL2002 PEXEL2002 commented Jun 14, 2025

Siemka napisałem podstrone z dodawaniem nowej gry /home/create-game:

Należy sprawdzić:

  • Poprawność ortograficzną
  • Podstawowe działania mechaniki podstrony

Myjem and others added 30 commits June 2, 2025 04:14
Musiałem trochę pogrzebać, najważniejsza zmiana to teraz owner w game może być null.
@PEXEL2002
Copy link
Collaborator Author

  • nie ma sprawdzenia długości tytułu ani opisu
  • tytuł nie powinien móc zaczynać się od spacji
  • znaków specjalnych chyba też nie można?

Ad.1 Ograniczyłem do 75 znaków
Ad.2 W późniejszym JS sprawdzam to i usuwam więc nie ma się martwić
Ad.3 Na moje nie ma sensu bo możesz mieć tytył na zasadzie: Lustria (odłam głównej kampanii Warhammer)

@PEXEL2002
Copy link
Collaborator Author

  • przycisk ANULUJ nie przekierowuje nigdzie - nie ma jeszcze tej strony, bo to będzie do gier, ale zakładam url /home/games (ew. bez home?), możesz @Myjem potwierdzić, aczkolwiek myślę, że warto to zrobić, żeby potem nie trzeba było się wracać do tej strony
  • bardzo duży przycisk plus poniżej użytkowników w grze - można by go zmniejszyć, żeby było widać więcej użytkowników naraz
  • duże odstępy między użytkownikami w grze - tak samo, można by zmniejszyć, żeby było widać więcej użytkowników naraz
  • znak rozwinięcia do wyboru roli mógłby nie być tak blisko krawędzi/na granicy przycisku, a bardziej taki odstęp jak nazwa roli ma od lewej strony
  • dynamiczne wyszukiwanie użytkowników - pokazywanie wyników po każdym nowym znaku/usunięciu znaku, a nie po enterze

to zmieniłem

@szxxlc
Copy link
Collaborator

szxxlc commented Jun 19, 2025

@PEXEL2002 czy na pewno wszystkie zmiany zostały zpushowane? bo już widzę, że pierwszej (jedynej nieopcjonalnej) nie ma i tak nie wiem ile sprawdzać i opisywać, że nie ma, a i ile wyjdzie, że może siedzi lokalnie

@szxxlc
Copy link
Collaborator

szxxlc commented Jun 20, 2025

to zmieniłem

Brakuje jednego (jedynego) nieopcjonalnego fixa do problemów, o których wspomniałam - przycisk ANULUJ nadal nie przenosi do /home/games. Także teraz zauważyłam, że albo przycisk UTWÓRZ GRĘ również nic nie robi, albo nie ma informacji zwrotnej dla użytkownika po walidacji z informacją dlaczego nie można utworzyć gry.

@szxxlc
Copy link
Collaborator

szxxlc commented Jun 20, 2025

@PEXEL2002 utworzyłam zbiorowe issue #331 z problemami, które trzeba rozwiązać (bez obowiązkowych) - są tam rzeczy wspomniane przeze mnie w ostatnim komentarzu oraz jeden z wcześniej wspomnianym problemów jako opcjonalny, lecz postanowiłam, że jest on jednak konieczny (fioletowe iksy i przycisk ANULUJ)

@szxxlc szxxlc linked an issue Jun 20, 2025 that may be closed by this pull request
4 tasks
@szxxlc
Copy link
Collaborator

szxxlc commented Jun 20, 2025

Poza tym, to fajnie, że część z opcjonalnych fixów się pojawiła. Te co się nie pojawiły, to rozumiem, że została podjęta decyzja, że ich nie robisz, lecz wstawiam je tutaj spisane, dla uporządkowania (też żebym ja wiedziała, co się już ode mnie pojawiło jako zgłoszone):

  • bardzo duży przycisk plus poniżej użytkowników w grze - można by go zmniejszyć, żeby było widać więcej użytkowników naraz
  • przy zmianie roli użytkownika z wybranej na inną, wszystkie możliwości mają kolor tła taki jak poprawna - najlepiej jakby miały każda swój lub wszystkie na biało (to pomijając fakt, że obecnie kolor roli nie działa poprawnie)
  • znak rozwinięcia do wyboru roli mógłby nie być tak blisko krawędzi/na granicy przycisku, a bardziej taki odstęp jak nazwa roli ma od lewej strony

@PEXEL2002

@Kosiorny
Copy link
Collaborator

zauważyłem że po utworzeniu gry, jak sprawdzam uczestników to nie dodaje osoby która miała role SPECTATOR, ale nie wiem czy to kwestia frontu w sumie
image
image
image

@xEdziu
Copy link
Owner

xEdziu commented Aug 1, 2025

@Kubis10 chciałbyś przejąć ten task na sam start?

@Kubis10
Copy link
Collaborator

Kubis10 commented Aug 1, 2025

Spoko mogę przejąć

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] | Frontend | Problemy ze stroną create-game [FRONTEND] Strona z tworzenia gry

8 participants