@Bonrey
Posted
Hello, Yaiza! I think you did a pretty good job on this challenge! π I couldn't find any bugs as regards the JS part. π
However, there are still some things you might want to improve. π
- First of all, I would fix your accessibility issues by adding the
aria-label
attribute to yourinput
elements. This way, you'd let assistive technologies know what all these radio buttons and text fields are about. - Each of your sections should have a heading. So, it'd be a better idea to put your
section
tag insidediv
and not vice versa. π - Probably, not so important, but still worth mentioning: I'd make the Crowdfund logo clickable (and refresh the page on the click, for example).
As for your SASS folder structure, I think it's fine! I refer to this site whenever I want to make sure I put a new .scss file in the right place.
Anyway, it's great that you put so much effort into this challenge! I think you learned a lot along the way. π So, keep on coding, and good luck! π
@Yaiza16
Posted
@Bonrey Hi, Bonrey! Firstly, thanks a lot for taking your time to answer! I really appreciate it :)
I've taken your advices into account so as soon as I finish my last challenge, I'm going to do change the code!!