@hello-sabira
Submitted
What are some bad coding practices do you see, that I should improve on??
@eewa-SANJ
@hello-sabira
Submitted
What are some bad coding practices do you see, that I should improve on??
@eewa-SANJ
Posted
Hello!. Your solution looks fantastic and looks exactly like the design. Here are fix issues I found
That's it. I think, my findings will help you to improve your coding.
Happy coding😊
Marked as helpful
@malek-bt
Submitted
Hello, thank you for coming and looking at my code!
This is my solution for this challenge, I hope it's up to the standards, but if it's not, please feel free to tell me! Thank you in advance.
@eewa-SANJ
Posted
Hello!. Your solution looks fantastic and I have gone through your code. There are a few issues found in your codes. Here are my findings.
I think, my findings will help you to improve your coding.
Happy coding😊
Marked as helpful
@diogomwbdv
Submitted
First project on Frontend Mentor!
@eewa-SANJ
Posted
Hello!. Your solution looks fantastic and I have gone through your code. There are a few issues found in your codes. Here are my findings.
I think, my findings will help you to improve your coding.
Happy coding😊
@okraks
Submitted
Built this Responsive News Homepage using CSS Flexbox
@eewa-SANJ
Posted
Hello!. Your solution looks good but I have gone through your code. There are a few issues found in your codes. Here are my findings.
Your site is not responsive, so it crashes and overlap here and there. You can fix those issues using media queries. If you want to help me more, contact me and I will help you
Your favicon is not rendered due to the wrong path. Could you fix it?
Your main image(image web 3 desktop) only rendered in small screens. But on small screens, your mobile version of that image should be rendered. <img srcset="assets/images/image-web-3-mobile.jpg 686w, assets/images/image-web-3-desktop.jpg 1460w" sizes="(max-width: 375px) 686px 1460px" src="assets/images/image-web-3-desktop.jpg" alt="Web 3.0" >
. If you want how to implement this Read this
There is an alignment issue in your sidebar and below three column divs. You can adjust it by resizing the sidebar according to the three-column width.
You have missed the mobile hamburger menu in the tab and mobile screen sizes
I think, my findings will help you to improve your coding.
Happy coding😊
Marked as helpful
@hkalita20
Submitted
Give review on this and find out the errors......
@eewa-SANJ
Posted
Hello!. Your solution looks fantastic and I have gone through your code. There are a few issues found in your codes. Here are my findings.
I think, my findings will help you to improve your coding.
Happy coding😊
@eewa-SANJ
Submitted
I completed the News Home Page challenge. I have a little bit of confusion while coding the mobile hamburger menu.
*Do you have another approach to coding the hamburger menu?
I invite you to review my code and give me feedback. I appreciate it.
Thank you
@eewa-SANJ
Posted
Thank you for the suggestions, I have to fix the scrolling issue, and want to figure out how to fix it
@Barissev
Submitted
@eewa-SANJ
Posted
Hello! Your solution is good and looks nice but there are a few issues in your solution. Here are my findings.
These issues and suggestions might help you to improve your coding.
Happy coding😊
@Omeshcoding
Submitted
Had fun building this project. Love to get some feedback.
@eewa-SANJ
Posted
Hello! Your solution is pretty nice. Here are some of my issue findings and suggestions.
Overall your solution is amazing and clean. I think above mention suggestions will help you
Happy coding😊
Marked as helpful
@amd42
Submitted
@eewa-SANJ
Posted
Hello!
Your solution is nice and clean but there are some issues. Here is my findings,
I think you will receive some help from the above explanation.
Happy coding😊
Marked as helpful
@AshrafUzzaman04
Submitted
@eewa-SANJ
Posted
Hello, Your solution is quality and I have one suggestion for you which is, to use a rem unit for your design instead of px
Happy coding😊
Marked as helpful
@MohamedAtTop
Submitted
If you have any comment please put it.😎😎😎
@eewa-SANJ
Posted
Hi, I have found some issues regarding your solution. Here are my findings
There is a shadow all around the "Register" button, which is a lot different from the design. It is not an issue but a suggestion to fix it to be suitable for the design
In the social icon section, there is no space between each item, try to give some margin for each item suitable way
Also, try lowering line height to give a decent looking for the main heading
In mobile screens, there are a lot of spacing issues. For example, there is a lot of pace between the Logo and the main image.
I think these suggestions, help you to improve your coding Happy coding😊
@JoseEliasMorales
Submitted
@eewa-SANJ
Posted
Amazing work! Here is my suggestion to make some improvements
I think this will help to improve some issues Happy coding😊
Marked as helpful