@Hoda96
Submitted
@azick99
@Hoda96
Submitted
@azick99
Posted
Well-done Hoda! I see you did a lot and I like your css code.
However, I got some advice to improve your code and user-experience.
width: 1160px;
it can look like well in your PC but user will have different screens that developer must consider. Try to find better alternative like width: 90vw;
. Bonus for user-experience make border of image avatar: instead of .user-profile{ height: 3.8rem; cursor: pointer; } .user-profile:hover{ border: 2px solid #f98001; border-radius: 50px; }
make .user-profile{ height: 3.8rem; cursor: pointer; border: 2px solid #fff; border-radius: 50px; }
then hover: {border-color:#f98001}
.z-index
of popup higher and dropdown lower.I hope it would be useful. Keep coding!!!
Marked as helpful
@fsp3012
Submitted
I'm unable to show the cart count, if you know a simple way to do it, please comment below
@azick99
Posted
Well-done Fakorede, I like you project!
Here is some advice:
1.In your case it's not hard to make cart count pass count to Navbar through props and create ternary count === 0 ? " " : <span className='cart-count'> {count}</span> and style with position: absolute.
2.To make setCount(count - 1) is not correct because it goes minus like -1, -2 and so on, in order to prevent it use condition in onClick like: if (count > 0) { setCount(count - 1); }.
3.Pay attention to the styles try to make them better as in challenge.
I hope it will be useful for you. Keep coding!!!
Marked as helpful
@murilonicemento
Submitted
Este é um projeto de encurtamento de URL desenvolvido com React. Ele permite que o usuário insira uma URL longa e obtenha uma URL encurtada como resultado.
@azick99
Posted
Hi Murilo! I like your project and I see you did a lot.
However, there are some problems:
https://api.shrtco.de/v2/shorten?url=${longURL}/very/long/link.html
but it should be https://api.shrtco.de/v2/shorten?url=${longURL} . You see after requesting, it shows Error page because of /very/long/link.html.Keep coding!!!
Marked as helpful
@Gaurang-gupta
Submitted
@azick99
Posted
Hi! I really like your project and I see you did a lot. If you don't mind I will give you some feedbacks to improve your web app.
Work with responsiveness in smaller screens images and words overlapping. So, it looks like not good, I think you got larger screen in your PC but in mine it has bad layouts.
Input Link my advice instead link.length === 0 , it's better to create new useState and make it status state like changing status when you click button setStatus('empty') if input empty and setStatus('typing') when you typing. To better understand you can checkout my solution I made it in Features Section.
When you refresh the page it does not save the changes like links that is posted. But in the challenge it is required, to make it save you can use local storage. Try to find more information about local storage better choice ask for it in chatGPT.
I hope it would be useful for you!
Marked as helpful
@enn-ko
Submitted
This project I create with HTML, CSS and JS. I used framework with Bootstrap, Fontawesome and Jquery. Thank you for your Time and welcome all feedback.
@azick99
Posted
Hi Ennko ! I like your job and I see you worked on this project a lot. However, I could not find your code repository it does not work, to say exact something.
But any way here some advice from my point of view:
Any way keep going!
Marked as helpful
@natanaelrusli
Submitted
What I find difficult? Passing props between various button, field and label components especially with typescript
Which areas of my code that I am unsure of?
Please give feedback about my code regarding the best practices that I should follow
@azick99
Posted
Hi there! I really like it and see you did good job!! keep on, Here some advice to improve your code from my point of view.
I think It would be easier, when you make main functionality in app.tsx from there you can pass props easier to components.
handling buttons. First make button like container with children prop and create 5 buttons with different percent pass event. you can check my solution I did like I said it is easier.
I think there is problem with custom input and total. total must be tip amount value that you have overall but tip amount must divide total to number of people like this total / numPeople.
I hope you understand me if no you can check my solution although I'm not sure I did it perfectly but it was easy for me
Marked as helpful
@AhmadYousif89
Submitted
Hi there 👋
This challenge was really awesome 👍 so I thought why not make it a little bit bigger so I decided to add some adjustment and features and I hope you guys like it.
For now I just built the home, product and checkout pages but I might continue developing this site in the future and add more features and pages, and maybe one day will take it to a full-stack level 🚀.
Let me know if you guys like the project I really appreciate your comments and your feedback.
Keep Coding 💻
@azick99
Posted
Hey there! I really like it and I see you worked on this project a lot !
Let me give some feedbacks that you can improve further,
I think you about mobile version, it would be better to make.
adding to cart, It would be better to add for each product, but in your cart only one addition.