I would like to know if there is any way i can make my code cleaner and more efficient.
Any and all feedback is also welcome. Thanks.
import not fetch because it felt easier for me to understand and use. is fetch a better option? or i can keep using import without expecting future problems?none, but any feedback is always welcome. 😊
is there a better way i could have done/gotten the shape of the box??
I loved the project, I used React which is very flexible with the use of components.
What challenges did you encounter, and how did you overcome them?The challenge was to use components in the buttons to make the solution more dynamic.
What specific areas of your project would you like help with?Any advice or improvement on the solution would be greatly appreciated.
Hello Alejandro,
Good job on completing the challenge.
One thing i noticed is that the selected tip does not reset when then reset button is clicked.
I also noticed that the webpage assumes the mobile view when the viewport is anywhere below 1440px. so, check your code for responsiveness again.
Good Luck. Happy Coding.
This is my first time using the JSON file and making dynamic content in this project.
What challenges did you encounter, and how did you overcome them?Create dynamic content in the DOM. I searched videos and read some stuff on how to use JSON files.
What specific areas of your project would you like help with?While designing the cards, I noticed the section's background color peeking through where the task info is the child element—making it visually like it has a border-bottom on the right and left. I need some feedback if there is a way to improve this.
Instead of just copying the data.json contents into your main.js, you can import the data by using import
syntax:
import data from './data.json' assert { type: json }
See resource: Import your JSON and CSS as ES Modules
Congratulations on completing the challenge. Well done 👏
Looking at your work, here are some suggestions;
cont and card, so when it is clicked on, it'll return the form to the original state without having to refresh the entire pageIf this feedback was helpful to you, click on the 👍 MARK AS HELPFUL button. Thanks.
Happy Coding.
Stanmancer (►__◄)
doing mobile first
What challenges did you encounter, and how did you overcome them?nothing
What specific areas of your project would you like help with?none
Hello @jmahamed,
Nice work on your solution 👍. A nice addition to your solution will be to make the share card go back to original display i.e. display: none; when the share button is clicked on again.
Suggested Algo when button is clicked;
is the share card already displayed ?
Yes {
Hide the share card
}
No {
show the share card
}
Hello, R. HAYES,
I believe using individual classes for each card/section is a better option in the long run.
You might want to rearrange the cards later on or redesign a specific card, instead of going back to the html and editing the position of each card, you could just target the specific card that needs repositioning and do your magic.
