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
Yiorgos Bagakis
@yiorgosbagakisAll comments
- @lazydroideSubmitted 29 days agoWhat challenges did you encounter, and how did you overcome them?@yiorgosbagakisPosted 28 days 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 about 2 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 about 1 month 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 about 1 month ago@yiorgosbagakisPosted about 1 month 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 about 1 month agoWhat are you most proud of, and what would you do differently next time?
Learned bascics of making responsive layout using grid technique.
@yiorgosbagakisPosted about 1 month 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 about 1 month 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 about 1 month 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 about 1 month 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 -
- @YosefHayimSubmitted about 1 month ago@yiorgosbagakisPosted about 1 month ago
Very good work. I would try to match better the sizes of the elements to the original design. Eg the main container seems quite smaller and packed compared to the original.
0 - @TrianapetitSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
Un muy buen reto, para iniciar
What challenges did you encounter, and how did you overcome them?Los desafios comprender el modelo de caja, display: block, display: flex.
@yiorgosbagakisPosted about 1 month agoThis is a very good and accurate solution. Maybe some semantic HTML would improve it, because there are a lot of 'div' elements at the moment.
0 - @DavyZaneSubmitted about 1 month ago@yiorgosbagakisPosted about 1 month ago
-
Quite good solution but the card is smaller and in particular the fonts.. That makes them less legible.
-
There are too many divs used so some semantic elements could be used more.
-
Overall very good work the code is well structured and it works well in all screen sizes
0 -