Latest solutions
FAQ Accordion Using SCSS/SASS
#accessibility#sass/scssSubmitted 13 days agoI would very much appreciate any suggestions that could improve my skills.
Thank you!
Interactive Rating Component Using SCSS and JS
#accessibility#sass/scssSubmitted about 1 month agoI would very much appreciate any suggestions that could improve my skills.
Thank you!
Responsive Tip Calculator App Using SCSS and JS
#accessibility#sass/scssSubmitted 2 months agoI would very much appreciate any suggestions that could improve my skills.
Thank you!
Responsive Time Tracking Dashboard Using SCSS and JS
#sass/scssSubmitted 3 months ago- I'm not sure if what I did for the ellipsis svg using img tags is correct.
- I'm not sure if the ellipsis should be clickable or hoverable. Also, I'm not sure about the expected behavior when the ellipsis is hovered.
I would very much appreciate any suggestions that could improve my skills.
Thank you!
Responsive Newsletter Sign-up Form Using SCSS
#sass/scssSubmitted 4 months agoI would very much appreciate any suggestions that could improve my skills.
Responsive Article Preview Component Using SCSS and JS
Submitted 5 months agoI would very much appreciate any suggestions that could improve my skills.
Latest comments
- @6alactico@telsabate-hub
Hi @6alactico! You did great here! I thought the code is clean and the page has 100% accessibility according to Lighthouse report.
Good job! Happy coding! :)
- @6alactico@telsabate-hub
Good job! Thank you and happy coding! :)
- @NitiemaAllassane@telsabate-hub
Hi @NitiemaDev!
You did a really great job here!! :)
I like how well structured your code is. I cannot comment much about the React part because I don't know it yet. Hehe. The SCSS code is clean and I like that you used mixins for repeating styles.
Maybe just a "nice-to-have" is an error message for invalid characters entered in the input fields or you could have used a number input to prevent entering non-numeric characters. :)
Other than that, your implementation is really good!
Thank you and happy coding! :)
- @anhvu1012@telsabate-hub
Hi Anh!
You did really great here! I like how well-structured your SCSS code is. The page is also responsive so good job on that! :)
Just a couple of suggestions for your JS.
-
When calling the
populateDOM(allValidData, targetId)
, you don't need to pass the allValidData since you have declared it as a global variable. -
You could have directly updated the
innerHTML
insidecreateContainer
instead of having a separate functiondisplayContainers
.
Also, I think you missed out on updating the text
"Previous - "
when changing timeframes.Overall, you still did a nice job here and you were able to complete this project!
Happy coding! :)
Marked as helpful -
- P@stevensunaWhat are you most proud of, and what would you do differently next time?
- Implemented an RFC 5322 compliant email validation system
- Created an accessible form with proper ARIA attributes and error handling
- Used CSS Grid for responsive layout switching between mobile and desktop views
- Developed a modular SCSS architecture with proper separation of concerns
- Implemented smooth transitions for success message display
@telsabate-hubHi Steven!
You did really great here! I like that there is a particular error message for empty email. :) Also, good job on making it responsive.
Maybe the only issue I found is that the inputs are still tabbable when the "Thanks for subscribing" modal shows.
Overall, I think your solution is very good.
Thank you and happy coding! :)
- @AdamCygan@telsabate-hub
The solution is different from the supposed challenge.