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

[BUG] - DatePicker styles #2901

Closed
theonlyway opened this issue Apr 28, 2024 · 2 comments · Fixed by #2908
Closed

[BUG] - DatePicker styles #2901

theonlyway opened this issue Apr 28, 2024 · 2 comments · Fixed by #2908
Assignees
Labels
📦 Scope : Components Related to the components 🐛 Type: Bug Something isn't working

Comments

@theonlyway
Copy link

theonlyway commented Apr 28, 2024

NextUI Version

2.3.6

Describe the bug

I'm not sure if this is intended or not but the default styles for the DatePicker have blue selections and hovers matching all of the other NextUI styling but when you set the variant to bordered or underlined it changes all the default styling to be monochrome. The behavior can be seen on the documentation pages where the default variant has normal styling and the bordered and underlined variants don't.

Your Example Website or App

No response

Steps to Reproduce the Bug or Issue

  1. Create a DatePicker using the default example
          <DatePicker
            label="Birth date"
            className="max-w-[284px]"
            showMonthAndYearPickers
          />

and the styles will match the example
2. Add variant="bordered" and the Calendar styles will no longer be using the same styles as the default variant

Expected behavior

I would expect the default styles to still apply to the variants unless they are being overridden either via the slots or by tailwind config.

Screenshots or Videos

No response

Operating System Version

Windows

Browser

Edge

Copy link

linear bot commented Apr 28, 2024

@wingkwong
Copy link
Member

supplement a bit with screenshots.

default (flat, faded):

image

monochrome (bordered, underlined):

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 Scope : Components Related to the components 🐛 Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants