I struggled a lot with this, specially with the modal and its particular behavior. I'm pretty sure that if i were to do it again I would try to do it with a "custom" modal, a hidden screen or something like that. It seems an easy way to deal with the challenge
Latest solutions
Newsletter validation with Javascript
Submitted 5 months agoWith the JavaScript part. I would like feedback and suggestions for better solutions.
Responsive card component with on click hide and reveal JS feature
Submitted 6 months agoI would like some comments about how I could simplify the tooltip css or other methods than mine.
Responsive grid card layout
Submitted 6 months agoSemantic html choices. I was a bit puzzled and not sure if I made the right choices at the end.
Responsive 4 card feature
Submitted 6 months agoChoice of html tags and the choice of html structure.
A recipe page challenge to focus on writing semantic HTML.
Submitted 6 months agoI would like some feedback on the choice of semantic elements.
Latest comments
- @lazydroideSubmitted 5 months agoWhat challenges did you encounter, and how did you overcome them?@yiorgosbagakisPosted 5 months ago
Very accurate implementation of the design. In particular the main screen. Works very well in small screens as well. Error messages are always pretty following the design.
I find interesting the JS code as well and I will spend some time to study it since I don't understand some of the parts at the moment.
Excellent work!
0 - @MamdohSamerSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
using JavaScript in this project
What challenges did you encounter, and how did you overcome them?using JavaScript in this project
What specific areas of your project would you like help with?this page can be like an article or ads for any service
@yiorgosbagakisPosted 6 months agoNice tryhowever I see that the elements are a bit off. The picture the elements around the share button and the tooltip doesn't include the social media icons.
Interesting use of if statement on your JS code.
Marked as helpful0 - @ChikairoSubmitted 6 months ago@yiorgosbagakisPosted 6 months ago
Very nice work with clean and well structured html code.
Possibly adding a <picture> tag around the image avatars would help semanticly?
The only visual issue I see is that on some middle range screen widths the quotation mark icon is off the card. This should be adjusted.
Marked as helpful0 - @MckaaySubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
Learned bascics of making responsive layout using grid technique.
@yiorgosbagakisPosted 6 months agoGreat work and pretty accurate! Just need to add the orange border (at the moment you are using two cards with red border colour).
Marked as helpful0 - @jetobe95Submitted 6 months agoWhat are you most proud of, and what would you do differently next time?
- Usage of logic values on css, like max-inline-size, margin-block-end.
- Use of BEM
- @Ejiro-FrancesSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I am proud of how much I learnt in the course of this project.
What challenges did you encounter, and how did you overcome them?One of the challenges I had was using the font provided in the kit provided, I got the font from google font online since I was not sure why the one provided only gave me bold fonts.
Another challenge was trying to choose the ideal measurement for margins and paddings that would present the page as close to the designer's as much as possible.
What specific areas of your project would you like help with?I would like a review of what I have done and any pointers to make it better would be greatly appreciated.
@yiorgosbagakisPosted 6 months agoVery good work! It is very close to the original design and the only obvious issues I see are:
-
The headings look bolder than the original. Maybe you should try smaller 'font-weight' (500?)
-
The header image looks a bit larger than the original
Regarding the semantic html use I would add a header section and use more section elements.
Marked as helpful0 -