Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix auto-signing new input when usingaccountingSign option#399

Open
zabakala wants to merge 1 commit intodm4t2:mainfrom
zabakala:handle-accounting-sign-on-tab-key-focus
Open

Fix auto-signing new input when usingaccountingSign option#399
zabakala wants to merge 1 commit intodm4t2:mainfrom
zabakala:handle-accounting-sign-on-tab-key-focus

Conversation

@zabakala
Copy link

@zabakalazabakala commentedJul 30, 2023
edited
Loading

Hi. I have spotted an issue relating to the usage of theaccounting sign. It is a case when you focus a field using theTAB 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

BugA0 Bugs
VulnerabilityA0 Vulnerabilities
Security HotspotA0 Security Hotspots
Code SmellA0 Code Smells

No Coverage information No Coverage information
0.0%0.0% Duplication

@dm4t2
Copy link
Owner

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

codecovbot commentedAug 9, 2023

Codecov Report

Merging#399 (352b2c5) intomaster (21bd4c7) willdecrease coverage by0.41%.
The diff coverage is25.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 ChangedCoverage Δ
src/currencyInput.ts84.43% <25.00%> (-0.80%)⬇️
dm4t2 reacted with eyes emoji

@dm4t2dm4t2 changed the titlehandle-accounting-sign-on-tab-key-focusFix auto-signing new input when usingaccountingSign optionAug 9, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zabakala@dm4t2

[8]ページ先頭

©2009-2026 Movatter.jp