Submitted 3 months agoA solution to the Interactive rating component challenge
Interactive rating component using Flexbox & CSS Grid
accessibility, sass/scss, jest
P
@imrebartis

Solution retrospective
What are you most proud of, and what would you do differently next time?
Satisfied with:
- Well-structured JS code
- Semantic HTML
- a11y features
- Use of SCSS variables and mixins
- Implementing DRY principles
- Unit testing
What I would do differently next time:
- Definitely consider using React or Next.js instead of vanilla JS
Figma:
- Challenge: the dev mode of the Figma design provided (the one that offers code snippets for implementation) is available only if you have a paid version of Figma, which I don't have ATM
- Solution: my interpretation of the design is in many cases sort of intuitive (e.g. approximating spacing)
GitHub Actions Workflow:
- Challenge: Complex deployment process with multiple checks
- Solution: Implemented staged workflow with caching
Event Handler Parameter Order
- Challenge: Inconsistent parameter order in event handlers caused bugs
- Solution: Standardized event handler signatures
Memory Leaks in Event Listeners
- Challenge: Event listeners weren't properly cleaned up
- Solution: Implemented cleanup method and unload handler
All feedback is welcome, but since this is a web accessibility project, reviewing the ARIA implementation would be the most helpful.
Code
Loading...
Please log in to post a comment
Log in with GitHubCommunity feedback
No feedback yet. Be the first to give feedback on Imre Bartis's solution.
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord