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

notepad module logic setup #3

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

notepad module logic setup #3

wants to merge 14 commits into from

Conversation

FiX1iN1iT
Copy link
Collaborator

@FiX1iN1iT FiX1iN1iT commented Feb 28, 2024

Исправить:

  • Поддержка темной темы
  • Завершение тренировки, не только когда все упражнения выполнены
  • Переход из Training в Exercise по кнопке, а не по нажатию на ячейку
  • Пропуск упражнения по нажатию на checkbox
  • Число упражнений до, а не 0

Созвон:

  • Локализовать модуль Notepad
  • Изменить сущности (Entities)
  • CustomError -> AlertManager

Добавить:

  • Loading view в модуль Notepad
  • Свой кастомный чекбокс
  • Взаимодействие с Firebase
  • Экран Extra с дополнительными замерами
  • Кастомную таблицу из Training в отдельный класс
  • Перемещение между днями в Notepad
  • Добавить tableView и label в scrollView в Notepad (убрать scroll у tableView)
  • План max
  • Нормальный UI

Созвон:

  • Кастомную кнопку в отдельный класс
  • Создать тестового юзера в Firebase

@FiX1iN1iT FiX1iN1iT requested review from MX126 and whtelie February 28, 2024 18:42
@MX126
Copy link
Collaborator

MX126 commented Feb 29, 2024

Добавь SwiftLint + я бы подумал насчет использования фреймворка M13Checkbox, может есть смысл сделать свой класс с анимациями, чтобы не переполнять проект, так как в других модулях похожего функционала на макетах не увидел

@whtelie whtelie self-requested a review March 1, 2024 09:30
Copy link
Collaborator

@whtelie whtelie left a comment

Choose a reason for hiding this comment

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

нарушена последовательность, как мне кажется

как я видел, что в день, когда чел идет в зал у него на экране начала тренировки просто список упражнений и он нажимает кнопку начать ->
1е упражнение начать/|закончить, свой результат| -> предложить отдых( показываем какое будет следующее упражнение)
-> начать следующее

и так мы будем просто подсказывать и помогать с временем отдыха, но не грузить, щас почти тоже самое только чел выбирает упражнение которое он сделал, нажимает закончить, выбирает отдых или продолжает

при ином результате можно сделать две кнопки (закончить, иной результат) прост в моей голове у тебя есть план тренировки в котором расписано, что и сколько делать, поэтому не думаю, что иной результат обладает таким приоритетом, что нужно выделять возможность записать иной результат после каждого упражнения, а просто скипать вместе с выполненным упражнением

таймер отдыха я видел после каждого упражнения, тк если я правильно понял, там в день фулбоди 2 сета и только после каждого идет отдых, хотя это можно сделать опциональным, при создании тренировки

мне кажется что пользователь в таком решении слишком много внимания уделяет приле, нам нужно сократить максимально, но удерживать в нем, чтобы не было итакусталпох

пока не забыл, можно в начале трени включать секундомер и после окончания тренировки вычитать время отдыха = время тренировки

Copy link
Collaborator

@MX126 MX126 left a comment

Choose a reason for hiding this comment

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

1 - при запуске первого экрана модуля сначала ничего видно, потом появляется информация, предлагаю добавить индикатор подгрузки

2 - хотелось бы большей тактичности с ячейкой программы в первом экране модуля (например свайп чтобы удалить или запланировать эту же тренировку)

3 - на экране с тренировкой есть вариант, что пользователь хочет завершать программу, даже если выполнил не все упражнения, надо эту кнопку сделать доступной всегда а не по клику на все упражнения



4 - если я хочу изменить количество повторений упражнения и жму на это, то хотел бы видеть какое число было до этого( а не 0)

5 - как-будто можно план max сразу начать тут делать, в модуле много «свободного места»

6 - поддержка темной темы

Everyday/Entities/Workout.swift Show resolved Hide resolved
Everyday/Services/Day/DayService.swift Show resolved Hide resolved
@FiX1iN1iT
Copy link
Collaborator Author

я бы подумал насчет использования фреймворка M13Checkbox, может есть смысл сделать свой класс с анимациями

Может есть, но сейчас я решил рациональнее сделать рабочий модуль

@FiX1iN1iT
Copy link
Collaborator Author

нарушена последовательность, как мне кажется

Мне не совсем понятно то, что ты написал, поэтому предлагаю обсудить на созвоне.
Более того, ты не совсем верно описал логику работы модуля: изменить записанные результаты можно не после каждого упражнения, а после каждого сета. Тут идея заключалась в том, что пользователь, может, не захочет заходить в каждое упражнение, а просто хочет записать результаты в конце всего сета.

@FiX1iN1iT
Copy link
Collaborator Author

1 - при запуске первого экрана модуля сначала ничего видно, потом появляется информация, предлагаю добавить индикатор подгрузки

Отличная идея. Так и планировалось.

2 - хотелось бы большей тактичности с ячейкой программы в первом экране модуля (например свайп чтобы удалить или запланировать эту же тренировку)

Я думаю, это уже второстепенный функционал, так как, как мне казалось, планирование тренировок будет происходить на другом экране. В целом идея хорошая, предлагаю вынести на созвон.

3 - на экране с тренировкой есть вариант, что пользователь хочет завершать программу, даже если выполнил не все упражнения, надо эту кнопку сделать доступной всегда а не по клику на все упражнения



Хорошая идея.

4 - если я хочу изменить количество повторений упражнения и жму на это, то хотел бы видеть какое число было до этого( а не 0)

Так и планируется: в Figma я так и проектировал.

5 - как-будто можно план max сразу начать тут делать, в модуле много «свободного места»

Я думал, pr создается не тогда, когда модуль готов к финальной защите.

6 - поддержка темной темы

Да.

@FiX1iN1iT FiX1iN1iT requested a review from MX126 March 2, 2024 15:41
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.

4 participants