Skip to content
OverviewSolutions
8
Comments
12
Olalekan Ogundele
@whoiskekeanyway

All comments

  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Why did you preselect 15% and choose 1 for the number of people? when the user enters 0, the app should say, "Can't be Zero

    0
  • salem moqa•240
    @moqasalemSubmitted almost 3 years ago

    The project was very easy , but I am still looking for a good solution for an image problem. the problem is how to change the position of a image column with a text column like the "egg" was on the right of "transform" text on desktop screen ,but on mobile screen it become above the text ! I think maybe I can do it with positions property but it will make the design a little complex.

    please if you know a good trick for this problem share it .

    thank you !!

    simple sunnyside agency landing page

    1
    Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Hi Salem, excellent job For the coloured lines under learn more, you can check out CSS pseudo-class to do that, I added a link for you to read up on it https://www.w3schools.com/css/css_pseudo_elements.asp.

    Your image size needs to be corrected and also your website size, the max-width was meant to be 1440px for desktop

    • Center your clients testimonials title on mobile view
    • center learn more on mobile view
    0
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Excellent Job.

    0
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    This is perfect, and You did an excellent job. You need to adjust the size of the fonts to match the design. Add border radius to the container also. Instead of being justified to the left, your button is also at the centre. so I think making the button small should solve this remove the tiny space between the cards

    Marked as helpful
    0
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Hi For the colored lines under learn more, you can check out CSS pseudo-class to do that, I added a link for you to read up on it https://www.w3schools.com/css/css_pseudo_elements.asp.

    Your image size needs to be corrected and also your website size, i think max-width was meant to be 1440px

    Marked as helpful
    0
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    I like that you made your bill calculator update after every entry. When I enter 0, it doesn't say "Can`t be Zero " also the billing input needs to be a placeholder and not hard-coded 0.

    Marked as helpful
    1
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    You have done a very good job, I think you should continue with challenges and start with junior so you can learn as you build. I am a big advocate of learning by doing

    Marked as helpful
    0
  • Tereza•605
    @sirriahSubmitted almost 3 years ago

    This is my first time with ReactJS :). I am still at the beginning. What confused me a little is the "rule" how to split the design into the components, but I think that it will improve with next challenges.

    I have tried to rebuild the input radio with the list.

    Looking forward to you feedback. Happy coding! 👋👩‍🦰

    Tip Calculator - ReactJS, CSS Module

    2
    Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Also, your initial amount should have December point like $0.00 and not just $0

    0
  • Tereza•605
    @sirriahSubmitted almost 3 years ago

    This is my first time with ReactJS :). I am still at the beginning. What confused me a little is the "rule" how to split the design into the components, but I think that it will improve with next challenges.

    I have tried to rebuild the input radio with the list.

    Looking forward to you feedback. Happy coding! 👋👩‍🦰

    Tip Calculator - ReactJS, CSS Module

    2
    Olalekan Ogundele•290
    @whoiskekeanywayPosted almost 3 years ago

    Very good job, It is very nice that you solved this with Reactjs. It doesn't say " cant be zero" when you input 0 into the number of person =

    0
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted about 3 years ago

    Hi, you did an excellent job here. I noticed a few things here. you didn't add the line under the learn more. read on CSS selectors to do that especially after and: before. here is a link to help you

    https://www.w3schools.com/cssref/css_selectors.asp

    Also, you forgot to add the arrow down icon on your landing page on top of the orange

    1
  • Olalekan Ogundele•290
    @whoiskekeanywayPosted about 3 years ago

    Very good work here. there is a bug with the total amount per person. it displays the total amount of the bill not for each person, also center your main content on the laptop view.

    Marked as helpful
    0
  • jssol•120
    @jssolSubmitted about 3 years ago

    Sup guys! Here's my mobile version. I've struggled to implement media queries for screen sizes of 1440px. If you've got some tips for me concerning media queries, let me know. Thanks in advance.

    Sunnyside agency landing page

    1
    Olalekan Ogundele•290
    @whoiskekeanywayPosted about 3 years ago

    To make things easier for you. Wrap all your elements inside the main tag and then set max-width on main in your CSS. that is a good start but remember when you use max-width, you have to add margin: 0 auto to center the elements.

    Do not hesitate to ask if you have other questions

    Marked as helpful
    0