Skip to content

[GH-1543] Switch scss changed #1544

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jenyaftw
Copy link
Contributor

Summary

Changed height of outer switch and transition of inner switch

Ticket Link

Issue #1543

@jenyaftw jenyaftw requested a review from a team as a code owner October 13, 2021 13:00
@mattermod
Copy link
Contributor

Hello @jenyaftw,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@chenilim chenilim requested review from asaadmahmood and removed request for chenilim October 13, 2021 18:06
@chenilim
Copy link
Contributor

Thanks @jenyaftw! This looks good to me, but @asaadmahmood, could you do a quick review? Not sure how this was broken in the first place, or if we need to refactor the switch css later.

@asaadmahmood
Copy link
Contributor

@jenyaftw Tried those changes on the plugin, and those don't look good. Can you post screenshots for how this looks on the plugin and on the local server?

@jespino
Copy link
Contributor

jespino commented Oct 21, 2021

/update-branch

@jespino
Copy link
Contributor

jespino commented Oct 21, 2021

/update-branch

@mattermod
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

/cc @jasonblais @jfrerich @emilyacook

Copy link
Contributor

@jespino jespino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the code perspective, looks good to me. Thanks @jenyaftw! 🎊

Copy link
Contributor

@asaadmahmood asaadmahmood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not work on plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants