search

r-martins / PagBank-Magento2

Módulo PagSeguro/PagBank para Magento 2 (via Connect)
5 stars 0 forks source link

Improve payment UI #3

Closed ligiasalzano closed 11 months ago

r-martins commented 11 months ago

Parece ótimo, mas Acho que esqueceu de alguma coisa...

  1. A UI ficou assim pra mim:

    image

  2. Ao digitar o cartão, o ponteiro era jogado pra tras a cada etapa. Video

Limpei var/page_cache/ var/view_preprocessed/ pub/static/frontend/*, caches, s:d:c, s:up, e restartei tudo, mas não pareceu ser só um bug de cache.

ligiasalzano commented 11 months ago

Oi, @r-martins. Revisei aqui em uma instalação diferente, olhei no Chrome e no Firefox, e ficou assim: Screenshot from 2023-11-29 14-21-27

Não consegui replicar essa quebra das labels que você mostrou. Qual navegador e versão você usou?

Agora, a mácara com o ponteiro maluco eu repliquei no Firefox :( Vou ver se consigo encontrar alguma solução ou se vou precisar trocar de plugin.

r-martins commented 11 months ago

To usando Firefox + Fire store (pagseguro-exemplo-m2). Após a última modificação, a única coisa que mudou é que tive que ativar o cartao, boleto e pix novamente no admin. Mas o problema da máscara maluca e do css continuam. Testei no Chrome, e deu no mesmo (inclusive a máscara).

r-martins commented 11 months ago

Estamos quase lá...

image
  1. A imagem do cartão e do ? ficaram deslocadas. Temos que ver um jeito que garantir que isso não aconteça em outros checkouts/temas, se possível.
  2. Ao tentar finalizar, nenhuma msg foi exibida, mas foi logado: [2023-12-04T00:43:17.520384+00:00] main.CRITICAL: Placing an order with quote_id hvZY0WwJBYbDKcR4oDMIOSYPipe9WKDO is failed: [40002] Parâmetro inválido. Algum dado foi informado em um formato inválido ou o conjunto de dados não atendeu a todos os requisitos de negócio. (charges[0].payment_method.card.encrypted) [] [].

Precisamos exibir a msg no front (problema antigo?). Sobre o erro em questão, ocorre pq o PB não está mais aceiando o 4111... precisa ser algum cartão de teste válido.

ligiasalzano commented 11 months ago

@r-martins, as labels ainda estão posicionadas de forma errada por causa daquele position: initial !important. Ele sobrescreve o que fiz. Por isso que quebra. Não queria colocar um !important no nosso módulo... Encontrei da onde vem aquele que está sobreescrevendo o nosso: vendor/ricardomartins/brazilcustomerattributes/view/frontend/web/css/source/_module.less:22 A gente pode modificar isso e tirar esses !important?

Sobre o outro problema, estou verificando aqui.

r-martins commented 11 months ago

Confome conversamos, vamos tentar encontrar outra forma (em último caso, colocando !important no nosso css, ou usando outros seletores). Isso porque muita gente usa o módulo mencionado (seja com o meu fork ou o original), e não vão atualizá-lo com nossas alterações.

ligiasalzano commented 11 months ago

@r-martins Por enquanto, só consegui usando o !important mesmo. As outras alternativas esbarraram no z-index, que não funcionava por causa do position: initial !important.