Skip to content
OverviewSolutions
8
Comments
4
Sourabh Singh
@6ixline

All comments

  • ApplePieGiraffe•30,545
    @ApplePieGiraffeSubmitted about 4 years ago

    Hey, everybody! 👋

    This was another fun challenge that I quite enjoyed! 😊

    I focused on trying to make my class names understandable and organized and I tried to increase the accessibility of the toggle by sandwiching it between two labels and adding the aria-pressed attribute to it (but IDK if that really helps or just makes things slightly more confusing)! 🙃

    I also added a cool little sliding animation to the left and right pricing cards in the desktop layout. 😎

    Feedback is welcome and appreciated!

    Happy coding! 😁

    Pricing Component With Toggle And CSS Animations

    2
    Sourabh Singh•165
    @6ixlinePosted about 4 years ago

    Hey there 👋,

    As always your solution is really great 👌.

    I just suggest one thing though:

    • You should add this .toggle-switch__btn{ outline: none; } on the toggle button for the outline that comes whenever it toggle.

    Hope it's helpful!! Happy coding 😄

    1
  • ApplePieGiraffe•30,545
    @ApplePieGiraffeSubmitted about 4 years ago

    Hey, everybody! 👋

    This was my first challenge in which I actually had to write more than a few lines of JS. 😆 My solution works (as far as I know) but IDK about my JS... I'd love some feedback, of course, to see how I've done. 😊

    I also added a subtle floating animation to the image just to keep things interesting!

    Happy coding! 😄

    EDIT: An issue I'm aware of is that there's this weird glitch where the background of the buttons on the page flickers or turns transparent when clicked on in Safari (because of a transform I added to their active state, I think). I added one or two properties that seem to take the flickering away, but I'm still unsure of how I can completely solve this glitch (other than, perhaps, getting rid of the transform completely).

    Simple Testimonials Slider With JS

    3
    Sourabh Singh•165
    @6ixlinePosted about 4 years ago

    Your solution is so perfect! Nice Job

    1
  • Sourabh Singh•165
    @6ixlinePosted about 4 years ago

    Hey Fernando, Your solution looks really great

    I only suggest one thing: - If you add max-width in main it will fix the problem of growing main.

    0