Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
8
Comments
5
Daniela Darnea
@mdanieladla

All comments

  • Erik Emmanuel Navarrete Reyes•400
    @apuntesnavarrete
    Submitted over 3 years ago

    flex-box , css grid , css variables , media queries

    #accessibility
    1
    Daniela Darnea•180
    @mdanieladla
    Posted over 3 years ago

    Hi! Good job :)

    If I were you I would not mix two languages in the code, i mean, you have almost every class in english <header class="header"> , <div class="header_content">... and then you have some classes in spanish <p class="header_subtitulo">, <h1 class="header_titulo">... and also i wouldn't use uppercase for the beggining of some words like <div class="content_card Supervisor"> because that can confuse you, I think is better to avoid confusion to use all the words in the code in 1 language and also use only lowercase or uppercase, but not both :) (obviuosly the comments you can put them in the same language or in 2,3... like you wish).

    Also semantically instead of using so many divs, is better to use sections and divide the code in sections, for example, instead of having <div class="main_content">, is better to have <section class="main_content">, so then you don't have like 10000 divs :)

    Cheers.

    Marked as helpful
  • Orkhai Dickson•320
    @orkhai
    Submitted over 3 years ago

    Multi-page website built with React and React-router

    #react#react-router
    1
    Daniela Darnea•180
    @mdanieladla
    Posted over 3 years ago

    Good job!! I think (if I remember correctly) that when the user clicks the white button of EXPLORE on the main page, that has to take the user to the destinations page. Cheers :)

  • Berlianto Ferdynand Pongbubun•200
    @xvferdy
    Submitted over 3 years ago

    fem-time-tracking-dashboard

    #sass/scss#react
    2
    Daniela Darnea•180
    @mdanieladla
    Posted over 3 years ago

    Grid is difficult, but once you understand it, it magically becomes so easy. Here you have 2 resources that have helped me :)

    https://grid.malven.co/ https://cssgridgarden.com/

  • Anh•90
    @Taro2211
    Submitted over 3 years ago

    Faq accordion card main challenge using HTML and CSS only

    1
    Daniela Darnea•180
    @mdanieladla
    Posted over 3 years ago

    Hi! Great job :) If I where you I would have put the click event on the arrow because it is more intuitive for users (in my opinion). Cheers :)

    Marked as helpful
  • Diego Alvarez Garcia•110
    @DiegoSt23
    Submitted over 3 years ago

    Made with React and React Router, CSS and a bit of Framer Motion.

    #react#react-router#motion
    2
    Daniela Darnea•180
    @mdanieladla
    Posted over 3 years ago

    Good job!! I think (if I remember correctly) that when the user clicks the white button of EXPLORE on the main page, that has to take the user to the destination page. And also if I were you I would fix the HTML issues :)

    Marked as helpful
Frontend Mentor logo

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub