feat: 重构管制员中心 - #58
Merged
Merged
Conversation
0x00A0
marked this pull request as draft
July 18, 2026 21:25
0x00A0
changed the title
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Separate the applications page into two distinct sections: - All users see their own application card and can apply when eligible - Training directors see additional review section with the full review table The canApply logic now treats both 'rejected' and 'aborted' statuses as re-applicable, allowing users to resubmit after abort. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- identity chip in green for legibility in both color schemes - add SOP and LOA links to controlling resources - move resources into a sticky right sidebar on desktop so long training lists no longer push them below the fold - cap training list height with its own scrollbar Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…terpart name - training list now caps at max(24rem, viewport minus page chrome) on desktop instead of a fixed 32rem, and scrolls with the page on mobile - list rows and detail meta show the other party: trainees see the trainer, trainers see the trainee Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- inner master-detail grid uses minmax(0) tracks with min-w-0 children so long monospace lines no longer force horizontal page overflow on phones and tablets - resources sidebar moves to xl so iPad landscape keeps a single column - selected training marked with a green left border; upcoming rows keep their badge - training list rows, mobile back button, and resource cards now use Mantine UnstyledButton/Button Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Revert the UnstyledButton conversions; the Mantine guideline means no third-party UI libraries besides Mantine, not wrapping every native element. Overflow fixes, green selected border, and xl sidebar stay. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Prevents the become-a-controller guide from flashing before the dashboard for logged-in controllers on refresh. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…dy toolbar Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ear contrast in both schemes Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… translations Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…black too Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…t via search param Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…t view Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…side the center layout Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…lications item Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…on refetch loop Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…etches cannot loop it Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ment roles to lib Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The two-role gate locked out controller-training-instructor and controller-training-director, who could reach the old ungated page. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…t tab The instructor identity lives in ATC permission state (UserControllerState "mentor"), not in UserRole, so the role-only gate hid the tab from mentors. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…e application entry at the form Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…e application entry at the form Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Item hover was gray-50 on a white dropdown (near-invisible) and large tiles lightened on hover; both now darken one step in light mode. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…utton Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…dcoded military roster Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Hide the redundant global search (the controller column already has its own text filter), pin the header while scrolling, center the position and military column headers, sharpen the mentor/military badge colors, and shorten the military column label to MIL. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
0x00A0
marked this pull request as ready for review
July 22, 2026 02:18
xfoxfu
previously approved these changes
Jul 22, 2026
Closed
xfoxfu
approved these changes
Jul 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
新增
移除