Time tracking dashboard that updates the data on the UI using the provided JSON file. The JSON part was quite challenging for me, as that was the first time I work with JSON data, but I did my best. So, if you have any suggestions on how I can improve the challenge, their very welcome !!
JosephGabriel
@Joseph1032All comments
- @abanicaisseSubmitted almost 3 years ago@Joseph1032Posted almost 3 years ago
This looks great! Don't forget to add alternative text to your <img> elements to avoid the accessibility issues :)
<img src="" alt="alternative text">
0 - @CamiloGavSubmitted almost 3 years ago
Hello everyone! Thanks for looking at my code ... I don't have any questions, but if I would like to hear your recommendations: What should be done from mobile to desktop, or from desktop to mobile?
Greetings!! that is very good.
@Joseph1032Posted almost 3 years agoI would just add more space between the paragraph text and the button so it looks more like the design example. Other than that, it looks good!
0 - @Nidhikumari-4Submitted almost 3 years ago@Joseph1032Posted almost 3 years ago
Don't forget to add alt text to your images to prevent getting those HTML issues :)
<img src="" alt="">
0 - @CamiloGavSubmitted almost 3 years ago
Hello everyone! Thanks in advance for checking my code ... question: how could I have added a blue background to the eye image? I did not find the solution.
Greetings from Argentina
@Joseph1032Posted almost 3 years agoHi, Camilo. For the blue background, I would suggest taking a second look at the pseudo element ::before. Try using the background property instead of content for calling the eye image. It would look something like this:
content: " "; position: absolute; background: url(./images/icon-view.svg) cadetblue; background-repeat: no-repeat; background-position: center; height: 100%; width: 100%; opacity: 0;
You might need to re-work your HTML to get everything right, but it's a start :)
Marked as helpful0 - @Destiny-16Submitted almost 3 years ago@Joseph1032Posted almost 3 years ago
To avoid this type of error from the Accessibility Issues: Heading levels should only increase by one
Ensure that you're not skipping heading levels. In your HTML, you have an <h2> on line 33 and then an <h4> on line 40, skipping <h3>. You can fix this simply by replacing <h4> with <h3> and your <h5> with <h4>.
Also, wrap your markup in <main></main> inside the <body> to avoid the landmark error from the Accessibility Issues as well.
Nice work :)
Marked as helpful1