jquery-editable-select icon indicating copy to clipboard operation
jquery-editable-select copied to clipboard

copyAttribute issue

Open albear-dev opened this issue 7 years ago • 6 comments

test : http://codepen.io/anon/pen/OXEmdr

When applying this... "Option" tag within a "value" attribute of the data is incorrectly copied. This is a problem that only occurs in IE.

ex) `

`

=> converted (Only IE) `

  • select1
` original option value data is "0200_111111" converted value data is "200"

albear-dev avatar Jul 27 '16 09:07 albear-dev

Hi @hyeonnykim, thank you for reporting this. It seems to be caused by jQuery's attr() method, it converts the string "0200_111111" into the integer 200... and I don't know why. Anyway, I found a fix for this issue and I'm going to patch it tomorrow with a new release of the plugin. Thank you again man! 👍

indrimuska avatar Aug 04 '16 10:08 indrimuska

Sorry, but I am not able to make it work. I thought I had a workaround but I was wrong and it seems it doesn't work with IE (tested on IE11). This is the plunker I've tested (check out the method copyAttributes on line 123), if you find some way to make it work, just let me know.

EDIT: There's a not-so-beautiful workaround that you can use in your application, adding a non blank string prefix to the option's value. If the value starts with a NON-NUMBER character (i.e. "v", "_", ...) then everything works very well. I think this is an internal bug of IE.

indrimuska avatar Aug 08 '16 22:08 indrimuska

I'm seeing this on Chrome too.

3noch avatar Aug 18 '16 19:08 3noch

@indrimuska It looks like you're copying the select options into a ul? The value attribute on li tags is always interpreted as a number, which might be the problem: http://www.w3schools.com/tags/att_li_value.asp

3noch avatar Aug 18 '16 19:08 3noch

Well, you're right, but I still get the correct value in Chrome:

Elements Console

indrimuska avatar Aug 19 '16 00:08 indrimuska

I see. I was using .val() so perhaps that was my problem.

3noch avatar Aug 19 '16 03:08 3noch