@wakandakingz
Latest solutions
Latest comments
- @kartardeveloper
Hey Farhan, You did Great. If you need some improvements I will help you with this. Here are some suggestions -- 1). First of all what catches my eye is your card shadow. Great, you can try different from the challenge. But I say you use but use lighter shadow now its too hard. You can take shadow codes from there https://getcssscan.com/css-box-shadow-examples 2). Next, You can increase your heading size and make the description text a little bit lighter as in the reference. 3). Always wrap your images in a figure HTML element. Add image widths there and use height, and width 100% on your images. And object-fit property if you want to use it. Object fit CSS property is very helpful when er dealing with images. This is the same as we add background size to our background images. If you want more on object-fit and object position then you can comment to me I will definitely tell you. 4). Last but not least as @MelvinAguilar said. Use semantic tags in your code. It will help you and other developers to see and find something in your code easily. Everything else is looking good. I hope this will help you. If you find this comment helpful for you, then you can mark this as a helpful comment. It will increase my confidence. 😊 And If you need any help then I'm always there.👍
Marked as helpful - @mohamedshawgi@kartardeveloper
Hey Mohamed Shawgi, You did Great. But as you said you need more improvements to achieve the design you want. And if you need some improvements I will help you with this. Here are some suggestions -- 1). First of all you can use max-width and width CSS properties to give as much width as you want and if you use max-width it will also help in making our webpage responsive. 2). Next, if you want to center your entire card, then you can use display: flex; align-items: center; justify-content: center; properties on your card's parent element. That is the one way. Another way is a little bit complicated if you want to know that you can comment to me, and I will definitely answer it and help you in improving your skills. 3). You can use the cursor: pointer; on your buttons so that the user can easily catch it. This will increase the overall user experience on your webpage. And you can use the same font family on your button so that it looks beautiful as our other content looks. 4). You can use the grid on your main container instead of flex because we are creating a layout there. And grid helps you in creating the most appealing layouts. But if you want to use flex then you can it's not the big issue. But in flex you need to add width in its child elements, in the grid, we don't need to do it. 4). Last but not least spacing in your cards. You can decrease this by using simple padding properly on your cards. Everything else is looking good. I hope this will help you. If you find this comment helpful for you, then you can mark this as a helpful comment. It will increase my confidence. 😊 And If you need any help then I'm always there.👍
Marked as helpful - @beshoyyatef@kartardeveloper
Hey Beshoy, You did Great. But if you need some improvements I will help you with this. Here are some suggestions -- 1). You can use the line-height CSS property to give a space between text lines. Now the gap is too narrow. Now the text is looking a little bit ugly and messed up. After giving the gap you can see the changes. 2). Responsive is good. But you can increase your card width by a little in your mobile design. 3). You can use max-width and width properties on your card so that it will automatically be responsive on mobile. Everything else is looking good. I hope this will help you. If you need any help then I'm always there.👍
- @TheAli711@kartardeveloper
Hey Ali, You did a Great. But if you need some improvements I will help you with this. Here are some suggestions -- 1). You can increase your overall font size. Now it looks too small and hard to read. I hope this will help you. If you need any help then I'm always there.👍
- @dragouwu@kartardeveloper
Hey Drago, You did a great job. I really appreciate your solution to this challenge. The card is good, Responsive is good. Everything looks good.👌
Marked as helpful - @vishnuvk15@kartardeveloper
Hey Vishnu, You did a Great. But if you need some improvements I will help you with this. Here are some suggestions -- 1). Don't use width and height on your card as you did. Use max-width: 700px; and width: 100%; and use min-height instead of height. 2). You can use the letter-spacing CSS property to add space between letters in px. I hope this will help you. If you need any help then I'm always there.👍
Marked as helpful
