Skip to content

enh: allow Incubator.WheelPicker to take custom properties for the Fader #1947

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

Merged
merged 1 commit into from
May 8, 2022

Conversation

Asgaroth
Copy link
Contributor

@Asgaroth Asgaroth commented Apr 6, 2022

Description

Incubator.WheelPicker did not allow changing the fader tint. Which makes it hard to style for Dark mode

Screen Shot 2022-04-04 at 8 11 41 PM

This PR allows users to pass in faderProps to the Incubator.WheelPicker and forward them to the <Fader /> component.

Changelog

  • Allows to pass in faderProps to Incubator.WheelPicker and forward them to the <Fader /> component.

@Asgaroth Asgaroth force-pushed the wheelpicker-tint branch 2 times, most recently from 9f0a3cf to 8a7114d Compare April 8, 2022 00:45
@Asgaroth Asgaroth requested a review from ethanshar April 8, 2022 00:45
@Asgaroth Asgaroth force-pushed the wheelpicker-tint branch from 8a7114d to b785591 Compare May 5, 2022 04:12
@Asgaroth Asgaroth changed the title fix: allow Incubator.WheelPicker to change tint for fader enh: allow Incubator.WheelPicker to take custom properties for the Fader May 5, 2022
@Asgaroth Asgaroth requested a review from ethanshar May 6, 2022 14:36
@ethanshar ethanshar merged commit 7d5d6f0 into wix:master May 8, 2022
@Asgaroth Asgaroth deleted the wheelpicker-tint branch May 9, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants