Copy functionality not working properly in chrome.

[sujeetvkulkarni]

Describe the bug
Copy functionality of GlycoCT sequence does not propelry in all browsers, specifically chrome.

/GlycoGlyphPublic/src/modules/copyTextFromElement.js
/GlycoGlyphPublic/public/js/glycoglyph.js

copyText() function should return promise object and copyTextFromElement should wait on the object. I have added modified code below,

//If you want to copyText from Element
function copyTextFromElement(elementID) {
let element = document.getElementById(elementID); //select the element

let elementText = element.innerText; //get the text content from the element

if (element.tagName == 'INPUT' ) {
  elementText = element.value;
}

elementText = elementText.trim();

if (elementText != '') {
  //use the copyText function below
  copyText(elementText).then(() => {
    alert('Copied: \n' + elementText);
  })
  .catch(() => {
    alert('Could not copy text.');
  });
}else {
  alert('Nothing to copy.');
}

}

//If you only want to put some Text in the Clipboard just use this function
// and pass the string to copied as the argument.
function copyText(text) {
return navigator.clipboard.writeText(text);
}

[akulmehta]

Hi @sujeetvkulkarni. I cannot reproduce the error in chrome.

Example: https://glycotoolkit.com/Tools/GlycoGlyph/

[sujeetvkulkarni]

Hi @akulmehta copy pop-up comes but if you try to paste the copied GlycoCT string on text editor it was not working.

[akulmehta]

Sorry - can't reproduce yet. It copies it fine for me. I wonder if this is OS related - I am on Windows 10.

[sujeetvkulkarni]

Maybe I am on macOS Monterey Version 12.1 and Chrome Version 100.0.4896.75 (Official Build) (x86_64).

…

On Thu, Apr 7, 2022 at 1:36 PM Akul Mehta ***@***.***> wrote: Sorry - can't reproduce yet. It copies it fine for me. I wonder if this is OS related - I am on Windows 10. — Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADJ7PGXZRAMH27E5VTZCMNTVD4MKHANCNFSM5R3TT6UA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[akulmehta]

Does the code you proposed fix this for you?

[sujeetvkulkarni]

Yes, navigator.clipboard.writeText(text); function returns a promise object and we should handle it with then construct before proceeding. That's what the proposed code does.

…

On Thu, Apr 7, 2022 at 1:45 PM Akul Mehta ***@***.***> wrote: Does the code you proposed fix this for you? — Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADJ7PGWHQ5OVDKHYQADGBMDVD4NLDANCNFSM5R3TT6UA> . You are receiving this because you were mentioned.Message ID: ***@***.***>