Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ui5-special-date): add visual representation for Working/NonWorking types #9524

Merged
merged 3 commits into from
Aug 12, 2024

Conversation

hinzzx
Copy link
Contributor

@hinzzx hinzzx commented Jul 19, 2024

Previously, when adding a SpecialCalendarDate to the Calendar component and specifying the public types Working and NonWorking, these settings were not properly applied.

For example, setting a Working type to a weekend day did not visually reflect that the day was designated as a working day and vice versa for NonWorking type.

With this change we resolve that issue, ensuring that the specified types (Working and NonWorking) now take effect as intended.

Fixes: #9216

@unazko unazko self-requested a review July 22, 2024 07:12
Copy link
Contributor

@unazko unazko left a comment

Choose a reason for hiding this comment

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

The issue is fixed.

@hinzzx hinzzx merged commit 296373d into main Aug 12, 2024
10 checks passed
@hinzzx hinzzx deleted the fix-spec-cal-date-types branch August 12, 2024 06:44
hinzzx added a commit that referenced this pull request Aug 12, 2024
…ng types (#9524)

Previously, when adding a `SpecialCalendarDate` to the `Calendar` component and specifying the public types `Working` and `NonWorking`, these settings were not properly applied.

For example, setting a `Working` type to a weekend day did not visually reflect that the day was designated as a working day and vice versa for `NonWorking` type.

With this change we resolve that issue, ensuring that the specified types (`Working` and `NonWorking`) now take effect as intended.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SpecialCalendarDate]: not all types have a visual representation
3 participants