Skip to content

fix: datepicker selected state - #3225

Merged
balco0110 merged 17 commits into
mainfrom
fix/datepicker-selected-state
Aug 19, 2026
Merged

fix: datepicker selected state#3225
balco0110 merged 17 commits into
mainfrom
fix/datepicker-selected-state

Conversation

@balco0110

@balco0110balco0110 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Description:

Closes#2847

Definition of Reviewable:

  • Documentation is created/updated
  • E2E tests (features, a11y, bug fixes) are created/updated
  • Stories (features, a11y) are created/updated
  • relevant tickets are linked

@changeset-bot

changeset-botBot commented Aug 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0559790

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
NameType
@solid-design-system/componentsPatch
@solid-design-system/mcpPatch
@solid-design-system/stylesPatch
@solid-design-system/tokensPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@balco0110balco0110 moved this from 📋 Backlog to 👀 In review in Solid Design System Project BoardAug 5, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚀 Storybook has been deployed for branch fix_datepicker-selected-state

@coraliefeil

Copy link
Copy Markdown
Contributor

Approved

@coraliefeilcoraliefeil removed their assignment Aug 10, 2026
@rahulsuvarna18

Copy link
Copy Markdown
Contributor

I noticed that the selected date which is not in the same month has some different styling when selected.
image
Looking at Figma, the styling does not match any of the given styles. I believe the sale should be the same for the selected date (Blue bg and white text)
image

@rahulsuvarna18rahulsuvarna18 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check comment above

Comment threadpackages/components/src/components/datepicker/datepicker.ts
Comment thread.changeset/quick-snails-raise.md Outdated
Comment thread.changeset/quick-snails-raise.md Outdated
Comment thread.changeset/quick-snails-raise.md Outdated
Comment thread.changeset/quick-snails-raise.md Outdated
Comment threadpackages/components/src/components/datepicker/datepicker.test.ts Outdated
Comment threadpackages/components/src/components/datepicker/datepicker.test.ts Outdated
Comment threadpackages/components/src/components/datepicker/datepicker.ts
Co-authored-by: Aurora Vasconcelos <aurora.vasconcelos@virtual-identity.com>
balco0110and others added 2 commits August 19, 2026 15:37
Co-authored-by: Aurora Vasconcelos <aurora.vasconcelos@virtual-identity.com>
Co-authored-by: Aurora Vasconcelos <aurora.vasconcelos@virtual-identity.com>
Co-authored-by: Aurora Vasconcelos <aurora.vasconcelos@virtual-identity.com>
martascorreia
martascorreia previously approved these changes Aug 19, 2026
@balco0110
balco0110 merged commit 8d1a601 into mainAug 19, 2026
18 of 20 checks passed
@balco0110
balco0110 deleted the fix/datepicker-selected-state branch August 19, 2026 15:19
@github-project-automationgithub-project-automationBot moved this from 👀 In review to ✅ Done in Solid Design System Project BoardAug 19, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

fix: datepicker selected state

7 participants

@balco0110@coraliefeil@rahulsuvarna18@martascorreia@auroraVasconcelos@mariohamann@yoezlem