I find difficulties on controlling the left and the right gap of the QR code image and the white space. That resulted in me not being able to make a more similar looking website compared to the design, as I feel that aforementioned gaps in the design are smaller than the one I made. Instead of minimizing those gaps, I used paddings to push the image lower, to match the left and right gaps which I couldn't control. Feedbacks are very appreciated on how to solve it
F4Bz3
@FabianWassermannAll comments
- @IgnatiusVisnuSubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
First of all, you did a great job! I looked at your code and found why it was difficult for you to control the spacing.
You used % for nearly everything. I prefer using rem instead. It gives you better control and for smaller devices you can use media queries. You can give the card a max-width so that the card can not get bigger at some point but can always get smaller. Let the content inside of the card decide how big your card gets. When you use px, em or rem for the padding it will be much easier to control.
I hope this helps. 😉
0 - @bacigala25Submitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great work! Very clean! I only found one point to improve.
- Implement the seperator as shown in the design.
Keep it going!
Marked as helpful0 - @myrealstorySubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great work! Here are some points for improvement.
- You can make the card much wider.
- Use bigger padding to let the content breath.
- I would recommend using landmarks like <main>, <content> or <header>.
Keep it going!
0 - @japhielxSubmitted over 1 year ago
Any thoughts about my code? I used line break for my main and paragraph tags. To make sure that everything will be at the center I use display block.
@FabianWassermannPosted over 1 year agoGreat work! Here are some points for improvement.
- The qr-image should have a small border radius.
- You can use more padding to let the content breath.
- Regarding your code I have to say that the landmark main should wrap your whole modal and the heading should be wrapped in an h1-tag.
Keep it going!
Marked as helpful0 - @mygodefiSubmitted over 1 year ago
I tried to make it looking as close to the design sample as possible. Its actually very easy to make up the QR code card by using bootstrap card class. But I felt that I have too much classes for trying to place it in the center of the screen as well as trying to put a frame behind it. Hope you can share your solution to this one!THANKS!
@FabianWassermannPosted over 1 year agoGreat work! Here are some points for improvement.
- You should use a bigger border radius on the card as on the image.
- The card of the design has not border and a slight shadow effect.
I can not look at your code because you linked the wrong repo.
Keep it going!
0 - @JosueFreakSubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great work! Here are some points for improvement.
- You should not use such a hard shadow in this case.
- The text should be centered.
- You can use a bigger border radius on the card element.
Keep it going!
0 - @rohankallianpurSubmitted over 1 year ago
- Is this semantically correct?
- Am I using unnecessary elements?
- Could I have commented this better without making it cluttered?
- Is there a way to do this without so many Media Queries?
@FabianWassermannPosted over 1 year agoGreat work! Here are some points for improvement.
- You don't need to comment every end-tag. I make empty lines or commented lines before i start new sections or elements.
- Your img needs an alt text to show if the image can't be shown.
- It works with only one or two media queries as-well. Set a fixed max width in rem or px and a fixed margin left and right. On smaller devices I would recommend to shrink down the text a bit.
Keep it going!
Marked as helpful1 - @TarundeepJoshiSubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great work! Here are some points for improvement.
- You can use more padding to let the content breath.
- Make sure you are using the right font weights. The button text is too thin.
Keep it going!
Marked as helpful0 - @boubakerjouiniSubmitted over 1 year ago
Had a bit of trouble with the gradients but i think it's a good solution using Vite, React & tailwind 😃
@FabianWassermannPosted over 1 year agoGreat work! I like tailwindcss and used it myself for this challange. Here are some points for improvement.
- You can add a bigger padding to let the content breath.
- Apply a slight opacity on the text color of the summary numbers. They are too black right now.
- Give the button the right dark gray blue background color.
Keep it going!
Marked as helpful1 - @jorgegouvSubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great job! Here are some points for improvement.
- Use landmarks like <main>, <content> or <header>.
- Make sure you are using the right border radius and box shadows.
Keep that work going! 👍🏼
Marked as helpful1 - @hiddenLadderSubmitted over 1 year ago@FabianWassermannPosted over 1 year ago
Great work! Here are some improvements.
- You can adjust the line-height on your heading and text, so that is gets more readable.
- Use landmarks to mark where your content is. Landmarks are tags like <main>, <content> or <footer>. They basically tell the user where which section is located.
Keep doing!
Marked as helpful1 - @HIRWA13Submitted over 1 year ago
Hi everyone its been long without submitting but am back. so here I managed to do the design of the challenge but I would like to ask for help on: 1.How to do the hover on the image 2.how to design the box-shadows that look exactly the same as the ones in the challenge.
thanks have a productive day.
@FabianWassermannPosted over 1 year agoGreat work!
The hover effect on the image was a tricky one, but here is how I made it. I basically made a position relative container, which contains the image and the button. The button is position absolute and contains the view icon. The ::before element on the button creates the color effect. The z-index makes sure that the icon is above the color overlay so that the icon is fully white.
Here is the code:
<div class="main-image-box"> <img src="./images/image-equilibrium.jpg" class="main-image" alt="Equilibrium Image" /> <button class="main-image-overlay-button"> <svg width="48" height="48" xmlns="http://www.w3.org/2000/svg"> <g fill="none" fill-rule="evenodd"> <path d="M0 0h48v48H0z" /> <path d="M24 9C14 9 5.46 15.22 2 24c3.46 8.78 12 15 22 15 10.01 0 18.54-6.22 22-15-3.46-8.78-11.99-15-22-15Zm0 25c-5.52 0-10-4.48-10-10s4.48-10 10-10 10 4.48 10 10-4.48 10-10 10Zm0-16c-3.31 0-6 2.69-6 6s2.69 6 6 6 6-2.69 6-6-2.69-6-6-6Z" fill="#FFF" fill-rule="nonzero" /> </g> </svg> </button> </div>
.main-image-box { position: relative; height: min-content; border-radius: 0.5rem; overflow: hidden; } .main-image-overlay-button { cursor: pointer; display: none; position: absolute; background-color: transparent; border: none; justify-content: center; align-items: center; top: 0; left: 0; bottom: 0; right: 0; } .main-image-overlay-button svg { z-index: 11; } .main-image-overlay-button::before { content: ""; position: absolute; display: block; background-color: hsl(178, 100%, 50%); z-index: 10; opacity: 0.4; top: 0; left: 0; bottom: 0; right: 0; } .main-image-box:hover .main-image-overlay-button { display: flex; } .main-image { display: block; width: 100%; }
Keep doing!
2