Any feedback are welcome...
Latest solutions
recipe-page-main
#accessibilitySubmitted about 1 year agoIf you have any idea to improve markup or the style I'd be thankful.
social-links-profile-main
#accessibilitySubmitted about 1 year agoIf you have something to say about CSS I'm listening.
Latest comments
- @LuisAngel96Submitted about 1 year agoWhat specific areas of your project would you like help with?@9dorghamPosted about 1 year ago
You've done awesome work.
- You didn't your design on mobile devices looks as it suppose to be does I'm talking here about the image.
- You used too much classes for your elements, what you don't need don't use don't just put classes everywhere for every element. You really did good effort on stylesheet. Have a nice day coding :)
Marked as helpful1 - @TemceoSubmitted about 1 year agoWhat are you most proud of, and what would you do differently next time?
I have sought to use semantic html
What challenges did you encounter, and how did you overcome them?The biggest challenge was sizing and placing the image centrally. I overcame this by placing the image inside a div which enable me to control the size of the image and to control the div as a flex element.
What specific areas of your project would you like help with?Any general points for improvement are welcome.
@9dorghamPosted about 1 year agoGood work everything looks good.
- You used the "<main>" element to contain your hole content and that's good, but it's a simple card you'd better use container elements like "<div>".
- Why you used this structure "<button class="btn github">GitHub</button>" it's an unorder list of links. We should use the correct elements for the correct purpose you could do this "<ul><li><a href="#">GitHub</a></li></ul>" .
- Use the "<strong>" element for important text like the address "London, United Kingdom" it will not just make it in bold it's important for search engines and assistive technologies like screen readers, you should read about it if you want to learn more. You've done awesome work, have a nice coding day :)
Marked as helpful0 - @Collins-KinumbiSubmitted about 1 year ago@9dorghamPosted about 1 year ago
You've done awesome work. It's better to look at the figma file if it's available for you. If want to make your solution pixel perfect for example put the value for "width" and "height" properties as the design. things like these will make your solution better. Finally, have a nice day coding :)
1 - @Sujatha-17Submitted about 1 year ago@9dorghamPosted about 1 year ago
Good work. You should do more effort on styling choosing the correct colors for correct elements, I think you should use static width instead of 'relative units' like percentage. You can check the 'style-guide' file and check your fonts-type and size. lastly about your markup:
- you have an unordered list of links you can use "ul" element. In HTML you just need to choose the correct element for the correct purpose and probably you will get some accessibility features it's called "semantic elements". Have a nice coding day :)
Marked as helpful0 - @chkhsSubmitted about 1 year agoWhat are you most proud of, and what would you do differently next time?
I'm proud of being able to complete this project without following along on youtube. Next time I'd do things differently by coding this project in vite framework; challenging myself more and be able to grow from it.
What challenges did you encounter, and how did you overcome them?Challenges I encountered was for CSS, width and height, padding. I had to do it just right otherwise it wouldnt even come close to looking like the original design.
What specific areas of your project would you like help with?None at the moment.
@9dorghamPosted about 1 year agoGood work. I have some suggestions for you:
- You embedded the font in your HTML file, so use it.
- Follow the style-guide to use colors in the correct place. Have a nice coding day :)
Marked as helpful1 - @TheMcnafahaSubmitted over 1 year ago