Skip to content
This repository was archived by the owner on Jun 9, 2025. It is now read-only.

Conversation

@rfc2822
Copy link
Member

@rfc2822 rfc2822 commented Oct 29, 2024

  • update(): correctly reset color (implemented in buildEvent)
  • buildEvent: set both color value and index to null if there's no event color (previously: only index, so value was not reset on update)
  • populateEvent: set Event color from value (if there's a matching CSS3 color) even if no color name is set. We need test for detecting whether a color value is still present in the tests.

@rfc2822 rfc2822 added the bug Something isn't working label Oct 29, 2024
@rfc2822 rfc2822 self-assigned this Oct 29, 2024
@rfc2822 rfc2822 linked an issue Oct 29, 2024 that may be closed by this pull request
@rfc2822 rfc2822 marked this pull request as ready for review October 29, 2024 19:11
@rfc2822 rfc2822 requested a review from sunkup October 29, 2024 19:13
Copy link
Member

@sunkup sunkup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good 👍

@rfc2822 rfc2822 merged commit 59d2728 into main Oct 30, 2024
@rfc2822 rfc2822 deleted the 178-event-color-is-not-reset-when-event-is-updated-without-color-1 branch October 30, 2024 12:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Event color is not reset when event is updated without COLOR

2 participants