Skip to content

feature/AB#28856 - Reorder Additional Contact section in Applicant Info#2185

Merged
JamesPasta merged 2 commits into
devfrom
feature/AB#28856-group-contact-info
Mar 27, 2026
Merged

feature/AB#28856 - Reorder Additional Contact section in Applicant Info#2185
JamesPasta merged 2 commits into
devfrom
feature/AB#28856-group-contact-info

Conversation

@plavoie-BC
Copy link
Copy Markdown
Contributor

This pull request refactors how the "Additional Contact" section is rendered in the applicant information view and updates related styling and markup for consistency.

Applicant Info View Refactor:

  • Moved the "Additional Contact" zone (ApplicationContactsWidget) to render within the main zone section for better structure and consistency, and removed its previous rendering after the form. [1] [2]

Styling and Markup Consistency:

  • Updated the .summary-container CSS class to be included in the border styling for summary sections, ensuring consistent appearance across contact and address sections.
  • Adjusted the markup in ApplicationContactsWidget/Default.cshtml to wrap the section in a .summary-container div and improved heading spacing for visual consistency. [1] [2]

@plavoie-BC plavoie-BC requested a review from JamesPasta March 27, 2026 21:03
@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 565
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 565
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@JamesPasta JamesPasta merged commit b25760f into dev Mar 27, 2026
22 checks passed
@JamesPasta JamesPasta deleted the feature/AB#28856-group-contact-info branch March 27, 2026 23:25
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