Skip to content
OverviewSolutions
29
Comments
19
P
Smaylen5
@Smailen5

All comments

  • Nate Valline•420
    @nvallineSubmitted about 2 months ago
    What are you most proud of, and what would you do differently next time?

    This was a good challenge to focus on developing a contact form and some of the finer details that go into creating an accessible form. I am proud about completing the challenge based on the requirements provided.

    What challenges did you encounter, and how did you overcome them?

    Creating the toast animations took a little longer than it should have, but I did learn about a new method to animate from display: none.

    What specific areas of your project would you like help with?

    I would appreciate any tips/suggestions as to improve my code. Thanks!

    HTML, CSS, & TS

    1
    Smaylen5•530
    @Smailen5Posted about 2 months ago

    Great solution 😉

    But I noticed that the validation only happens when the form is sent, if I correct the input field I don't have immediate feedback if it is correct

    Marked as helpful
    0
  • Smaylen5•530
    @Smailen5Posted about 2 months ago

    Great solution 😃

    0
  • Smaylen5•530
    @Smailen5Posted about 2 months ago

    Great solution 😉

    0
  • Smaylen5•530
    @Smailen5Posted about 2 months ago

    Great solution 👍

    Marked as helpful
    0
  • Jan Kotvaš•290
    @DrakeHermitSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    To be completely honest I am not that proud of finishing this challenge. I feel like there must be an easier way to get this to look the way that the design is set up. But I am happy that I managed to get it to look at least similar to the design.

    What challenges did you encounter, and how did you overcome them?

    The main challenge was to get the desktop part of the design look the way it should. And to be honest I am not entirely proud of how I went about making it look the way it needed to be. I created a grid of 3 columns and 3 rows wide, and I just placed the cards the way they were supposed to be placed, and I finalized it by adding transform: translateY() to the middle cards to move them to the center. It worked but I feel like there is an easier, or more efficient way.

    What specific areas of your project would you like help with?

    I just want someone to look at my CSS and tell me if my solution to this challenge is decent.

    Four Card Feature Section

    1
    Smaylen5•530
    @Smailen5Posted 7 months ago

    Excellent solution, congratulations your css code seems well written, you made use of variables and pseudo selectors such as ::before, I liked that for the media query you used rem and not px. Is there any part of your code that gives you doubts? and still a good code 😉

    1
  • Rosanraj•100
    @RosanrajSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    not anything specific, need to improve my skill in responsive design to all screen size.

    What challenges did you encounter, and how did you overcome them?

    -I faced some issues in adjusting the content page to right side of the image and then, I used float left to move right the page content.

    What specific areas of your project would you like help with?

    -how to design a responsive design page for any screen size.

    Product preview card

    1
    Smaylen5•530
    @Smailen5Posted 7 months ago

    Unfortunately, the solution is not displayed correctly. This is because the index.html file is not located in the root directory, so GitHub Pages uses the readme file as the home page.

    Marked as helpful
    0
  • kaoutar ouadih•800
    @kaoutar-ouadihSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    I'm proud of completing this challenge! and learning new things from it.

    What challenges did you encounter, and how did you overcome them?

    I struggled with making thing work together and making the code more dynamic.

    What specific areas of your project would you like help with?

    Anything that can help me improve.

    Password generator app

    1
    Smaylen5•530
    @Smailen5Posted 7 months ago

    Great solution, well done, very similar to the original design.

    0
  • Smaylen5•530
    @Smailen5Posted 7 months ago

    Great solution, the script works well.

    Unfortunately, there are some display issues:

    • The width of the white container is fixed, it would be better if it were not and adapted to the content, this would prevent content overflow.
    • The tip buttons would be better if they were 6rem instead of 7rem, at 1440px they overlap otherwise.
    • You could give the reset button a width of 100%, so it takes the correct size from the parent, it would also be better to remove the margin-left.

    Hope I've been helpful :)

    0
  • kaoutar ouadih•800
    @kaoutar-ouadihSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    I'm proud of completing this challenge!

    What challenges did you encounter, and how did you overcome them?

    I struggled with displaying elements dynamically with javaScript without affecting my css layout.

    What specific areas of your project would you like help with?

    Anything that can help me improve.

    Time-tracking-dashboard

    1
    Smaylen5•530
    @Smailen5Posted 7 months ago

    Great, excellent solution 😊, everything looks perfect both on mobile and desktop. My only suggestion, if you want to match the requested design completely, you could increase the height and width of the container a bit.

    0
  • Smaylen5•530
    @Smailen5Posted 7 months ago

    Great solution, the design is very similar for smartphones and wide desktops. Note that between tablets and desktops <1440px, the design breaks. You would also need to add color on hover.

    1
  • Smaylen5•530
    @Smailen5Posted 8 months ago

    Great, the solution is similar to the request. You might have forgotten background-size: cover; for the image, so you should center the image better as in the requested layout.

    0
  • Smaylen5•530
    @Smailen5Posted 8 months ago

    Great solution, the layout is very similar to what was requested. It seems that in desktop mode, you can add a bit more space above the logo to align it even better with the layout.

    0