I used css flex and flex direction for chart bar

Community feedback

• 

  P

  Alan Ibarra•440

  @Ibarra11Posted over 2 years ago

  The issue with the data not showing on hover is because you are making a fetch to a url that does not exist. What is happening is that when you do fetch('/data.json'), it makes a request to "https://milandsharma.github.io/data.json" and there is no server or something to return data back to you. Also, I think fetch is not appropriate in this situation since you have the data already, instead you can just import the data and then use the data. It would look something like this inside of your script.js.

  import data from "./data.json";
  

  Marked as helpful

  1
• 

  Naleeka•470

  @NaleekaPosted over 2 years ago

  Hi @milandsharma You have a nice solution

  • You should use alt attribute in the img s, and describe it as where it navigate the user

  <img src="images/image-qr-code.png" alt=" qrcode to frontendmentor website">

  • and you could wrap up all your divs inside a <main>

  <main>
    <div class="balance">... </div>
    <div class="spending"> ...</div>
  </main>
  

  • You should make the website responsive for mobile devices using @media queries

  Overall, nice work ! Hopefully this feedback helps

  happy coding ❤

  Marked as helpful

  0