Skip to content
OverviewSolutions
6
Comments
8
winning Godspower
@winninggodspower

All comments

  • winning Godspower•160
    @winninggodspowerPosted about 2 years ago

    The link is not showing up. I checked out your repo and it seems that your project is in a subfolder therefore the GitHub page URL should be indexed by the folder as: /advice-generator-app-main/

    just update the challenge URL to this link https://maru10h.github.io/Desafios-Frontend-Mentor/advice-generator-app-main/

    Marked as helpful
    0
  • winning Godspower•160
    @winninggodspowerPosted about 2 years ago

    NICE!!, BUT I DON'T SEE THE LOCATION ICON IN MINE.

    Marked as helpful
    1
  • winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    Yeah. I also used chart.js l.. but how did you get rid of the graph behind the chart??

    0
  • winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    Nice. Love your creativity. But the search seems not to work very well. It doesn't give me feedback sometimes

    0
  • Mohammed Kasim•210
    @MO-KA19Submitted over 2 years ago

    I just added two extra features to this task, translate to ar (my native language) and copy to clipboard, try it and tell me if you found any bugs, also I will be glade to know how it is on your mobile screen and if you have any feedback about it :) Note: I didn't want to add multiple languages select cuz it will distract the project so I put ar

    Advice generator app solution with extra features

    3
    winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    Nice app. But it's not responsive on mobile. And by the way, is it the API that contains the language translation functionality??

    0
  • winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    Nice design. one of the best i have seen. but you forgot to put the horizontal line in the design.

    1
  • winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    Site isn't responsive. you should work on that. and the distance between the heading and the body was too large. great job by the way!!

    0
  • Saul•1,260
    @xsaulSubmitted over 2 years ago

    I have a problem when I click on any of the buttons it duplicates the cards and I couldn't fix it. Any feedback would be amazing 😅

    time-tracking-dashboard

    1
    winning Godspower•160
    @winninggodspowerPosted over 2 years ago

    I con see you are printing the whole cards again to the frontend instead of just changing the previous value. if you want to go ahead with this approach. then your should clear the previous cards first using

    let cards = document.getElementsByClassName('card')

    cards.forEach(element => {element.remove})

    1