Project completed, i had no files to see how it should look for mobile.. Thank you
Julian Ifesiokwu
@JulianIfesiokwuAll comments
- @JulianIfesiokwuSubmitted almost 3 years ago
- @JulianIfesiokwuSubmitted almost 3 years ago@JulianIfesiokwuPosted almost 3 years ago
It will take you to the error page but click home to see it works ok
0 - @JulianIfesiokwuSubmitted almost 3 years ago
- @JeuriMorelSubmitted about 3 years ago
Hello Frontend Mentors. I don't have a specific question, but if you see something that I could improve on, please don't hesitate to give criticism. This was my first GURU challenge and it took a lot out of me. Hopefully it came out okay.
- @JulianIfesiokwuSubmitted about 3 years ago@JulianIfesiokwuPosted about 3 years ago
The only problem I had was with background SVG. I cant seem to get it to work with two SVG elements. Any help is welcome. Thanks again.
0 - @JulianIfesiokwuSubmitted over 3 years ago
Please see my result here. The mobile view starts from 320px and the desktop starts from 741px. I could have done for tablets but I realized that's extra work because the requirements are just for desktop and mobile, I could do it but no need. Please any other feedback is greatly welcome. I am really happy with the JS. All complete.
@JulianIfesiokwuPosted over 3 years agoThanks. Silly me. Will do that. Your feedback was so so helpful
0 - @KunalK10Submitted over 3 years ago
Some feedback will help.
@JulianIfesiokwuPosted over 3 years agoHello well done, I just looked at your design, there shouldn't be a calculate button, what i did was to make a function that reacts to input on the number of people and it calculates the amounts to be paid. Hope this helps. The function works on input. Check here https://www.w3schools.com/js/js_events_examples.asp
Marked as helpful0 - @exist08Submitted over 3 years ago
feedbacks please hope it is close to the challenge.
@JulianIfesiokwuPosted over 3 years agoPlease check I think you left out the mobile view. Good try though
0 - @chrisbailey85Submitted over 3 years ago
I quite enjoyed doing this project, I've tried to make it as accessible as possible any tips for improvement will be greatly appreciated.
@JulianIfesiokwuPosted over 3 years agoSolid effort, i think you missed using the right fonts for the client job description. How did you get the underline completed on the "LEARN MORE"? Also how did you get the logo to be green in the footer? Solid effort though.
1 - @agkaysterSubmitted over 3 years ago
My questions are:
-
Please how do I get JavaScript to work on opening the navbar-burger while using small devices?
-
How do I make those underlines under "Learn more", are those underlines or text shadows?
@JulianIfesiokwuPosted over 3 years agoHey bossman very good effort. You missed a few things though like the fonts to be used, the background color and the width of certain sections? For the width you can put both section in a div and use flexbox with a flex of 50% for each, worked for me. Other than the few i mentioned, solid effort.
0 -
- @JohndiddlesSubmitted over 3 years ago
Hello people!!! Please check out my solution to this challenge. This actually took me more time than I anticipated. Desktop view works fine for now so also mobile view (375px). Still working on other screen sizes. For the drop down menu, I need a way to stop the neighboring links from dancing around when a drop down menu is triggered. If you find anything I still need to work on, please feel free to roast me 😁.
One love 😘
@JulianIfesiokwuPosted over 3 years agoVery well done John. Awesome job. I think the only think you missed was the padding on the buttons on the landing section, its perfect. Also thank you for the feed back, I will try your method.
0 - @JulianIfesiokwuSubmitted over 3 years ago
I completed the project with all major requirements, but I the left side of my input boxes seem to have some borders disappearing, please any feedback on this will be helpful. Thank you for checking out my solution.
@JulianIfesiokwuPosted over 3 years agoThanks you so much. Really sorry for the late reply.
0