MakG10 / jquery-meme-generator

A jQuery plugin for interactive creating images with captions (memes) with basic image editing tools.
http://maciej.gierej.pl/en/jquery-meme-generator
59 stars 16 forks source link

Top and Bottom Text - Selection and Delete issue in browsers except Firefox #3

Closed hirenvjoshi closed 8 years ago

hirenvjoshi commented 8 years ago

I checked the demo but I am not able to select the text using Ctrl + A and even if I select it using mouse , I am not able to delete it using Del key. Even back arrow key is also not working. I can only delete the text using backspace.

This issue is not coming in Firefox but I checked in Chrome and Safari latest version and found this issue.

If possible, please let me know what is the issue.

Thanks

MakG10 commented 8 years ago

Hi. I can reproduce similar behaviour on Chrome, the Firefox seems to handling it fine in my case. I will investigate this issue later today, thanks for letting me know.

hirenvjoshi commented 8 years ago

Thanks for checking issue and replying promptly.

Thanks, Hiren Joshi

On Wed, Nov 9, 2016 at 3:29 PM +0530, "MakG10" notifications@github.com<mailto:notifications@github.com> wrote:

Hi. I can reproduce similar behaviour on Chrome, the Firefox seems to handling it fine in my case. I will investigate this issue later today, thanks for letting me know.

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/MakG10/jquery-memegenerator/issues/3#issuecomment-259375892, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHa30CsqCHFmdmk0Xfg9NGrZS0b43YgSks5q8ZmWgaJpZM4KsZyI.

MakG10 commented 8 years ago

The problem was due to forceUppercase option transforming text directly on value of input field. It's fixed now.

hirenvjoshi commented 8 years ago

Hmm. Seems like resolved. Good job.

Thanks, Hiren Joshi

On Thu, Nov 10, 2016 at 5:34 PM +0530, "MakG10" notifications@github.com<mailto:notifications@github.com> wrote:

The problem was due to forceUppercase option transforming text directly on value of input field. It's fixed now.

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/MakG10/jquery-memegenerator/issues/3#issuecomment-259673622, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHa30OK_V2w-1jA6fePrTMRTJ3cTav5Aks5q8wg8gaJpZM4KsZyI.