@MoamenAbdelrahman7
Submitted
@jamesbrown173
@MoamenAbdelrahman7
Submitted
@jamesbrown173
Posted
Looks great! No issues and functions well, also the fade in out looks good
Marked as helpful
@jamesbrown173
Posted
It's really good, everything updates well. It's clear and responsive.
@khalagai
Submitted
What are you most proud of, and what would you do differently next time?
What challenges did you encounter, and how did you overcome them?
What specific areas of your project would you like help with?
Any feedback on how I can further make the code simpler and more readable.
@jamesbrown173
Posted
Design:
Javascript code:
🤗 Great job and happy coding! 🤗
@robcrock
Submitted
What are you most proud of, and what would you do differently next time?
This challenge gave me a chance to use environment variable in NextJS, which I hadn't used before. It also gave me a chance to create a nice grid layout.
What challenges did you encounter, and how did you overcome them?
I hadn't used environment variable in NextJS before. So I was getting some errors while deploying to Vercel that I hadn't seen before, but now I know exactly how to handle those issues in the future.
What specific areas of your project would you like help with?
I am alway open to feedback on how I'm using Tailwind and NextJS. This specific project gave us a chance to create our own endpoint, so feedback on that is welcome as well.
@jamesbrown173
Posted
The grid overflows a little on the desktop view.
@steveostler
Submitted
What specific areas of your project would you like help with?
I did a little research to complete this challenge but it's mainly my work (about 85%). I had to get the code for an email validation script from stack overflow. This is the only thing I do not understand in the code. Do I need to understand regex? Is it ok to rely on javascript validation?
@jamesbrown173
Posted
Great, clean code easy to read with comments. Functions well with client side validation.
I'm not sure I used regex in mine but I don't understand it fully, I think it's a powerful tool for parsing data.
@lost50U1
Submitted
What are you most proud of, and what would you do differently next time?
It took me sometime even if i have some basic knowledge on javascript and it was challenging and i want to improve more on javascript and make the javascript code i added using less code even though it is challenging i finished the challenge.
What challenges did you encounter, and how did you overcome them?
if it was only making it show one of the share tooltip it was easy but making it show based on different screen is challenging and i solved it by making 3 functions and buttons.
What specific areas of your project would you like help with?
if someone can review both the tailwind and javascript code i used on the active state and tell me if there is a simple method to do it better i will appreciate it
@jamesbrown173
Posted
**First Impressions
It functions well at first glance, and the buttons work to show and hide the share element. However when transitioning between sm and md breakpoints the share element doesn't change. I think we need to add hidden class types for the different breakpoints in the JS code.
The site is responsive and functions well.
What are you most proud of, and what would you do differently next time?
I am most proud of my ability to make the project match the design requirements. Additionally, I built the page using dev tools to figure out what went wrong and fix it.
I don't know what I would do differently, but I welcome feedback to improve.
What challenges did you encounter, and how did you overcome them?
The footer!!! I tried grid and flexbox but had a hard time getting the three elements to line up and look good. After toying around, I ended up with two flex containers. It looks right, but I am sure it can be done better; feedback is welcome!
What specific areas of your project would you like help with?
Improving the footer. Suggestions on @media's; good/bad?
@jamesbrown173
Posted
** Design
Marked as helpful
What are you most proud of, and what would you do differently next time?
I'm proud that I aligned the cards correctly using grid. Next time, I'd like to change the default font size to be more similar to the layout.
What challenges did you encounter, and how did you overcome them?
The biggest challenge was creating a layout that didn't break on tablets. Unfortunately, I couldn't solve the issue; I think I need to set minimum sizes, but the text was overflowing from the cards.
What specific areas of your project would you like help with?
How to create the layout for tablets so that it doesn't break on desktop.
@jamesbrown173
Posted
**Design: Good! Clean easy to follow. Maybe consider using a lighter font in the lower paragraphs.
**Code: Well commented, easy to follow.
You mentioned the text overflowing, if you remove the lg:h-full class in this div it should stop the text from overflowing. I think when you're using lg:h-full it expands the div to the full amount of the parent, in this case the parent is restricted in size by it's padding, I think, therefore the text overflows.
<!-- Contenitore generale per grid --> <div class="lg:w-10/12 lg:h-full grid lg:grid-cols-4 lg:grid-rows-2 lg:gap-8 gap-6">Marked as helpful
@Shakil-a
Submitted
What are you most proud of, and what would you do differently next time?
most proud :
what i would do differently:
What challenges did you encounter, and how did you overcome them?
challenge and solution 1: how to code the design for the desktop version of the challenge solution: i learnt more and practised css grid on w3schools before attempting and also learn negative margin values
challenge and solution 2:
What specific areas of your project would you like help with?
i would like help on:
any feedback would be appreciated, thank you
@jamesbrown173
Posted
**Design: Maybe add a white color to the background of the cards to give contrast onto the background.
*Code: Consider using Tailwind CSS for writing inline styles, IMO this makes it easier to check your code. You mentioned about adding the shadow, this can be done easily through the shadow- property in tailwind.
@hadeedji
Submitted
@jamesbrown173
Posted
It's great, clean, easy to read.
Perhaps, on the mobile size you could have set a fixed width also as this would give you greater control of how the typography is displayed, rather than it being defined by the users screen size.
@idatenkiddy
Submitted
What are you most proud of, and what would you do differently next time?
I made it and i am proud. I used html tags that are not usually used 💀 in my opinion
What challenges did you encounter, and how did you overcome them?
nothing much.
What specific areas of your project would you like help with?
I used strong tag instead of bold property ,cause i wanted to try it.
@jamesbrown173
Posted
Clean sexy code, I love it! Super easy to read.
You used sections to mark different areas rather than just a div, I think this is semantically better. Why did you choose to do this?
The list spacing is not quite matching to the original image.
@jamesbrown173
Posted
It's great!