search

levxyca / diciotech

Um dicionário tech para pessoas que querem aprender mais sobre termos técnicos dentro da tecnologia 📖
https://diciotech.netlify.app/
GNU General Public License v3.0
391 stars 69 forks source link

refactor: favorite cards #87

Closed freitaschz closed 10 months ago

freitaschz commented 11 months ago

O que esse PR muda

Foi obtido como base o PR #77 e refatorado o JavaScript do mesmo, pois para a feature de favoritar um card funcionar, depende de adicionar uma chave "id" em cada termo no arquivo cards_pt-br.json. Com a refatoração, cada card é armazenado no localStorage através do seu título (trabalhando como id único) com todas as letras minúsculas, separadas por hífen e sem símbolos e acentos.

Além disso, todas as estilizações do CSS foram para o SASS e a largura máxima do modal de consentimento de cookies foi aumentada. A mensagem do modal também foi alterada e um link externo onde explica a utilização dos cookies acrescentada: Why websites use cookies?.

Dependências

Esse pull request depende desse outro:

netlify[bot] commented 11 months ago

Deploy Preview for diciotech ready!

Name Link
Latest commit 14a450492b8ea67caf76311d720b3ed8b7fb1fe9
Latest deploy log https://app.netlify.com/sites/diciotech/deploys/654bdaf68518220008cb14f6
Deploy Preview https://deploy-preview-87--diciotech.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

levxyca commented 10 months ago

@thiagofqs consegue atualizar sua branch? Vou dar merge no PR #77 e quero dar merge no seu logo em seguida :)

freitaschz commented 10 months ago

@thiagofqs consegue atualizar sua branch? Vou dar merge no PR #77 e quero dar merge no seu logo em seguida :)

Prontinho, @levxyca! Os conflitos foram corrigidos.

levxyca commented 10 months ago

@thiagofqs acho que vai precisar atualizar depois do merge do #77 também 🫂

freitaschz commented 10 months ago

@thiagofqs acho que vai precisar atualizar depois do merge do #77 também 🫂

Atualizei, mas está com muito commit, quer que eu reduza-os? 🥲

levxyca commented 10 months ago

@thiagofqs acho que vai precisar atualizar depois do merge do #77 também 🫂

Atualizei, mas está com muito commit, quer que eu reduza-os? 🥲

Seria bom se conseguisse 🥹

freitaschz commented 10 months ago

@thiagofqs acho que vai precisar atualizar depois do merge do #77 também 🫂

Atualizei, mas está com muito commit, quer que eu reduza-os? 🥲

Seria bom se conseguisse 🥹

Feito, @levxyca! :)