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 auto-signing new input when using accountingSign option #399

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zabakala
Copy link

@zabakala zabakala commented Jul 30, 2023

Hi. I have spotted an issue relating to the usage of the accounting sign. It is a case when you focus a field using the TAB key. Whenever you TAB into an input field (whole text gets selected), whatever you is being typed in there will first discard the last (closing) parenthesis and as the typing continues the caret uncontrollably shifts its location. I would like you in this respect to consider this MR as it resolves the issue....or any other solution leading to the resolution for the issue.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dm4t2
Copy link
Owner

dm4t2 commented Aug 9, 2023

Hi, first I want to thank you for contributing and reporting the issue.
I have tested your changes but there are still some issues (see review).

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #399 (352b2c5) into master (21bd4c7) will decrease coverage by 0.41%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master     #399      +/-   ##
==========================================
- Coverage   93.45%   93.04%   -0.41%     
==========================================
  Files           7        7              
  Lines         672      676       +4     
  Branches      189      190       +1     
==========================================
+ Hits          628      629       +1     
- Misses         44       47       +3     
Files Changed Coverage Δ
src/currencyInput.ts 84.43% <25.00%> (-0.80%) ⬇️

@dm4t2 dm4t2 changed the title handle-accounting-sign-on-tab-key-focus Fix auto-signing new input when using accountingSign option Aug 9, 2023
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