Conversation
commit 324b238 Merge: 2d4faa5 b29bdca Author: Yurin Andrey <andreyyurin@icloud.com> Date: Thu Sep 10 23:02:58 2026 +0300 Merge branch 'minorchange/schedule-color' of https://github.com/BitCodersNN/unn-mobile into minorchange/schedule-color commit 2d4faa5 Author: Yurin Andrey <andreyyurin@icloud.com> Date: Thu Sep 10 23:02:54 2026 +0300 Изменил цвет фона при чередовании commit c83544e Author: Yurin Andrey <andreyyurin@icloud.com> Date: Thu Sep 10 22:57:51 2026 +0300 Revert "Изменил цвет фона при чередовании пар" This reverts commit b4f831e. commit b29bdca Author: UNN MOBILE runner <unnmobile@mail.ru> Date: Thu Sep 10 18:08:03 2026 +0000 Change version in pubspec commit b4f831e Author: Yurin Andrey <andreyyurin@icloud.com> Date: Thu Sep 10 21:05:36 2026 +0300 Изменил цвет фона при чередовании пар
|
GitHub Actions
Error
- Вынесены общие виджеты и создан AppSearchController<T>. - Логика перенесена в ViewModels. - Экспорт вынесен в отдельный файл. - Нумерация недель привязана к семестрам, добавлен скролл к сегодня.
GitHub Actions
Failure
| DateTime get normalizeStartOfDay => copyWith( | ||
| hour: 0, | ||
| minute: 0, | ||
| second: 0, | ||
| millisecond: 0, | ||
| microsecond: 0, | ||
| ); | ||
|
|
||
| DateTime get endOfDay => copyWith( | ||
| hour: 23, | ||
| minute: 59, | ||
| second: 59, | ||
| millisecond: 999, | ||
| microsecond: 999, | ||
| ); |
There was a problem hiding this comment.
Почему один normalize, а другой нет?
There was a problem hiding this comment.
Мб который normalize назвать просто startOfDay?
| static String format( | ||
| DateTime dateTime, | ||
| String pattern, { | ||
| String local = 'ru_RU', |
There was a problem hiding this comment.
А давно мы локаль через local называем, вместо locale? :D
Просто так уже было, судя по всему, но чёто до этого не замечал как-то)
| DateTime get start => _reference.startOfWeek.addWeeks(weekOffset); | ||
|
|
||
| DateTime get end => start.endOfWeek; |
There was a problem hiding this comment.
Я конечно хз, мб дарт сам это оптимизирует как-то, но я бы не стал эти даты каждый раз пересчитывать, особенно учитывая, что weekOffset финальный
|
|
||
| extension IdTypeExtensions on IdType { | ||
| String getDisplayName() => switch (this) { | ||
| String get getDisplayName => switch (this) { |
There was a problem hiding this comment.
Если делать геттер, то пусть уж будет displayName тогда
| _exportScheduleService.requestCalendarPermission(); | ||
|
|
||
| Future<bool> exportSchedule(DateTimeRangeType type) async { | ||
| final week = WeekRange(weekOffset: weekOffset); |
There was a problem hiding this comment.
Мб сделать selectedTimeRange из поля класса, только переделать его на WeekRange тоже, вместо текущего DateTimeRange?
| final nextStart = _nextSemester(semester).start; | ||
| if (!monday.isBefore(nextStart) || | ||
| monday.addWeeks(1).startOfWeek.isSameDate(nextStart)) { | ||
| return _nextSemester(semester); |
There was a problem hiding this comment.
В продолжение предыдущего коммента - тут тогда тоже можно будет вместо понедельника принимать целиком weekOffset, или вообще из класса его брать - как больше нравится, и использовать оттуда даты для проверки
This branch was successfully deployed
3 participants
resolves #697
Я вижу дизайн этой странички так. Глянь тоже: если по дизайну всё устраивает, то маякни, чтобы я начал рефакторить этот код =) Также, если встретишь баги, пиши об этом.
Bugs: