Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the branding from "Ministry of Transportation and Infrastructure" (MoTI) to "Ministry of Transportation and Transit" (MoTT) across the application, and adjusts header navigation colors for different environments.
Changes:
- Updated organizational name from "MoTI" to "MoTT" in UI text, documentation, and project metadata
- Modified navigation header colors, replacing theme-color reference with hard-coded value
- Added environment-specific navigation color classes for test and prod environments
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| client/src/scss/_nav.scss | Updated default navigation color to hard-coded value, added test and prod environment color classes |
| client/src/js/components/fragments/Header.js | Changed application title from "MoTI" to "MoTT" |
| client/src/js/components/forms/saltreport/AddSaltReportFormFields.js | Updated table header text from "MoTI Owned?" to "MoTT Owned?" |
| api/Hmcr.Model/EmailBody.cs | Updated ministry name in email body |
| api/Hmcr.Hangfire/Hmcr.Hangfire.csproj | Updated Authors and Company metadata |
| api/Hmcr.Api/Hmcr.Api.csproj | Updated Authors and Company metadata |
| README.md | Updated ministry name in dependencies section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| .main-nav { | ||
| @include main-nav-colors(theme-color(secondary-nav)); | ||
| @include main-nav-colors(#053662); |
There was a problem hiding this comment.
The hard-coded color value #053662 is used here and also duplicated in the .main-nav.prod rule (line 63). Consider extracting this as a SCSS variable (e.g., $prod-nav-color) or using a theme-color to maintain consistency and make future color changes easier.
No description provided.