Ivรกn Garcรญaโข 185
@IvanGH4
Submitted
I wasn't sure on how to make the form validation, so any advice will be really appreciated! Thank you!
El Ghali Khalid
@limtedtorni000
All comments
- El Ghali Khalidโข 2,630
@limtedtorni000
Posted
Hi Ivan i like your solution it's responsive and looking great.
2 - Mijail Hernandezโข 950
@MishaHernandez
Submitted
This challenge has taken me a little longer than usual but I think it has helped me reinforce some knowledge. I appreciate any suggestions ๐
Bonus:
- Animations
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Mijail you did very well it's responsive and looking great.
2 - iivanov-2โข 80
@iivanov-2
Submitted
Any feedback is welcome.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Iivanov you did very well it's responsive and looking great you just need to change the background color of
third-containerto make it look exactly like the provider design.3 - Hikmahโข 1,090
@Hikmahx
Submitted
Any feedback would be appreciated. Thanks
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Hikmahx you did very well it's responsive and looking great.
3 - Neklesโข 10
@BWezka
Submitted
Did I do it right?
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Nekles, yeah you did right your solution look great in desktop you should also work on it more to make it responsive and look great in phones.
2 - Yuniacโข 65
@Yuniac
Submitted
Any feedback/criticism is welcome, also I'm not sure whether I had to make it max-width of 1440px or not so I went with a max-width of 1440px as the guide indicated, but now I think it won't look perfect on bigger than that screens.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Yuniac your solution its looking great to make it look better in screens bigger than 1440px i suggest to change your margin in
htm, bodyfrom0to0 auto.2 - Pharmackโข 320
@Pharm-ack
Submitted
I would love you guys response on this
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Pharmack, your solution its look great in screen size of (1440x800), i suggest to use PX or REMs for the width and use
grid-template-columns: repeat(4, 250px); justify-content: center;rather thangrid-template-columns: repeat(4, 1fr);.2 - Keven Molinaโข 60
@Kmolina009
Submitted
Hello, this is my first challenge. I have the elements placed in the right position for the desktop requirement, but I'm having issues with the mobile media-query. Will be refactoring in the meantime.
Any feedback is very much appreciated, Thank You.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Kevin i have nothing to say Milton Chung already said the important thing overall you great and keep coding
3 - Sarahโข 835
@sarahc-dev
Submitted
Would welcome any feedback :)
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Very good Sarah is 100% identical to provider design
4 - Samyr Oliveira Ribeiroโข 215
@SamyrOR
Submitted
Any feedback will be welcome!
In love doing these challenges!
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Samyr your solution look great and responsive for the stars icon i suggest using width: 2rem rather using percentage. keep contributing these awesome solutions
3 - Florin Pavelโข 240
@florinpavel22
Submitted
This looked simpler than it actually was. Gave me a few headaches but in the end I am happy with the result.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Florin your solution very good Its look great and very responsive.
3 - Nicoleโข 360
@nicole-tuznik
Submitted
Any feedback on how I could improve my code is always appreciated! Thanks!
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Nicole your solution very good Its look great and very responsive keep contributing these awesome solutions.
2 - Diana Ilanโข 40
@dianailan
Submitted
If you have any feedback on how can I make my code better feel free to leave a comment.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Diana your solution very good Its look great and very responsive keep contributing these awesome solutions
3 - MD Salahuddinโข 150
@mdsalahuddin2001
Submitted
Hey Gyz, I have completed fylo-dark-theme challenge . I am hunker after your valuable feedback. Please suggest me where I need to improve. Your feedback is like a gem to me.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi MD your solution look great and responsive, just one thing i suggest to add background-size: contain; for the bg-curvy it will make it look fantastic. Also, as your solution report suggests, it's considered a best practice to only use one
<h1>tag per page and use less important heading tags for the rest of the headings on the page (for both SEO and semantic reasons). Doing so will clear up your solution report and make more sense in the context of the importance of the different headings on the page.Happy Coding.
3 - Didinโข 50
@hafizhudin
Submitted
Hello, this is my first time submitting my project. Any feedback would be nice. Thank you.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
HI Didin firstly welcome to this amazing community of learners, and congratulations on submitting your first solution. Your solution seems great , keep contributing these awesome solutions
3 - GLLโข 105
@gllader
Submitted
Any feedback will welcome
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi GLL your solution seems fantastic just try to fix the mockups image for mobiles overall keep contributing these awesome solutions
3 - Willie Ryantamaโข 50
@otatame
Submitted
I'm still a bit rough at handling images for background, as you can see the pattern at the top is not responsive when you shrink it into mobile size, any tips would be appreciated Thank you
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Willie your solution its great its very very similar to the original design, i have nothing to say just keep contributing these awesome solutions
3 - Zuzkiโข 180
@LuisDGracia
Submitted
Is there anything else i could do better?
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Looks good Luis, there is just one problem when i press Enter to add a new TODO it dosent work try to fix it will make it cool and i suggest to change the cursor of the moon/sun icon from default to pointer. Overall you did great and keep contributing these awesome solutions
3 - N E I Tโข 390
@NiteArie
Submitted
I would love some feedbacks. I don't know if I used BEM principle correctly, correct me if I am not. Is there anything I could do to improve the website anymore? Really appreciate the feedback. Thanks
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Looks great for me NEIT, i like your animation if you just added also to cards it will be great.
3 - Janeโข 1,040
@janegca
Submitted
Before starting on the Junior projects decided to re-visit some earlier challenges and refactor the code in light of what I think I have learned. This is my first refactor.
Any and all suggestions welcome.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Great work Jane i have nothing to say just keep contributing these awesome solutions
2 - Achraf Elmouhibโข 460
@NightClover-code
Submitted
Any feedback on the code is welcome !
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Batal; tbarklah 3lk
3 - Alex Liโข 380
@awexli
Submitted
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Alex your solution it's look great and responsive for me , you just need to solve your html issue keep contributing these awesome solutions
3 - Peter Klinkโข 165
@Peterklink
Submitted
I used Grid for the Layout. I am still a little bit confused with the grid-rows. I feel like the way i did it is not good practice. If you have any recommendations please let me know! :)
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Nupur nyour solution it's look great and responsive for me keep contributing these awesome solutions
3 - Nupur J.โข 115
@DezineWings
Submitted
How to reduce code size? Please guide.
El Ghali Khalidโข 2,630@limtedtorni000
Posted
Hi Nupur nyour solution it's look great and responsive for me keep contributing these awesome solutions
3
