-
Notifications
You must be signed in to change notification settings - Fork 5
refactor(ui): homogenize Account, Organization detail, and Admin pages #4187
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
Merged
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
cc8d378
chore(git): ignore top-level .worktrees/ directory
PierreBrisorgueil 6836f22
feat(core): add coreConfirmDialog shared destructive-confirm component
PierreBrisorgueil e4179e4
feat(core): add coreAvatarUploader (v-badge camera overlay, no inline…
PierreBrisorgueil 3f37e9b
test(core): use flushPromises in coreAvatarUploader test (cleaner tha…
PierreBrisorgueil 6e89b87
feat(admin): currentBreadcrumb state so sub-views push titles to the …
PierreBrisorgueil 7c4e72d
test(admin): split currentBreadcrumb test for clearer failure messages
PierreBrisorgueil 1b08897
refactor(admin): banners on top, tabs in PageHeader, breadcrumb suppo…
PierreBrisorgueil b394082
refactor(admin): resolveActiveTab uses allTabs (single source of truth)
PierreBrisorgueil 1533b71
refactor(admin): drop v-container on list sub-views; users.view uses …
PierreBrisorgueil 71efb3d
refactor(admin): drop gratuitous div wrapper in admin.users.view (Vue…
PierreBrisorgueil 89e13be
refactor(admin): activity view — drop chrome + inline styles, use v-r…
PierreBrisorgueil 94b0a88
refactor(admin): user detail — drop nested PageHeader, push breadcrum…
PierreBrisorgueil e9075ac
refactor(users): profile component — coreAvatarUploader + native Vuet…
PierreBrisorgueil 140657b
refactor(users): drop dead isActiveOrg method from profile component
PierreBrisorgueil d50cfcc
refactor(admin): migrate layout to CoreSurfaceTabBar — chrome converg…
PierreBrisorgueil 2265422
refactor(users): account dialogs — Delete Account + Leave Org use cor…
PierreBrisorgueil ab2bae4
refactor(organizations): detail — Delete Organization uses coreConfir…
PierreBrisorgueil e36392e
fix(admin): remove unused beforeEach import in admin.user.view tests
PierreBrisorgueil 4380784
fix(admin): clear stale breadcrumb on blank user; fix org delete JSDoc
PierreBrisorgueil 9f545c7
fix(review): address CodeRabbit + Copilot findings
PierreBrisorgueil File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.