Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rozwijany FAB do filtrowania wyświetlanych zajęć #131

Open
mpygruu opened this issue Sep 29, 2024 · 4 comments
Open

Rozwijany FAB do filtrowania wyświetlanych zajęć #131

mpygruu opened this issue Sep 29, 2024 · 4 comments
Assignees

Comments

@mpygruu
Copy link
Contributor

mpygruu commented Sep 29, 2024

Zaimplementować rozwijany FAB, który będzie odpowiedzialny za filtrowanie widoku na podstawie mapy byStudentGroups w klasie ScheduleFiltersInfo. Narazie bez funkcjonalności filtrowania, niech tylko prezentuje dane o semestrach i grupach z byStudentGroups po kliknięciu odpowiedniej opcji

@mpygruu mpygruu self-assigned this Sep 29, 2024
@mpygruu
Copy link
Contributor Author

mpygruu commented Sep 29, 2024

FAB zrobiony, narazie na sztywno a nie na podstawie filtrów. Dodałem na figmie (screen poniżej) propozycję wyboru grup, semestru i planu.

Podepnę FABa do filtrów byStudentGroups jak issue 113 zostanie zmergowany.

Dodatkowo przyda się kilka ifów, żeby na przykład:

  • jeśli ktoś jest na pierwszym semestrze to nie pokazywać zmiany semestru,
  • jeśli ktoś patrzy na plan wykładowcy to nie pokazywać zmiany grupy i semestru, a może w ogóle zamiast rozwijanego faba to zrobić pojedynczy, na którym jest symbol edit_calendar.

Zrzut ekranu 2024-09-29 o 18 47 34

@matyjb
Copy link
Contributor

matyjb commented Sep 29, 2024

Semestry lepiej przenieść do kalendarza jako takie skróty co przenoszą do pierwszego dnia danego semestru jako tagi (coś jak zazwyczaj się daje taki guzik "Dzisiaj"), bo te semestry to właśnie takie w sumie labele na osi czasu danego toku studiów. Te kółeczka z cyfrą semestru fajne, coś takiego może da się upakować na tym kalendarzu (jak się kliknie w aktualny dzień to się otwiera ten kalendarz, o ten mi chodzi)

image

  • Te rzeczy dotyczące wybierania czasu są wtedy w jednym miejscu
  • Może będzie to zbyt ukryte? Ale w sumie jak user będzie się chciał cofnąć bardzo to i tak sobie odpali ten kalendarz, a tu pyk od razu przygotowane skróty do skakania do danego sema

Odnośnie pytania: Hmm, można wymusić tego faba by był zawsze widoczny albo w ustawieniach tam gdzie jest "Zarządzanie planami" będzie opcja zmiany widocznego planu (tak jak to jest w obecnej apce)

@mpygruu
Copy link
Contributor Author

mpygruu commented Sep 29, 2024

Hm, przemyślałem na świeżo, i wydaje mi się że skakanie pomiędzy semestrami to będzie event na tyle rzadki, że w sumie wystarczy przeklikanie dialogu z kalendarzykiem, bez dodatkowych przycisków. Takie przyciski byłyby bardzo fajne ale w tym przypadku skłaniałbym się do uproszczenia UI.

Myślałem o tym przycisku ze zmianą semestru też ze względu na to, że grupy studenckie się różnią między semestrami dla tego samego rocznika, ale ten problem wystarczy rozwiązać biorąc grupy dla tego semestru, do którego należy aktualnie wybrany dzień.

Jeżeli rezygnujemy z przycisku dla semestru to pozostają tylko dwa mini faby, czyli dokładnie tyle żeby rozwijalny fab nie był potrzebny 🤔 Myślałem co może się userowi przydać zamiast zmiany semestru ale na ten moment nie mam pomysłu. Jak nie wymyślimy żadnego power toola to bym też zaproponował powrót do dużego faba do zmiany planu oraz małego faba do zmiany grupy, bez rozwijania.

@mpygruu
Copy link
Contributor Author

mpygruu commented Sep 29, 2024

A co do odpowiedzi na pytanie to tak, zróbmy jak jest w obecnej apce. Jak ktoś wyłącza FAB to akceptuje konsekwencje. xD

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

When branches are created from issues, their pull requests are automatically linked.

2 participants