payform icon indicating copy to clipboard operation
payform copied to clipboard

Bugfix/63 caret moved to the end

Open andygi opened this issue 5 years ago • 7 comments

Bugfix of #63 (Caret moved to the end of input after change). Fixed for LTR only and testable manually only, by index.html and jquery.html. In the ticket there a clear description of how to reproduce the issue and, consequently how to test it. I have created a jquery and javascript index RTL page in order to test RTL as well.

andygi avatar Apr 04 '19 07:04 andygi

@ddayguerrero,

Do you mind un-staging all changes in the dist/ folder please?

Could you help me to better understand what do you need, please?

andygi avatar Apr 08 '19 09:04 andygi

@ddayguerrero , I see, no problem I have reverted the code into dist directory and thank you for the explanation. To be bulletproof, if you want, I can add this directory into gitingore file as well. Let me know.

andygi avatar Apr 09 '19 07:04 andygi

If you don't mind then that would be cool @andygi!

ddayguerrero avatar Apr 12 '19 04:04 ddayguerrero

If you don't mind then that would be cool @andygi!

@ddayguerrero Sure, I have done!

andygi avatar Apr 12 '19 10:04 andygi

Sorry, but we actually want to not ignore that directory, we need it for package managers (like bower) that require we commit the compiled files.

https://github.com/jondavidjohn/payform/blob/master/bower.json#L3

jondavidjohn avatar Apr 12 '19 15:04 jondavidjohn

Right, I completely missed that @jondavidjohn! Thanks for 👀, we can undo that change @andygi

ddayguerrero avatar Apr 12 '19 16:04 ddayguerrero

@ddayguerrero , no problem I have reverted the commit.

andygi avatar Apr 12 '19 17:04 andygi