AB#32660 update header info and improve mobile scaling#159
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Applicant Portal UI header to show applicant-specific identifiers (ref ID/name) and improves mobile layout scaling, while extending workspace/org data through the frontend state and backend plugin demo/unity responses.
Changes:
- Add
applicantRefIdandapplicantNameto workspace state and propagate them from organization responses. - Move org/applicant header display into the global header (remove page-local headers) and adjust styling for responsive/mobile layouts.
- Improve UX around workspace selection return navigation and prevent datatable dropdown clipping via wrapper overflow adjustments.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/shared/models/workspace.interface.ts | Extends WorkspaceState with applicant ref ID/name fields. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/shared/components/org-header/org-header.component.ts | Adds applicant/org display mode and computed display getters. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/shared/components/org-header/org-header.component.scss | Refines org header layout/typography and adds mobile scaling. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/shared/components/org-header/org-header.component.html | Updates template to render labeled ID/name fields based on display mode. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/shared/components/datatable/datatable.component.ts | Adjusts dropdown handling to allow overflow visibility to avoid clipping. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/layout.component.ts | Adds mobile header height measurement via ResizeObserver and wires notifications dropdown. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/layout.component.scss | Tweaks scrolling container behavior and styles mobile header actions/notification bell. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/layout.component.html | Adds template ref for mobile header and groups mobile notification/user actions. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/header.component.ts | Injects/workspace-subscribes org/applicant header data and imports OrgHeaderComponent. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/header.component.scss | Adjusts header spacing and responsive behavior for smaller viewports. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/layout/components/header.component.html | Renders the org/applicant header inside the global header. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/features/workspace/workspace-selector.component.ts | Adds returnUrl query-param support and navigates back after selection. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/features/payments/payments.component.ts | Removes page-level org header usage (now global). |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/features/payments/payments.component.html | Removes page-level org header markup (now global). |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/features/applicant-info/applicant-info.component.ts | Removes page-level org header usage; tracks applicantRefId in component state. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/features/applicant-info/applicant-info.component.html | Removes page-level org header markup (now global). |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/core/services/workspace.service.ts | Populates applicant ref ID/name into the global workspace state from org responses. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Frontend/src/app/core/guards/auth.guard.ts | Passes returnUrl to workspace selector to return users to their original destination. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Backend/src/Grants.ApplicantPortal.API.Plugins/Unity/Unity.Organizations.cs | Preserves applicant ref ID/name when patching cached org profile data. |
| applications/Grants.ApplicantPortal/src/Grants.ApplicantPortal.Backend/src/Grants.ApplicantPortal.API.Plugins/Demo/Data/OrganizationsData.cs | Adds applicant ref ID/name to demo org data and outputs them in generated payloads. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
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
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.



No description provided.