Skip to content

fix: correct semester numbering and rework the semester/week picker - #80

Merged
HeZeBang merged 1 commit into
masterfrom
fix/semester
Jul 26, 2026
Merged

fix: correct semester numbering and rework the semester/week picker#80
HeZeBang merged 1 commit into
masterfrom
fix/semester

Conversation

@HeZeBang

Copy link
Copy Markdown
Owner
  • Semester term numbers sent to /schedule/term_begin were derived from a 春/秋-substring check that silently collapsed 秋 and 暑 (and any raw digit-keyed term) onto "2"; switch to a shared 秋=1/春=2/暑=3 mapping used consistently for both display and the API request.
  • Replace the flat semester list (mobile bottom sheet, iOS native submenu, desktop popover) with a synced year/term scroll-wheel picker shared across all three surfaces.
  • Consume the full /schedule/term_begin response (teaching-week count, class-period counts, etc.) instead of just the term-begin date, so the week picker/pager sizes itself to the semester's actual teaching-week count and hides the "this week"/"today" indicators when today falls outside the selected semester.
  • Swap the hand-rolled fade+slide week-number transition for the reel_text package's roll animation.

- Semester term numbers sent to /schedule/term_begin were derived from a
  春/秋-substring check that silently collapsed 秋 and 暑 (and any raw
  digit-keyed term) onto "2"; switch to a shared 秋=1/春=2/暑=3 mapping
  used consistently for both display and the API request.
- Replace the flat semester list (mobile bottom sheet, iOS native submenu,
  desktop popover) with a synced year/term scroll-wheel picker shared
  across all three surfaces.
- Consume the full /schedule/term_begin response (teaching-week count,
  class-period counts, etc.) instead of just the term-begin date, so the
  week picker/pager sizes itself to the semester's actual teaching-week
  count and hides the "this week"/"today" indicators when today falls
  outside the selected semester.
- Swap the hand-rolled fade+slide week-number transition for the
  reel_text package's roll animation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this to Backlog in TechPie Jul 26, 2026
@HeZeBang
HeZeBang merged commit 9436fe0 into master Jul 26, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in TechPie Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant