Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(Auto)Link Feature should respond to dropped text/uri-list, e.g., from browser's location bar #10464

Closed
mmichaelis opened this issue Sep 1, 2021 · 2 comments
Closed

(Auto)Link Feature should respond to dropped text/uri-list, e.g., from browser's location bar #10464

mmichaelis opened this issue Sep 1, 2021 · 2 comments
Labels
resolution:expired This issue was closed due to lack of feedback. status:stale type:feature This issue reports a feature request (an idea for a new functionality or a missing option).

Comments

@mmichaelis
Copy link

mmichaelis commented Sep 1, 2021 •
edited
Loading

📝 Provide a description of the new feature

If you drag and drop and link from the location bar of a browser, this link is written in raw format to CKEditor text.

Open https://example.org/. From the location bar, drag into CKEditor.

Expected: https://example.org/ is written to text, linking to https://example.org/
Actual: Only the text is written, without any linkHref attribute applied.

The data are provided in text/uri-list and may contain a list of URIs, and even comments according to RFC 2483, Section 5.

Note, that similar to this, #5152 proposes to detect images within this URI-list and embed an image instead. This feature may have to integrate with that image behavior.

If you'd like to see this feature implemented, add a 👍 reaction to this post.
@mmichaelis mmichaelis added the type:feature This issue reports a feature request (an idea for a new functionality or a missing option). label Sep 1, 2021
mmichaelis added a commit to CoreMedia/ckeditor-plugins that referenced this issue Sep 1, 2021
@mmichaelis
refactor(content-links): Remove Support for text/uri-list
6cd6029 
text/uri-list is from standard drag-data as for example from
dragging from browser's location bar to CKEditor. This should
not be handled as part of content-link plugin.

It should rather be part of CKEditor's Link Feature.

See-also: ckeditor/ckeditor5#10464
mmichaelis added a commit to CoreMedia/ckeditor-plugins that referenced this issue Sep 2, 2021
@mmichaelis
refactor(content-links): Remove Support for text/uri-list
3676575 
text/uri-list is from standard drag-data as for example from
dragging from browser's location bar to CKEditor. This should
not be handled as part of content-link plugin.

It should rather be part of CKEditor's Link Feature.

See-also: ckeditor/ckeditor5#10464
@CKEditorBot
Copy link
Collaborator

There has been no activity on this issue for the past two years. We've marked it as stale and will close it in 30 days. We understand it may be relevant, so if you're interested in the solution, leave a comment or reaction under this issue.

@CKEditorBot
Copy link
Collaborator

We've closed your issue due to inactivity over the last year. We understand that the issue may still be relevant. If so, feel free to open a new one (and link this issue to it).

@CKEditorBot CKEditorBot added the resolution:expired This issue was closed due to lack of feedback. label Oct 15, 2023
@CKEditorBot CKEditorBot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
resolution:expired This issue was closed due to lack of feedback. status:stale type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mmichaelis @CKEditorBot