Skip to content
OverviewSolutions
3
Comments
3

ama-lyn

@ama-lyn50 points

Coding with a purpose, building a better tomorrow.

Latest solutions

Latest comments

  • ama-lyn•50
    @ama-lyn

    Separation of concerns

    Inline styles (or styles placed directly in the <head> section) mix HTML structure with CSS, making the code harder to maintain and scale in larger projects.

    It's better to keep your styles in a dedicated stylesheet (like style.css in this case), which keeps your HTML clean and easier to manage.

    Move the styles for .attribution into the external CSS file (style.css)

    Marked as helpful
  • ama-lyn•50
    @ama-lyn

    One thing I would recommend is using CSS custom properties (variables) for values that are repeated across your stylesheet. This can help improve maintainability and flexibility.

  • ama-lyn•50
    @ama-lyn

    The use of Flexbox for centering the card both horizontally and vertically is effective, and the layout is generally responsive, adapting well to various screen sizes.

    The CSS is organized, and the naming conventions for classes are clear, making the code readable and maintainable.

    Accessibility has been considered with the inclusion of alt text for the image.

    The design matches the intended look.

    The code is well-structured and looks great.