Skip to content
OverviewSolutions
16
Comments
13
Leroy-sama
@Leroy-sama

All comments

  • Leroy-sama•300
    @Leroy-samaPosted about 1 month ago

    Hello there. Form validation is missing from your solution. When I click the subscribe button the page reloads. Another thing, the color of the label for the input field is wrong.

    0
  • Leroy-sama•300
    @Leroy-samaPosted about 1 month ago

    Hello. Your live link is not showing anything, check your github pages to find the issue.

    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Zainabdullahali. Congrats on your first solution. On your solution you forgot to load the font-family from google fonts. Otherwise you are good

    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello there Nabid. Great work on the design and functionality of the app. Just one thing I noticed, the button should not be inside the the advise box, only halfway in. You can use position: relative; on the parent and position: absolute on the child (button) to move it half way in

    Marked as helpful
    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Mohsin. Your dice button is really big, i don't why because I'm not yet familiar with react. I'm sure its a small thing.

    1
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Mohd. Your solution looks good just that you forgot the hover effects of the numbers and the submit button. Otherwise, great work.

    0
  • Dharshini S•20
    @dharshini02Submitted over 1 year ago

    This is my first challenge and I struggle in moblie responsive .Please tell me the mistake that I have done.All the feedbacks are welcome .

    QR_Code

    1
    Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Dharshini. Congratulations on your first solution. Just a few things that maybe you can adjust. You can lower the font-size of the h1 element and give it spacing. The paragraph could also use some padding to give it breathing space (white space).

    Marked as helpful
    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Thank you for the reminder, Dessidy. I forgot to add the background for the image, which I will correct. Salute.

    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello FortuneEli, your design is working well but there are some few fixes you can make. The images are not displaying, you can fix that by changing their file path. There is very minimal spacing, increasing the container's padding will fix that.

    0
  • Elisabeth Erkekoglou•180
    @elic4vetSubmitted over 1 year ago

    Hello hackers, here is my solution to the project. What do you think ? Is the page full responsive to you ? Any other tips / suggestions ? Thanks in advance for your help, Kind regards, Elisabeth

    Results summary component

    2
    Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Elisabeth, your design looks good except for some small things. The reaction, memory, verbal and visual sections could use some adjustments. Make the icons a little bit smaller and try using display: flex and align-items: center to make the content centered in those sections. The width of the summary sections should be the same as that of the buttons.

    Marked as helpful
    0
  • Leroy-sama•300
    @Leroy-samaSubmitted over 1 year ago

    what's up guys, my solution has a small problem. I used align-items: stretch and the last button was offset. How can i fix this? I welcome your solutions to the problem.

    CSS flexbox was used in this project

    2
    Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Thanks a lot Aditya. I'll definately switch to semantic tags and learn more about rem units.

    0
  • Leroy-sama•300
    @Leroy-samaPosted over 1 year ago

    Hello Hugo. Your design looks good but one small thing, you forgot to put a hover effect on your submit button so that it changes the text color and the background color

    0