Yana Antonna• 20
@yanaantonna
Submitted
This is my first project on Frontend Mentor. Could you please provide me some feedback on the work I have done. Tkank you.
Aayush Sood
@soodaayush
All comments
- Aayush Sood• 1,175
@soodaayush
Posted
Maybe fix the accessibility and HTML issues. Your font on the buttons does not look right compared to the design photo.
That's all I have to say.
Happy Coding!
0 - Abhik• 4,840
@abhik-b
Submitted
Hi Frontend Mentors 👋,
I created some staggering animations using
setTimeouts , however I personally feel there's a better a way for that. So if anyone knows that please help me out!Any feedbacks,tips,suggestions will be highly appreciated 🙏
Aayush Sood• 1,175@soodaayush
Posted
You might want to add media queries for the input and button because on bigger screens, the input and button are starched.
Otherwise you did great!
Happy coding!
Marked as helpful
1 - Robert Freck• 40
@UofMBob
Submitted
Aayush Sood• 1,175@soodaayush
Posted
I can't preview your website so fix that and the HTML and accessibility issues.
Let me know when you fixed the problem!
0 @JonathanCierp
Submitted
Please let me know if there is something wrong or if we can do it in a better way 😀
Aayush Sood• 1,175@soodaayush
Posted
Your website is very good! I do suggest fixing the HTML issues.
Happy Coding!
1- Mykola Boychuk• 110
@m-boychuk
Submitted
Aayush Sood• 1,175@soodaayush
Posted
Your website is good but there are some things that I suggest you improve on.
-
You forgot to add colors on some of the buttons. They weren't supposed to be the color when hovering over it. They were supposed to be that color for display
-
On bigger screens, the image below the header will fall out of it's box
-
The logo on the footer isn't the same compared to the design. What you should do is copy the original logo and make another one for the footer. When you Look in the svg file, there should be multiple instances with the word fill. In that box, your supposed to make the color white.
-
The copyright at the footer should be below all of the content
That's all I have to say!
Happy Coding!
0 -
- Stephen Buley• 75
@StephenBuley
Submitted
Mostly responsive, I know this isn't a huge page.
Can't for the life of me figure out the two background images to look like the design, other than that just general tips and fixes to help make my code more readable/efficient! Thanks!
Aayush Sood• 1,175@soodaayush
Posted
The way to position those 2 background images were to say background-position: top right, bottom left;
Happy Coding!
0 - Solomon• 130
@SoloLere
Submitted
Your feedback and suggestions are highly appreciated!
Aayush Sood• 1,175@soodaayush
Posted
Your website is very good but I suggest fixing the accessibility and HTML issues.
That's all I have to say!
Happy Coding!
0 - Delconer• 10
@Delconer
Submitted
Aayush Sood• 1,175@soodaayush
Posted
Your website is very good! I see no issues at all!
1 - mohamed maghraby• 220
@mohamed1maghraby-div
Submitted
please leave your feedback
Aayush Sood• 1,175@soodaayush
Posted
Your website is great! There is some minor issues that I suggest fixing.
-
The "Verified Graduate" font size is a bit bigger than the design
-
The paragraphs need to have some opacity
That's all I have to say!
Happy Coding!
1 -
- Debbyyy• 80
@Debbyyy
Submitted
Feedback is required. Ways to improve is also very important. Thank you!
Aayush Sood• 1,175@soodaayush
Posted
there are some problems when viewing this on bigger screens. This is coming from someone who uses a 4K monitor.
-
Some images are very big
-
the background image at the very top doesn't take up the whole width. The way to fix this is background-size: contain;
-
Some of the gaps are way too much. I suggest toning it down.
-
Some widths of the images are kind of short
Otherwise your website is good!
Happy Coding!
0 -
- Karim• 810
@Karimsamir112
Submitted
hey , Is this good ?? and what is the mistakes I did ?
I care so much about feedback because it let me improving myself
Thanks in advanceز
Aayush Sood• 1,175@soodaayush
Posted
your website is decent but there are some things I suggest you improve on.
-
Fix all of the HTML and Accessibility issues. If you press view report, you will see all the issues in your code.
-
You should probably use the pictures that they have given you in the files.
-
The boxes have barely any spacing so I suggest adding padding to those boxes.
-
The alignment of the boxes is kind of broken so I suggest making all heights of the boxes equal.
-
There are also some weird line breaks in all of the paragraphs so I suggest adding more spaces to the boxes.
-
Your content isn't centered so I suggest using CSS Flexbox to center the content.
That's all I have to say!
Happy Coding!
0 -
- Crespo Maxime• 75
@macrespo42
Submitted
I'm a total newbie in front end dev, so if you have any suggestions to get a cleaner html/css I will be very happy 😍 thanks guys 😉
Aayush Sood• 1,175@soodaayush
Posted
Your website is great! I suggest fixing your HTML issues.
Happy Coding!
1 - soodmax• 20
@soodmax
Submitted
Feedback requested...
Aayush Sood• 1,175@soodaayush
Posted
Great work @Soodmax however, there is some things that I suggest you improve or add.
-
Your font size is big compared to the design so tone it down a little.
-
The arrow buttons are in a different place than they should be. I suggest increasing the width of that box. And you also forgot to add shadows to the button.
-
I suggest fixing the accessibility and HTML issues.
That's all I have to say!
Happy Coding!
0 -
- sravan• 40
@Irobot-07
Submitted
Any kind of suggestion or feedback is appreciated. Thank you
Aayush Sood• 1,175@soodaayush
Posted
Good job on the website! I suggest fixing the accessibility and HTML issues.
That's all I have to say!
Happy Coding!
0 - José Eduardo• 65
@joseeduardorp
Submitted
I couldn't keep the bottom circles fixed. Any tips?
Aayush Sood• 1,175@soodaayush
Posted
Your website is very good! There are a few things that you can improve on.
- The shadows around the box are a bit too strong. I suggest toning it down.
That really all I have to say.
Happy Coding!
1 - Kenard Ellis• 255
@Nard00
Submitted
Advice would be appreciated.
Aayush Sood• 1,175@soodaayush
Posted
Your website is very good! I suggest fixing the 2 HTML issues that Frontend Mentor detected.
Happy Coding!
1 - Abdul• 285
@abdulrahman-rwim
Submitted
Any feedback and suggestions on how I can improve are very welcome!
Aayush Sood• 1,175@soodaayush
Posted
Your website is good! But there are things you can improve on:
-
As grace-snow said, the text is darker than the design
-
The font size of the people's names is a bit too large. Same with the header beneath that.
-
I noticed that there were no shadows around the boxes.
That's all I have to say!
Happy Coding!
0 -
- Eladio Feijoó• 200
@thotluna
Submitted
I could never match the size of the paragraphs
Aayush Sood• 1,175@soodaayush
Posted
If I recall, whenever you start a challenge, you install a folder which you have to extract. In that folder there is a file called style-guide.md. In there it should tell you what font size your supposed to be using. Your website is great otherwise!
0 - Kenneth Laidlaw• 80
@Kenlaidlaw
Submitted
Really not sure if using fixed position on round images is the correct option but it seems to work.
Aayush Sood• 1,175@soodaayush
Posted
Using fixed position on round images is the correct option.
0 - Abhik• 4,840
@abhik-b
Submitted
I have used SortableJS for making drag n drop available for mobile devices Please provide a feedback on how I should improve this 🙏
If you are new to frontend mentor and you are struggling with this challenge then I have also created youtube video while creating this solution which can be used as a reference:https://youtu.be/lZE0Qydl4ko
* This video shows pure javascript way for drag n drop and also array sorting after drag n drop
Aayush Sood• 1,175@soodaayush
Posted
Good job on that page! The only things I'd recommend you do is to decrease the width of the todo list, Make the borders a bit thinner and that's all I have to say.
1
