Skip to content

Shared UI fixes and short toasts - #144

Closed
HarryRandall wants to merge 5 commits into
mainfrom
feat/stack-1-shared-ui
Closed

HarryRandall wants to merge 5 commits into
mainfrom
feat/stack-1-shared-ui

Conversation

@HarryRandall

@HarryRandall HarryRandall commented Sep 26, 2026 •

Copy link
Copy Markdown
Owner

Requested by haz · project thread

Summary

First of six PRs that split feat/requirement-placements by feature: #144 → #145 → #146 → #147 → #148 → #149. Each builds on the one before, so merge with merge commits, not squash.

Before: toasts were long and inconsistent, breadcrumbs showed years, empty states stopped short of the page bottom, search boxes could drift from the URL while typing, and admin search needed an exact phrase.

After: every toast is a short title with at most two lines of detail through a shared showToast, long admin tasks show a progress bar that fills before completing, breadcrumbs show the parent page, empty states stretch to the bottom, search boxes stay in sync, admin search matches every word, and a few layout fixes land (course search drops its Requisites column, option picker lists scroll in dialogs, collapsed sidebar padding, dashboard width, tuition bars).

How: five commits (toasts, breadcrumbs, empty states, search, layout fixes). Toast call sites in files that later PRs rewrite arrive with those PRs.

Verification

  • pnpm format:check, pnpm check:repository, ESLint, tsc and unit tests pass; CI green
  • Browser smoke test

Deployment notes

None.

@vercel

vercel Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
coursemap Ready Ready Preview Sep 26, 2026 8:22am UTC

Copy link
Copy Markdown
Owner Author

Replaced by #150 (same commits, branch renamed to feat/shared-ui-fixes).


Generated by Claude Code

This branch was successfully deployed

1 active deployment
Preview — 90c445dc Deployed Sep 26, 2026 by vercel[bot]
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.

2 participants