Design comparison
Solution retrospective
I'm proud that I could do it without figma files, I got a bit perfectionist with pixel while doing it. 🤣
What challenges did you encounter, and how did you overcome them?I think that was using Sass for a really small project where I tried to separate everything as for a large project.
What specific areas of your project would you like help with?Right know I don't know much, but I hope you will find something strange in my project to help me improve. 🤣
Community feedback
- @R3ygoskiPosted 7 months ago
Olá @mendesdomingosdev, novamente parabéns pelo projeto, ele ficou mutio semelhante ao design proposto.
Como o comentário acima(ou abaixo) disse, você usou muitos scss, mas calma que vou explicar isso, o que aconteceu não é que você utilizou muitos arquivos scss, mas sim que você utilizou todos como um arquivo normal, geralmente quando vamos fazer um projeto no scss, nós criamos um arquivo onde será importado todos arquivos parciais, os arquivos parciais, nós escrevemos eles com um underline como prefixo, exemplo:
_header.scss
, dessa forma você fala pro compilador não transformar esse arquivo em um CSS, assim evita que tenha vários CSS no seu projeto.No seu projeto por exemplo,
base.scss
seria o único sem underline, já o omain.scss
ebutton.scss
teriam que ter o underline, para que não fossem compilados em CSS também.Se me permite, esses dias atrás eu fiz um projeto, e acho que talvez ele ilustre bem o que eu quero dizer, é esse o projeto: Time Tracking Dashboard - Repo, nele vá no arquvio
index.scss
que é o scss base, ali você verá como é feita esse parte de importação, e logo em seguida vá na pastascss
que você verá como são os arquivos parciais.Agora sobre a metodologia BEM, ela está com a ordem trocada, no seu projeto tem por exemplo
<div class="card-content__text">
o correto na metodologia BEM, seria:<div class="card__content-text">
, a separação de Bloco e Elemento é feita com "__", e palavras compostas são separadas por "-".Edit: Tanto que se você utilizasse a classe dessa forma
card__content-text
, você conseguiria usar um parent selector, que ficaria assim:.card { /* Propriedades omitidas */ &__content-text { /* Propriedades omitidas */ } }
Esse,
&__content-text
, seria equivalente a.card__content-text
, isso porque ele pega o nome do seletor pai com o&
, e adiciona o resto que você escrever ali, que no caso é__content-text
E novamente parabéns pela conclusão de mais um projeto, e principalmente por ter aplicado novos conceitos, caso algo que eu tenha dito tenha ficado confuso, já sabe, só comentar aqui embaixo.
Marked as helpful1@mendesdomingosdevPosted 7 months ago@R3ygoski Acredita que eu tava realmente com essa dúvida de ter tanto arquivos css? kkkkkkkk eu achei estranho, mas fiquei tipo "humm deve ser assim mesmo", essa do parent selector é mto interessante, nem sabia que dava pra fazer algo assim, vou aplicar 😶🌫️obrigado por me ajudar dnv kkkkk edit: ficou lindinho e arrumado, agora falta tirar um dia pra fazer em todos os outros já feitos
1 - @carstenkoernerPosted 7 months ago
Your result looks really good. Great work. I'm not a professional yet, but if you'd like some tips, I can tell you the following.
- You are working with several scss files. You should set your tool for compiling so that only one css file is created, which then contains everything.
- For more accessibility on your pages, you can also set a focus on links. This helps people who use the keyboard and the tab key to scroll through the links or input fields on the page.
JM2C
Marked as helpful1@mendesdomingosdevPosted 7 months ago@carstenkoerner I see, I'm gonna take a look about this focus, since it's new for me tnhx
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