Skip to content

mobile toggle position plus other minor edits #5

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 1 commit into
base: main
Choose a base branch
from

Conversation

Nix7amcm
Copy link

@Nix7amcm Nix7amcm commented May 9, 2023

-toggle positioning for mobile with border-top line
-- header media query flex-direction column
-- minor change to breakpoints-down small to cover missing 639px

-toggle focus edited
-- slight height adjustment
-- toggle focus color variables added

-toggle label color
-toggle button position minor adjustment
-slight margin-left to centre Followers more
-h1 h2 margin-bottom increased
-h2 color

-toggle position for mobile with line
-header media query flex-direction column
-minor change to breakpoints-down small to cover px 639 jump
-toggle label color
-toggle focus state and height slight adjustment
-toggle focus color variables added
-toggle button position slight adjustment
-slight margin-left to centre Followers
-h1 h2 margin-bottom increase
-h2 color
@Nix7amcm
Copy link
Author

Nix7amcm commented May 9, 2023

Thanks so much for your fab Youtube course on this! You're so easy to follow and explain everything so well. I really learned a lot! At the end, I decided to try fix the toggle position for mobile size, and came across a few other minor edits along the way. So thought I'd share in case you wanted to add them! Can't wait to check out more of your vids 😃

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.

1 participant