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

Fix/ apps card #204

Merged
merged 10 commits into from
Sep 8, 2024
Merged

Fix/ apps card #204

merged 10 commits into from
Sep 8, 2024

Conversation

juanzeen
Copy link
Contributor

@juanzeen juanzeen commented Sep 4, 2024

Descrição

Correções feitas na seção de aplicativos na plataforma, somente alterei a imagem do card de aplicativos na landing e corrigi também subtítulos e imagens nos cards dentro da página de aplicativos.

Pontos para atenção

  • Fiz uma pequena alteração no CSS dos cards que possuem link na landing, pois reutilizei esse estilo para fazer os cards de aplicativos.

Possui novas configurações?

  • A alteração no CSS;
  • Troquei o text-wrap: nowrap; para text-wrap: wrap;,

Prints de resultado

image

image

Última alteração

image

@juanzeen juanzeen self-assigned this Sep 4, 2024
@juanzeen juanzeen requested a review from zoedsoupe September 5, 2024 01:15
@zoedsoupe
Copy link
Member

@juanzeen tá quebrando o linter!

@juanzeen
Copy link
Contributor Author

juanzeen commented Sep 6, 2024

o erro é o mix format? Eu rodei antes de subir o código e aparentemente tava tudo certinho, vou checar aqui.
Já sei o que aconteceu, estava dando conflito com a main e eu tirei resolvi ele pelo github mesmo, provavelmente nessa hora quebrou a formatação.

@zoedsoupe
Copy link
Member

sim, é o mix format msm

@juanzeen
Copy link
Contributor Author

juanzeen commented Sep 7, 2024

Zoey, consegui resolver aqui! Mas acabou que não consegui dar push direto da minha máquina, precisei fazer a alteração direto pelo github. Sabe me dizer se o CI tá com algum problema ou se foi eu que não me liguei e esqueci de fazer algo?

image

@zoedsoupe zoedsoupe merged commit e4942fd into main Sep 8, 2024
3 checks passed
@zoedsoupe zoedsoupe deleted the feat-apps-card branch September 8, 2024 15:51
douglastofoli added a commit that referenced this pull request Sep 13, 2024
…arte-plataforma into feat/responsive-landing

* 'feat/responsive-landing' of github.com:peapescarte/pescarte-plataforma: (25 commits)
  fix: please DOCKER SAVE ME FROM THIS CRUEL LIFE
  fix: ditch faker from dev env
  fix: define project env on compile-time
  fix: re-state arg from env in Dockerfile
  fix: simplify CI
  Feat/publi artigos (#211)
  fix: i can't handle CI anymore 😭
  fix: build prod ignore manual MIX_ENV
  Feat/publi artigos (#210)
  Feat/livros (#209)
  corrigido rota boletin para boletins, e ajuste no width da imagem das capas dos boletins (#208)
  fix: formatter
  fix: avoid docker login on dev/test ci
  Fix/style pictures pesquisa (#205)
  fix/quem somos update (#206)
  Fix/ apps card (#204)
  fix: only fetch specific env deps for each ci part
  fix: adding agenda warning text (#207)
  fix: git hooks
  feat: improve ci, reduce build time
  ...
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