Standardize admin page headers and main page layout#686
Merged
sei-aschlackman merged 13 commits intomainfrom Mar 12, 2026
Merged
Standardize admin page headers and main page layout#686sei-aschlackman merged 13 commits intomainfrom
sei-aschlackman merged 13 commits intomainfrom
Conversation
Display a "TEMPLATE" badge next to view names in the admin views list when isTemplate is true, making it easier to distinguish template views from regular views at a glance.
- Update admin sidebar header to use mat-toolbar with app icon - Reorder admin menu items to match other apps - Fix admin Users section table layout and row height - Update main Views page to match CITE layout - Add consistent spacing and Material Design styling - Remove unnecessary wrapper divs and CSS classes - Fix table cell padding and alignment
Contributor
|
The Administration link in the top left of the Administration section is no longer a link that can be opened in a new tab. I think we should maintain that functionality in all of the apps. Maybe for the app icon in the topbar that brings you to the home page as well? |
| } | ||
|
|
||
| goToHome(): void { | ||
| window.location.href = '/'; |
Contributor
There was a problem hiding this comment.
Will this work in a path-based routing deployment?
Replace click event handler with routerLink directive to enable proper browser navigation behavior including middle-click to open in new tab
Change icon from button with click handler to link with routerLink
Add explicit color style to maintain icon color after changing to anchor tag
Update focused-app container top position from 40px to 46px to match topbar height and prevent overlap
- Change sidebar header to use mat-toolbar for consistency - Set header height to 40px to match admin panel - Use 35px icon size matching admin styling - Apply same border-bottom styling as admin header
Refactored view page sidebar to match Blueprint's collapsible design with 50px mini mode. Added toggle buttons for collapse/expand, improved icon sizing and centering, and included small Crucible logos for both themes in collapsed state. Made all navigation elements middle-clickable for better UX.
Moved routerLink from mat-toolbar to anchor tag to enable middle-click opening in new tab. Addresses PR feedback to maintain middle-click functionality on administration header.
sei-aschlackman
approved these changes
Mar 12, 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.
Summary
Changes
Test Plan