@moeen-mahmud
Latest solutions
Latest comments
- @ellienndea
Hi Moeen, this looks really good despite some minor issues, I'd say, and the html and css is clearly written as far I can judge it being a newbie myself. ;) I did that challenge not long ago and got helpful feedback, which I want to pass on to you (some of it). :)
- The buttons in "normal" state should have a transparent border, so they dont move, when you hover on them.
- Giving the cards a max-width would prevent, that they stretch out too much.
Happy coding! 😊
Marked as helpful - @jing0429@ellienndea
Hi Jing, this looks pretty good. 😊 One tip that I got, is to not use px but em and rem instead. On my screen the card is not properly centered, instead of using position: absolute; top:...; left:... you could make the body a flex-container width height: 100vh(viewport height) and use justify-content: center and align-items: center. The mask is a little too light. For your first project it looks pretty good! 😊 Happy coding!
- @Moh2106@ellienndea
This looks good so far. 😊 For a better horizontal centering and a vertical centering, you could make the body a flex-container with a min-height of 100vh(viewport height) and justify-content: center and align-items: center. Happy coding! 😀
- @AgataLiberska@ellienndea
Hi Agata, this calculator app is really great. On my laptop it works perfectly fine with mouse and keyboard. :) I really like your focus on accessibility, it's so important! Happy coding 😊 Linda
- @calvin17@ellienndea
Hey Calvin, that is well done. It looks preety much like the original and in your index.html I really like the class names you chose! 😊 Cheers, Linda
