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

[Bug]: Utilização do componente Money em live() com Placeholder #10

Open
mliell opened this issue Aug 31, 2023 · 6 comments
Open

[Bug]: Utilização do componente Money em live() com Placeholder #10

mliell opened this issue Aug 31, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@mliell
Copy link

mliell commented Aug 31, 2023

What happened?

Ao utilizar o componente Money e obter um get em um Placeholder o mesmo está retornando valores incorretos.

Segue vídeo demonstrando em uma aplicação teste:

Gravacao.de.tela.de.2023-08-31.14-42-04.webm

Como podem ver no vídeo, somente obtendo um get do live() que os valores retornados estão incorretos, ao salvar no banco está ok.

Segue o repositório do código utilizado: https://github.com/mliell/Erro-money-placeholder

How to reproduce the bug

Crie um input tipo Money com live():
Money::make('valor1')->live(),

Retorne os dados em um placeholder:

Placeholder::make('Valor_1')->content(function ($get) { return $get('valor1'); }),

Ou baixe o repósitório https://github.com/mliell/Erro-money-placeholder

Package Version

3.0.1

PHP Version

8.1.12

Laravel Version

10.21.0

Which operating systems does with happen with?

No response

Notes

Banco de dados utilizado: sqlite

@mliell mliell added the bug Something isn't working label Aug 31, 2023
@leandrocfe
Copy link
Owner

Testou live(onBlur:true)?

@mliell
Copy link
Author

mliell commented Aug 31, 2023

Sim, retorna o mesmo.

@mliell
Copy link
Author

mliell commented Aug 31, 2023

Um detalhe interessante. Se colocar o cursor no lado esquerdo do valor (ficando a esquerda da vírgula, sendo os números inteiros) e adicionar um número, o retorno é correto.
Agora se adicionar os números pelo lado esquerdo (decimal) que é o default, acontece o bug.

@leandrocfe
Copy link
Owner

é por conta do javascript..

@leandrocfe
Copy link
Owner

@leoblanski
Copy link

leoblanski commented Oct 14, 2024

Opa Leandro!!

Testei aqui mas o problema persiste...

leandrocfe/filament-ptbr-form-fields: 3.1.0

Exemplo:

Screen.Recording.2024-10-14.at.09.48.21.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants