Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Painel de controle de tempo responsivo

@tatyanepgoncalves

Desktop design screenshot for the Time tracking dashboard coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

Have managed to finish the project using tailwind.

What challenges did you encounter, and how did you overcome them?

The lack of practice with JS so far continues to be a problem for me.

What specific areas of your project would you like help with?

JS and JSON files.

Community feedback

@R3ygoski

Posted

Olá novamente Tatyane! Parabéns por ter concluído mais um projeto.

Dei uma olhada no seu CSS, e percebi que você está referenciando suas imagens de forma errada, suas imagens estão na pasta images e você está buscando a url delas no seu style.css, então o correto a se referenciar seria: ../images/<nome-da-imagem>.svg

Dei uma olhada no seu código e no HTML notei que você utilizou varios Id's repetidos, isso não é nem um pouco recomendável, os Id's são únicos para cada elemento. Caso você queira dar uma "identidade" semelhante a um grupo de elementos, você pode usar a classe, mas como você utilizou o Tailwind, acredito que você ficou com medo de deixar verboso ou confuso, então você poderia optar por utilizar os Data-attributes. Exemplo de um:

<p id="daily" data-timeFrame >Daily</p>
<p id="weekly"  data-timeFrame >Weekly</p>
<p id="monthly"  data-timeFrame >Monthly</p>
obs.: Removi o class e style, para ficar mais claro e conciso.

Daí para referenciar você faria isso:

const timeFrames = document.querySelectorAll("[data-timeFrame]");

Bom, eu novamente tomei a liberdade de editar o seu código JS, e te mandarei ele com algumas correções novamente lá na aba de issues do seu projeto. A única parte que eu modifiquei novamente, foi a parte de lógica.

Uma outra dica que é mais algo pessoal do que "técnico", é sobre o Tailwind, você poderia usar ele de uma forma mais simples, que é usando só o cdn, sem a necessidade do Node.js. E o que você achou do Tailwind?

E novamente, parabéns pelo seu esforço e dedicação na conclusão do projeto, caso algo que eu tenha dito aqui não tenha ficado claro, já sabe, só comentar aqui embaixo que tentarei ajudar da melhor forma possível.

Marked as helpful

0

@tatyanepgoncalves

Posted

Olá, @R3ygoski Realmente esse meu projeto ficou com esse problema nos id's e alguns outros em relação ao tailwind.

O Tailwind é bacana! Estou acho melhor do o CSS puro. Acho bem mais simples deixar o layout responsivo com ele do com CSS. Dica anotada na próxima tentarei usar o cdn.

Muito obrigada!

1

Please log in to post a comment

Log in with GitHub
Discord logo

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