Skip to content

Conversation

@cyrax1800
Copy link

To make a better User Experience

@codecov
Copy link

codecov bot commented Mar 20, 2018

Codecov Report

Merging #62 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #62 +/- ## ========================================== + Coverage 97.95% 98.03% +0.08%  ========================================== Files 1 1 Lines 49 51 +2 Branches 20 21 +1 ========================================== + Hits 48 50 +2  Partials 1 1
Impacted Files Coverage Δ
src/vue-numeric.vue 98.03% <100%> (+0.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f639863...bb00921. Read the comment docs.

@kevinongko
Copy link
Owner

Hi @cyrax1800,

Thanks for the PR however as for user experience, we shouldn't interfere with user input and "correct" the input after the user has finished (on blur) IMHO.

Please consider to file an issue first before proposing implementation changes as in contribution guidelines

@kevinongko kevinongko closed this Mar 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants