Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Kirin 105

    @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

    0
  • Kirin 105

    @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

    0
  • Brotli 90

    @Brotli

    Submitted

    I would like some tips on how to make pages to adapt with mobiles too, I welcome any comments

    Kirin 105

    @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

    0
  • Kirin 105

    @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

    1
  • Kirin 105

    @Kirin02

    Posted

    Good Job!

    0
  • @EnasHamdy

    Submitted

    This is my first design using CSS flexbox. Would you please give me any feedback about my code. Thanks in advance.

    Kirin 105

    @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

    0
  • Feuzi 20

    @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.

    Kirin 105

    @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!

    0
  • @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🌼.

    Kirin 105

    @Kirin02

    Posted

    Fantastic work claps

    1
  • medo 25

    @medohb

    Submitted

    When Im trying to move to phone mode in my laptop the picture looks weird , any idea ?

    Kirin 105

    @Kirin02

    Posted

    Hello, You did a mistake in the border-top-left-radius i believe you wanted to target border-bottom-right-radius !

    Marked as helpful

    0
  • Kirin 105

    @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

    1