Skip to content
Submitted 2 months ago

QR-code avec flexbox

Gurbala Laurent•90
@laurentGurbala
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

En réalisant ce projet, j'ai appris à utiliser Flexbox de manière efficace pour gérer la mise en page et l'alignement. J'ai aussi perfectionné l'utilisation des unités 'rem' pour assurer une meilleure adaptabilité aux différentes tailles d'écran. La prochaine fois j'aimerai me pencher sur l'approche mobile-first

What challenges did you encounter, and how did you overcome them?

J'ai rencontré un problème avec GitHub Pages où mes fichiers CSS n'étaient pas correctement chargés, ce qui a rendu le style de mon projet incorrect. J'ai résolu ce problème en rajoutant un './' devant le chemin relatif pour pallier ce "bug" apparemment récurant sur GitHub Pages. Tester la mise en page sur différentes tailles de navigateur a également été un défi intéressant, mais grâce à Flexbox et aux unités rem, tout s'est bien ajusté

What specific areas of your project would you like help with?

J'aimerais avoir des retours sur la manière d'optimiser encore davantage mon design pour des résolutions d'écran très larges ou très petites. Je me demande si j'aurais pu structurer mon CSS différemment afin d'améliorer la flexibilité et la réactivité du design. Aussi, des conseils sur la gestion des polices et des tailles de texte pour encore plus d'accessibilité seraient très appréciés.

Community feedback

  • MikDra1•6,980
    @MikDra1Posted 2 months ago

    Well done, here are some things to review 😊:

    • CSS Reset: Consider adding a full modern CSS reset (like normalize.css or custom resets at the beginning of the stylesheet) to ensure consistent styling across different browsers. Here is a link to one I really like.

    • Clamp() for Responsiveness: Use the clamp() function for fluid typography and spacing, allowing elements to resize smoothly between a minimum and maximum value based on the viewport size (e.g., font-size: clamp(1rem, 2vw, 1.5rem)).

    • Responsive Card: To make the card responsive, ensure the layout uses flex or grid combined with max-width instead of fixed width values. This will make the design more flexible and adapt better to different screen sizes.

    • Use max-width/min-width and max-height/min-height: Instead of using fixed width and height, opt for max-width or min-width to allow the elements to resize smoothly on different screen sizes, improving overall responsiveness.

    Hope you found this comment helpful 💗💗💗

    Good job and keep going 😁😊😉

    Marked as helpful
    1
    Gurbala Laurent•90
    @laurentGurbalaPosted 2 months ago

    Thanks very much @MikDra1

    0
  • tjnut3•90
    @tjnut3Posted 2 months ago

    it's better to bring your font down.

    1
  • Fouad Gantri•20
    @fgantriPosted 2 months ago

    Does the solution include semantic HTML? Yes your code is semantically structured

    Is it accessible, and what improvements could be made? I didn't see big accessibility issues

    Does the layout look good on a range of screen sizes? Yes.

    Is the code well-structured, readable, and reusable? Yes.

    Does the solution differ considerably from the design? A bit. The paragraph content has different width sizes. I suggest putting the title and the paragraph inside a container element and give it a proper padding

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord