@toledolucassSubmitted 7 months ago
Fin Mardi
@Yakobus-MardiAll comments
- @Yakobus-MardiPosted 7 months ago
I recommend using more semantic HTML tags instead of divs. But overall, you are doing a great job.
0 - @DivasJaglanSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
Making this awesome project
What challenges did you encounter, and how did you overcome them?No challenges at all
What specific areas of your project would you like help with?Not in this
@Yakobus-MardiPosted 8 months agoWell done! Your solution is cool and you should continue doing what you're doing. Keep it up.
Marked as helpful1 - @dialomaSubmitted 9 months agoWhat specific areas of your project would you like help with?
I don't understand why when I use border radius on my images , it works fine on the top left and right corners, but on the bottom left and right corners it doesnt apply the same way.
@Yakobus-MardiPosted 9 months agoGood solution. I like the style of the image border-radius.
0 - @samdahbossSubmitted 9 months ago@Yakobus-MardiPosted 9 months ago
I think for the acknowledgment (Coded by), you have to put it outside the card, so the card of the QR code can be more compact. The title should be bolder and the font size a little bit too small. But overall, it's a good solution. Keep going.
1 - @herbrasSubmitted about 2 years ago
I started back my learning journey as web dev
@Yakobus-MardiPosted about 2 years agoHello, friend
These are what I think you can improve your code on the next project:
➡️ I think it's a good practice if you use more semantic HTML tags instead of div tags. ➡️ Write the code consistently. Ex: for the colors name. ➡️ Using CSS Variable is a good practice too. Especially for the cleaner code.
Above all that, well done on the project. Keep it up. 🙂
1
