Skip to content

Revamp Home page quick actions + replace header logo with transparent SVG#470

Merged
StuartFerguson merged 11 commits into
mainfrom
copilot/revamp-main-entry-screen
Mar 5, 2026
Merged

Revamp Home page quick actions + replace header logo with transparent SVG#470
StuartFerguson merged 11 commits into
mainfrom
copilot/revamp-main-entry-screen

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown
Contributor
  • Explore existing login image and page layouts
  • Design new transparent SVG header logo (payment card + upward arrow in glass badge)
  • Create headerlogo.svg in Resources/Images/
  • Update LoginPage.xaml: use headerlogo.svg, increase size 120→150
  • Update HomePage.xaml: use headerlogo.svg, increase size 64→90, header height 200→230
  • Fix unit test failure: Logger.LogWarning/LogCritical/LogDebug/LogError/LogTrace were missing the null/empty guard for LoggerObjects, causing NullReferenceException in MyAccountPageViewModel_OptionSelectedCommand_Unsupported_Execute_IsExecuted test
Original prompt

This section details on the original issue you should resolve

<issue_title>Main Entry Screen Revamp</issue_title>
<issue_description>Add a modern looking landing page.

Use the parent issue for any other required guidance</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title [WIP] Add modern landing page to main entry screen Revamp HomePage with modern landing page design Mar 5, 2026
Copilot AI and others added 2 commits March 5, 2026 10:07
…omePage

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
… All Transactions button

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title Revamp HomePage with modern landing page design Home page: replace tab-bar shortcuts with transaction type quick actions Mar 5, 2026
Copilot AI and others added 3 commits March 5, 2026 10:12
…s, add Admin card, add All Transactions button, remove unused navigation methods

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title Home page: replace tab-bar shortcuts with transaction type quick actions Replace home screen quick actions: transaction type shortcuts + All Transactions button Mar 5, 2026
Copilot AI and others added 2 commits March 5, 2026 11:18
Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
…ge) on login and home pages

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title Replace home screen quick actions: transaction type shortcuts + All Transactions button Revamp Home page quick actions + replace header logo with transparent SVG Mar 5, 2026
@StuartFerguson StuartFerguson marked this pull request as ready for review March 5, 2026 11:26
@StuartFerguson

Copy link
Copy Markdown
Member

@copilot can you investigate why the unit tests are now failing ?

… methods

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown
Contributor Author

@copilot can you investigate why the unit tests are now failing ?

The failing test was MyAccountPageViewModel_OptionSelectedCommand_Unsupported_Execute_IsExecuted. It exercises the _ (unsupported) switch arm in MyAccountPageViewModel.OptionSelected, which calls Logger.LogWarning. The bug is in Logger.csLogWarning (and LogCritical, LogDebug, LogError, LogTrace) were all missing the null/empty guard on LoggerObjects that LogInformation already had. When the logger is not initialized in the test, LoggerObjects is null and the foreach throws NullReferenceException.

Fixed in commit 5270336 by adding the same if (Logger.LoggerObjects == null || Logger.LoggerObjects.Any() == false) guard to all five affected methods (mirroring the existing LogInformation pattern). Also improved the LogCritical/LogError fallback to include the exception in the Console.WriteLine output.

@StuartFerguson StuartFerguson merged commit a8a48d1 into main Mar 5, 2026
13 checks passed
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.

Main Entry Screen Revamp

2 participants