Refactor color parsing and improve FrameAdapter structure

[vb2ae]

This pull request includes changes to improve code readability, simplify logic, and address potential memory management concerns in the Caliburn.Micro.Platform project. The most significant updates include refactoring the ToColor method for better clarity, reorganizing event handler initialization in FrameAdapter, and removing redundant code.

Code readability and simplification:

• Refactored the ToColor method in AppManifestHelper.cs to simplify logic for determining the start position and default alpha value. Introduced a boolean variable isEightCharactersLong for clarity and reduced redundant code.

Event handler management:

• Reorganized event handler initialization in the FrameAdapter class. Added a new AddEventHandlers method to encapsulate the setup of navigation-related event handlers, improving code organization and addressing potential memory leaks.

Miscellaneous cleanup:

• Removed an unnecessary blank line in the FrameAdapter class to maintain consistent formatting.
• Corrected the formatting of the copyright comment in AppManifestHelper.cs.

[Copilot]

Pull Request Overview

This PR refactors color parsing in AppManifestHelper.cs and reorganizes event handler initialization in FrameAdapter.cs to improve readability and maintainability while addressing potential memory management concerns.

• Refactored the ToColor method by introducing a boolean check for an 8-character hex string and simplifying the parsing logic.
• Reorganized event handler initialization in FrameAdapter by encapsulating them in a new AddEventHandlers method and removing redundant initialization and comments.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/Caliburn.Micro.Platform/Platforms/uap/FrameAdapter.cs	Moved event handler registrations into a dedicated method and cleaned up redundant code and comments.
src/Caliburn.Micro.Platform/Platforms/uap/AppManifestHelper.cs	Refactored ToColor method to simplify color parsing logic and improve clarity in managing the alpha channel.

Comments suppressed due to low confidence (2)

src/Caliburn.Micro.Platform/Platforms/uap/FrameAdapter.cs:58

• [nitpick] Consider adding a brief comment inside the AddEventHandlers method to explain why the event handlers are grouped together, which could aid future maintainers.

        private void AddEventHandlers()

src/Caliburn.Micro.Platform/Platforms/uap/AppManifestHelper.cs:99

• [nitpick] Adding an inline comment to explain that the startPosition is set to skip the alpha channel (when present) may improve code clarity.

            int startPosition = isEightCharactersLong ? 2 : 0;