FAQ-Accordion using CSS Grid, SASS and JS Vanilla
Design comparison
Solution retrospective
“Hello everyone! I would like to share the solution I developed for the challenge. As I am currently enhancing my knowledge in SASS, I decided to incorporate what I have learned so far into this solution. I am completely open to receiving feedback, as I want to understand where I can improve or what I need to study more. I appreciate in advance any contribution! 😊
Olá a todos! Gostaria de compartilhar a solução que desenvolvi para o desafio. Como estou atualmente aprimorando meus conhecimentos em SASS, decidi incorporar o que aprendi até agora nesta solução. Estou totalmente aberto a receber feedbacks, pois quero entender onde posso melhorar ou o que preciso estudar mais. Agradeço antecipadamente por qualquer contribuição! 😊
Community feedback
- @danielmrz-devPosted 11 months ago
Fala Renan!
Meu amigo, teu projeto está excelente. Tanto o resultado final quanto a organização dos arquivos do seu codigo e também a parte semântica do seu HTML.
A única sugestão que eu daria seria agrupar todas as variáveis em um único arquivo. Notei que você fez um arquivo pras cores e outro pra tipografia. A não ser que você tenha algum motivo específico pra isso, acho que não há necessidade.
Fora isso, como eu disse, você fez um excelente trabalho! 😁
Marked as helpful1@irenanrodriguesPosted 11 months ago@danielmrz-dev, muito obrigado por ter reservado um pouco do seu tempo para analisar meu desafio.
Confesso que fiquei muito na dúvida sobre os arquivos de cores e tipografia, se deixava tudo em um arquivo ou separados 🤣. Mas obrigado pela dica para agrupar todas as variáveis
0
Please log in to post a comment
Log in with GitHubJoin 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