Article preview component

Community feedback

• 

  Car•270

  @Lukas3162000Posted 4 months ago

  Great job on this submission! The design is really well-executed, and it’s clear that you’ve put a lot of effort into getting it as close to the original as possible. The overall layout looks clean and professional, and the functionality appears to work seamlessly, which is a big plus.

  Your tooltip implementation is clever, and it integrates nicely into the design. If you’re unsure about using position: absolute, don’t worry—it’s a valid approach, but experimenting with position: relative in combination with flexible parent containers might make the positioning more adaptable, especially for responsive layouts. Speaking of responsiveness, separating the logic for desktop and mobile behavior more distinctly could make your code easier to manage and fine-tune for different screen sizes.

  Overall, this is an excellent submission, and you should be proud of how close you’ve come to matching the design and functionality. Keep up the great work, and best of luck with future challenges! 😊

  Marked as helpful

  0

  ---

  

  CoolNight99•350

  @CoolNight99Posted 4 months ago

  @Lukas3162000 Thank you for the detailed feedback! I appreciate it.

  0
• 

  Mayssa Ghanmi•170

  @MAY55APosted 4 months ago

  Nice effort, here are some changes to consider:

  • add semantic HTML (at least a main tag) and alternate texts to images for better accessibility.
  • make the container and the font smaller to match the design.
  • instead of defining the width of the article container by specifying a margin, just try specifying a width so it stays constant even when the screen size is changing.
  • try adjusting the width of the image when the screen is smaller by giving it a width relative to the total card width ( you can use % for example).
  • try positioning the share div exactly on top of the author div for the mobile layout and making the share div invisible.

  0