Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 9, 2025

Removes the calendar color picker to simplify theming and reduce configuration complexity. The app now uses a fixed default color (#FF9FDF) until theme synchronization is implemented in #7.

Changes:

  • Removed palette button from calendar toolbar
  • Deleted ColorPickerDialog.razor component
  • Removed OpenColorPicker() method from CalendarPage.razor
  • Removed ChangePalette test
  • Removed PreferenceService.PrimaryColor property and hardcoded the color in GenerateMoodColors()

<issue_title>Remove Calendar color picker</issue_title>
><issue_description>It's hard to guarantee a good-looking theme with the current picker because there are too many choices. Simplify the app now by removing the color picker and using the existing default color (the dark primary), then later we will sync it with the theme in #7.</issue_description>
>
> ## Comments on the Issue (you are @copilot in this section)
>
>
>
>

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove Calendar color picker</issue_title>
<issue_description>It's hard to guarantee a good-looking theme with the current picker because there are too many choices. Simplify the app now by removing the color picker and using the existing default color (the dark primary), then later we will sync it with the theme in #7.</issue_description>

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


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove calendar color picker for simplification Remove calendar color picker Nov 9, 2025
Copilot AI requested a review from danielchalmers November 9, 2025 03:32
@danielchalmers
Copy link
Owner

@copilot Clean up by removing PreferenceService.PrimaryColor too since we only want "#FF9FDF".

Use hardcoded #FF9FDF color directly in GenerateMoodColors since the app now only uses a fixed color.

Co-authored-by: danielchalmers <7112040+danielchalmers@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 9, 2025

@copilot Clean up by removing PreferenceService.PrimaryColor too since we only want "#FF9FDF".

Done in ed69cdd. Removed the PrimaryColor property and updated GenerateMoodColors() to use the hardcoded color #FF9FDF directly.

@danielchalmers danielchalmers marked this pull request as ready for review November 9, 2025 04:27
@danielchalmers danielchalmers merged commit 89668a7 into main Nov 9, 2025
2 checks passed
@danielchalmers danielchalmers deleted the copilot/remove-calendar-color-picker branch November 9, 2025 04:28
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.

Remove Calendar color picker

2 participants