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 type3 switch component hover effect #289

Closed

Conversation

kena0ki
Copy link

@kena0ki kena0ki commented Nov 29, 2022

Brief description

Fixed box-shadow and rotate when the switch component is checked and hovered.

...

Developer Certificate of Origin

Sample pictures

Before
https://user-images.githubusercontent.com/29003390/204531980-581d4284-e2f6-44ce-b24c-dbd9fe358b18.mov

After
https://user-images.githubusercontent.com/29003390/204532452-c1cecb35-6f4d-43f0-87be-b015951dda22.mov

@Oli8
Copy link
Contributor

Oli8 commented May 31, 2023

@kena0ki Could you rebase/fix conflicts?

@kena0ki kena0ki force-pushed the feature-type3-switch-hover-effect-fix branch from 3bed424 to 5ba4c7d Compare June 1, 2023 08:23
@kena0ki
Copy link
Author

kena0ki commented Jun 1, 2023

@Oli8 Thank you for the review! I fixed conflicts.

@Oli8 Oli8 force-pushed the feature-type3-switch-hover-effect-fix branch from 5ba4c7d to 1f2367f Compare June 1, 2023 12:50
@Oli8 Oli8 closed this Jun 1, 2023
@Oli8 Oli8 force-pushed the feature-type3-switch-hover-effect-fix branch from 1f2367f to 05094e2 Compare June 1, 2023 12:51
@Oli8
Copy link
Contributor

Oli8 commented Jun 1, 2023

@kena0ki Sorry I messed up trying to remove the unnecessary commits 😬 , could you reopen the PR using the changes you should have locally ?

@kena0ki
Copy link
Author

kena0ki commented Jun 1, 2023

Sorry for inconvenience🥲

I rebased the branch after I merged it once. I think that's the reason why the unnecessary commits were showed up on GitHub. (I thought It's not a problem for the source code though)

@kena0ki
Copy link
Author

kena0ki commented Jun 1, 2023

@Oli8 I created a new PR #302.

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.

None yet

2 participants