Skip to content
OverviewSolutions
8
Comments
1
Edyta
@Infinit-dot

All comments

  • Edyta•100
    @Infinit-dot

    Hi, overall good job, but I see some things you can improve

    • to solve the problem with the buttons you shouldn't use the explicit height of the container, you can use height: auto, min-height: 600px or height: max-content.
    • you have container-flex but you are not using it for anything, fewer containers possible is always better unless you need them for additional styling.
    • try to use HTML semantics like main, footer etc. this will solve accessibility issues
    • you can improve styling to match better the design
    Marked as helpful