cheminfo / cheminfo/nmrium

Copy publication string is broken

Open
#3,960 7 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
80
Forks
34
Avg merge
2d 9h
Merged PRs (30d)
36

Description

From Ranges / Multiplet analysis

Reproduction steps:

* Go to https://dev.nmrium.org/#/SamplesDashboard/jro8bswrkba/Viewdatacytisinedatacytisine1Hjson
* Click on "Range picking and multiplet analysis" button in left toolbar
* Click on "Auto ranges picking" button in topbar
* Click on "Publication string" button in "Ranges / Multiplet analysis" panel
* Click on "Configure"
* In the opened dialog, click on the clipboard button (in preview area, it float at right)
* Try to paste it in a plain text file, textarea. It paste nothing (and if you had something in your clipboard before, it's gone).

Supposition:

Publication string is generated with `rangesToACS` from `nmr-processing`, it returns a string compatible with html for formatting (ex: `` node). It may interfere with clipboard api, maybe it needs to be copy with a `` parent node.

---

I forgot to mention, copy actions seems broken (at least on zen browser). But it was broken before my refactor during dev.

I will investigate from main branch before we merge and raise an issue if needed.

_Originally posted by @tpoisseau in https://github.com/cheminfo/nmrium/issues/3958#issuecomment-3848135665_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.