@Nada-El-shater
Submitted
All feedback is welcome
Looking to hire developers?
@Kirin02
@Nada-El-shater
Submitted
All feedback is welcome
@Kirin02
Posted
The Link to your live page doesn't work, but on a quick scan on your code, i noticed you put font-weight: 900; but you only imported 400 and 700 font-weights, other than that i recommend you check other sizing units other than px like (em, rem..)
Marked as helpful
@patricia-hurst
Submitted
I'd love feedback as I am still learning (especially JavaScript)!
@Kirin02
Posted
Hey, i'd say try to make the submit button hover animation a bit more interesting i would say, the position of the image is kinda odd i'd say espcially since it got sharp edges i would say move it to the extreme right or limit the height or something if you use inspect tool you'll find that it overflowing from the <body> Also It highly unrecommended to Use !important other than that i like the mobile display it looks really good but i think you can improve the desktop one a bit more
@Brotli
Submitted
I would like some tips on how to make pages to adapt with mobiles too, I welcome any comments
@Kirin02
Posted
i would recommend that you study more flexbox so that you avoid positionning you elements using top and right and float with specfic use of px which made ur H1 and Description collapse into each other and media queries to make your website responsive on most devices
Marked as helpful
@ClariceAlmeida
Submitted
Feedbacks are important to me, please feel free to give me any tips.
@Kirin02
Posted
Good Job! I would say avoid using width and height properties too much espcially with px, your website is responsive but i think on the long way when you code other websites it might create some troubles for you.
Marked as helpful
@erelropeta
Submitted
I want to hear your feedback in what I can improve on on this challenge :)
@Kirin02
Posted
Good Job!
@EnasHamdy
Submitted
This is my first design using CSS flexbox. Would you please give me any feedback about my code. Thanks in advance.
@Kirin02
Posted
First thing first, About your HTML you should Check semantic HTML and apply it, also it better if you could link the font on your CSS Using @import and you spammed many properties many times like font-family, font-size, font-weight, flex.... also the card isn't properly centered and using flex in body is a bad move i would say
Marked as helpful
@Feuzi
Submitted
Hey, this is my first Frontend Mentor challenge. I could've made the code a bit cleaner, but I think I did fine. If u have some feedback regarding CSS or anything else, any kind of feedback is appreciated.
@Kirin02
Posted
I Think Your code is pretty clean, but i would suggest adding some hover transition unto the button or the anchor tag, Good job!
@Shelcy
Submitted
Hi Frontend-mentor community 👋!
I solved this challenge using only css, if you have a recommendation or any comments, I will be grateful.
See you, best wishes🌼.
@Kirin02
Posted
Fantastic work claps
I feel like i could have found a less hacky way to make this page responsive. If you have any idea how, feel free to share !
@Kirin02
Posted
I don't think it was "hacky" i feel like the code just looks pro but i feel it could be much more simplified, also you have some useless properties if you inspect with your browser you'll find some of them who aren't doing anything like " margin: 0 auto" ...
Marked as helpful