Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Estruturação layout construction-01 #37

Merged
merged 9 commits into from
Jun 22, 2021
Merged

Conversation

rodrigoKulb
Copy link
Contributor

Estruturação da página do layout construction-01

  • Faltam ajustes mobile. (construction-01.public.js)
  • Atualização de fonte personalizada (/css/globals.css)
  • Upload de uma imagem (/pages/pocs/img/teclado.jpg)
  • Instalação do next-images (package.json e next.config)

Estruturação da página do layout construction-01
- Faltam ajustes mobile. (construction-01.public.js)
- Atualização de fonte personalizada (/css/globals.css)
- Upload de uma imagem (/pages/pocs/img/teclado.jpg)
- Instalação do next-images (package.json e next.config)
@vercel
Copy link

vercel bot commented Jun 20, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/filipedeschamps/tabnews/GuZLyHj3zzACcyRFY296MKiFbxcJ
✅ Preview: https://tabnews-git-rodrigokulb-patch-1-filipedeschamps.vercel.app

@rodrigoKulb
Copy link
Contributor Author

rodrigoKulb commented Jun 20, 2021

@filipedeschamps deu alguma falha no deployed da Vercel, mas acho que não tenho permissão para visualizar. Ao acessar o link de Inspect apresenta o erro 404.

@rodrigoKulb
Copy link
Contributor Author

localhost funcionou de boa!
Captura de tela em 2021-06-20 16-55-26

@filipedeschamps
Copy link
Owner

hahahah sensacionaaaal @rodrigoKulb 😂 👍 e testando a URL, de fato não funciona, e analisei os logs do build e tava tudo certo :/

Em paralelo, recomendo não utilizar o next-images, porque está incompatível com a versão 11 do Next.js e por essa issue abaixo, parece que essa nova versão do framework já importa estáticos por padrão agora 🎉
twopluszero/next-images#73 (comment)

E também não vai ser possível fazer o merge por conta da alteração da estilização global, teríamos que dar um jeito de ficar tudo isolado no escopo da poc 🤝 como fazer? Será que é possível deixar a fonte inline?

@rodrigoKulb
Copy link
Contributor Author

rodrigoKulb commented Jun 21, 2021

@filipedeschamps realmente removi o next-images e funcionou normal no NEXT 11, só precisei adicionar o .src porque ele retorna a imagem como um objeto muito legal!

A Vercel está com o Next 11 ?

Deixei a fonte como style in line.

Falta somente a imagem de fundo.

https://tabnews-git-rodrigokulb-patch-1-filipedeschamps.vercel.app/pocs/construction-01

Ps. Não consigo ver o log de deploying

@filipedeschamps
Copy link
Owner

A Vercel está com o Next 11 ?

Na verdade a Vercel é apenas o server, e cabe o next@11 ser listado no projeto (e ser compatível com a versão de Node.js que a Vercel estiver rodando 👍

https://tabnews-git-rodrigokulb-patch-1-filipedeschamps.vercel.app/pocs/construction-01

Agora abriu 100% 😍

Ps. Não consigo ver o log de deploying

Eu to usando conta pessoal na Vercel, não vai dar para ver mesmo :( mas também quando vi, não tinha nenhum erro, muito estranho 😂 👍

@rodrigoKulb
Copy link
Contributor Author

u to usando conta pessoal na Vercel, não vai dar para ver mesmo :( mas também quando vi, não tinha nenhum erro, muito estranho

Pelo que verifiquei quando tirei o static image deu erro aqui:
Captura de tela em 2021-06-21 15-13-42

@rodrigoKulb
Copy link
Contributor Author

rodrigoKulb commented Jun 21, 2021

@filipedeschamps Resolvido!!🙏️ faltou adicionar a typeface-poppins font do layout!

  1. next-images => Removido
  2. estilização global = > Removido (adicionei inline)

https://tabnews-97y7bg2wx-filipedeschamps.vercel.app/pocs/construction-01

@filipedeschamps
Copy link
Owner

Show!! O que acha de, para isolar ao máximo a alteração feita pela poc, declarar a fonte por um link externo como documentado aqui e daí é possível remover a dependência global de typeface-poppins e substituir por essa fonte hospedada Google Fonts

@rodrigoKulb
Copy link
Contributor Author

rodrigoKulb commented Jun 22, 2021

Nossa!! Esse projeto está sendo uma puta aula, valew @filipedeschamps!

https://tabnews-2rs0wnuhp-filipedeschamps.vercel.app/pocs/construction-01

@filipedeschamps filipedeschamps merged commit 8d1542f into main Jun 22, 2021
@filipedeschamps filipedeschamps deleted the rodrigoKulb-patch-1 branch June 22, 2021 00:38
@filipedeschamps
Copy link
Owner

Merged 😍 let's gooooo!!!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants