Skip to content

feature/szkielet wybierania cwiczenia oczu#20

Open
FaziSPB wants to merge 6 commits intodevfrom
feature/cwiczenia-oczu
Open

feature/szkielet wybierania cwiczenia oczu#20
FaziSPB wants to merge 6 commits intodevfrom
feature/cwiczenia-oczu

Conversation

@FaziSPB
Copy link
Copy Markdown
Collaborator

@FaziSPB FaziSPB commented Mar 3, 2026

Opis

przelaczanie widoku strona glowna, wybor, cwiczenie

Powiązane zgłoszenie (Issue)

#17 , #16 , #15

Motywacja i kontekst

spelnienie wymagan dostarczonych przez kn visus dot. dbania o oczy

Jak zostało to przetestowane?

lokalny host, przyciski w pelni dzialaja, wyswietla sie instrukcja cwiczen po najechaniu na przyciski, cwiczenia zajmuja pelny ekran

Zrzuty ekranu (jeśli dotyczy):

Rodzaje zmian

  • Naprawa błędu (zmiana niełamiąca kompatybilności, która naprawia problem)
  • Nowa funkcja (zmiana niełamiąca kompatybilności, która dodaje funkcjonalność)
  • Zmiana łamiąca kompatybilność (naprawa lub funkcja, która spowodowałaby zmianę istniejącej funkcjonalności)

Lista kontrolna:

  • Mój kod jest zgodny ze stylem kodu tego projektu.
  • Moja zmiana wymaga zmiany w dokumentacji.
  • Zaktualizowałem odpowiednio dokumentację.

@FaziSPB FaziSPB changed the title szkielet wybierania cwiczenia oczu feat: szkielet wybierania cwiczenia oczu Mar 3, 2026
@FaziSPB FaziSPB changed the title feat: szkielet wybierania cwiczenia oczu feature/szkielet wybierania cwiczenia oczu Mar 3, 2026
@FaziSPB FaziSPB changed the base branch from main to dev March 3, 2026 18:33
@FaziSPB FaziSPB force-pushed the feature/cwiczenia-oczu branch from 9461ca0 to 5f900a0 Compare March 3, 2026 18:38
@xEdziu xEdziu requested a review from serplay March 3, 2026 19:47
Copy link
Copy Markdown
Collaborator

@serplay serplay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Szkielet wydaje się ok, ale jednak w tym projekcie używamy Tailwind, więc warto go użyć a nie raw style.

@FaziSPB FaziSPB self-assigned this Mar 6, 2026
@FaziSPB FaziSPB requested a review from serplay March 6, 2026 15:23
Copy link
Copy Markdown
Collaborator

@serplay serplay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dobra wszystko się wydaje spoko, ale

Główne uwagi:

  1. Użyj istniejącego hooka useTimer
    Projekt ma już gotowy hook w useTimer.ts z obsługą localStorage. Zamiast pisać timer od zera, lepiej go wykorzystać - zachowamy spójność kodu i unikniemy duplikacji.

  2. Użyj komponentu Button z projektu
    Mamy już ostylowany Button w button.tsx. Warto go użyć zamiast zwykłych <button>, żeby UI było spójne z resztą aplikacji.

  3. Zamień magiczne liczby na typy
    Zamiast currentView: 1 | 2 | 3 lepiej użyć czytelnych nazw:
    type ViewState = 'start' | 'selection' | 'exercise';

  4. Dodaj typowanie dla ćwiczeń
    type ExerciseType = 'osemka' | 'drzewo';

Poza tym reszta okej

@FaziSPB FaziSPB requested a review from serplay March 12, 2026 19:06
@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 18, 2026

@FaziSPB rozwiąż proszę konflikty :D

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 18, 2026

I możesz dodać już całkowicie nową stronę do navbara z deva, wtedy będzie od razu twoje ćwiczenie w nawigacji :)

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

@FaziSPB dalej konflikty :/

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

@FaziSPB co do ósemki, na telefonach średnio działa

Nagrywanie.ekranu.2026-03-24.122855.mp4

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

Jeśli chodzi o merga do maina, nie wiem czemu nie masz navbara?????? @KBruy co się tu stało xD

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

Menu wyboru ćwiczeń powinno być w navbarze

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

Ćwiczenie drzewo jest bez sensu, to po prostu pusty ekran, timer leci, ale nawet po zakończeniu timera nic się nie dzieje, zero żadnego opisu ćwiczenia, to pole jest strasznie mylące, trzeba to obowiązkowo poprawić

@FaziSPB
Copy link
Copy Markdown
Collaborator Author

FaziSPB commented Mar 24, 2026

jesli chodzi o drzewo no to je usunalem z wyboru cwiczen i zastapilem je innym, dodalem je wczesniej zeby po prostu bylo jako baza na pozniej do zrobienia bo najpierw zajalem sie kulką. a co do dodania menu wyboru cwiczen do navbara to srednio wiem o co chodzi, myslalem ze chodzi o dodanie cwiczen tutaj:
obraz_2026-03-24_130503005

@FaziSPB
Copy link
Copy Markdown
Collaborator Author

FaziSPB commented Mar 24, 2026

a na telefonie dziala w poziomie czy tez srednio? bo raczej to cwiczenie jest skierowane do urzadzenia trzymanego w poziomie a nie w pionie, wiec moze na cwiczenia lepiej warto wymusic tryb poziomy (ukryc cwiczenie i poprosic uzytkownika o obrocenie telefonu) niz tworzyc wersje pionowa?

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

Z tym wymuszeniem poziomego to jest jakiś sposób - ale w poziomie też nie działa

nyYTBsWZ.mp4

i kolejny błąd - na ekranach mniejszych na pc tez nie dziala - trzeba zmniejszyc kulke i ograniczyć box po którym może się poruszać:

Nagrywanie.ekranu.2026-03-24.170757.mp4

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

jesli chodzi o drzewo no to je usunalem z wyboru cwiczen i zastapilem je innym, dodalem je wczesniej zeby po prostu bylo jako baza na pozniej do zrobienia bo najpierw zajalem sie kulką. a co do dodania menu wyboru cwiczen do navbara to srednio wiem o co chodzi, myslalem ze chodzi o dodanie cwiczen tutaj: obraz_2026-03-24_130503005

jeśli chodzi o to, to ja miałem jakiegoś laga i tego menu wgl nie było nic a nic, także tu jest git, sorki

@xEdziu
Copy link
Copy Markdown
Member

xEdziu commented Mar 24, 2026

Tylko co właśnie z tym drzewem - wg opisu prki rozwiązujesz to zadanie a drzewa nie ma :/

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.

[TASK] | Trening ruch oczu w dal [TASK] | Menu wyboru ćwiczeń [TASK] | Ćwiczenia oka, menu wyboru ćwiczeń

3 participants