-
Notifications
You must be signed in to change notification settings - Fork 13.4k
feat(datetime): add wheel part to ion-picker-column #30934
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
base: feature-8.8
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
brandyscarney
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Some optional changes requested. 🚀
| .custom-grid-wheel::part(wheel):focus, | ||
| ion-picker-column:focus { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a comment that the 1st selector applies to the month/year picker and the wheel style datetime and the 2nd selector applies to the time picker?
| }); | ||
|
|
||
| test.describe(title('CSS shadow parts'), () => { | ||
| test('should be able to customize wheel part', async ({ page }, testInfo) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional: should we break this up into the different ways where you can customize the wheel part?
- Month/year picker
- Wheel style (
prefer-wheel) picker - Time picker
Issue number: resolves #30420
What is the current behavior?
It is not possible to style the
ion-picker-columnwithinion-datetime. This leads to developers unable to style it for any states like focus.What is the new behavior?
wheelpart toion-picker-columnwhen it's withinion-datetime.Does this introduce a breaking change?
Other information
Preview