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

[Feature] Align toggle styles with style guide #336

Merged
merged 9 commits into from
May 26, 2020

Conversation

aappoalander
Copy link
Contributor

Description

This PR aligns Toggle component styles with style guide. Focus styles have now rounded corners and only focus on toggle button / input instead of label also. Positioning of the toggle knob has been slightly adjusted. A stronger shadow is also present.

Due to the changes, the component width changes slightly.

How Has This Been Tested?

Tested with Create react app Typescript project and styleguidist build.

@aappoalander aappoalander added the enhancement New feature or request label May 25, 2020
@aappoalander aappoalander added this to the 1.0.0 milestone May 25, 2020
@aappoalander aappoalander requested review from ketsappi and LJKaski May 25, 2020 08:31
@aappoalander aappoalander self-assigned this May 25, 2020
@netlify
Copy link

netlify bot commented May 25, 2020

Deploy preview for gallant-mahavira-de9f7b ready!

Built with commit f069977

https://deploy-preview-336--gallant-mahavira-de9f7b.netlify.app

@aappoalander aappoalander force-pushed the feature/align-toggle-styles-with-styleguide branch from dea8f46 to 2d39f9f Compare May 25, 2020 08:48
@aappoalander aappoalander changed the title Feature/align toggle styles with styleguide [Feature] Align toggle styles with style guide May 25, 2020
ketsappi
ketsappi previously approved these changes May 25, 2020
@aappoalander aappoalander force-pushed the feature/align-toggle-styles-with-styleguide branch from d4029c4 to 50677d5 Compare May 25, 2020 18:53
@aappoalander aappoalander requested review from LJKaski and ketsappi May 25, 2020 19:15
@aappoalander
Copy link
Contributor Author

Tested with Safari, Chrome, Brave and Firefox and all seem to work in a consistent manner now.

@aappoalander aappoalander merged commit e9bb7c0 into develop May 26, 2020
@aappoalander aappoalander mentioned this pull request May 26, 2020
@aappoalander aappoalander deleted the feature/align-toggle-styles-with-styleguide branch December 4, 2020 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants