[GH-ISSUE #2199] [FR] Handle invalid URLs when formatting text as a link #899

Closed
opened 2026-03-23 20:42:48 +00:00 by mirror · 12 comments
Owner

Originally created by @annieappflowy on GitHub (Apr 5, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2199

Description

On the link edition/creation dialog you can type anything and it's still accepted, it should only accept URL (from web URL to system (I don't know how they are called) URL like mailto, call, etc.)

In the link editing menu, we should let the user know "Type a valid URL to link"

Impact

Users who format text as links

Additional Context

No response

Originally created by @annieappflowy on GitHub (Apr 5, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2199 ### Description On the link edition/creation dialog you can type anything and it's still accepted, it should only accept URL (from web URL to system (I don't know how they are called) URL like mailto, call, etc.) In the link editing menu, we should let the user know "Type a valid URL to link" ### Impact Users who format text as links ### Additional Context _No response_
Author
Owner

@GouravShDev commented on GitHub (Apr 5, 2023):

I would like to work on this issue.

<!-- gh-comment-id:1497264767 --> @GouravShDev commented on GitHub (Apr 5, 2023): I would like to work on this issue.
Author
Owner

@rwt-2001 commented on GitHub (Apr 5, 2023):

@GouravShDev would you let me work on this issue?

<!-- gh-comment-id:1497321325 --> @rwt-2001 commented on GitHub (Apr 5, 2023): @GouravShDev would you let me work on this issue?
Author
Owner

@GouravShDev commented on GitHub (Apr 5, 2023):

@rwt-2001 sure.

<!-- gh-comment-id:1497327450 --> @GouravShDev commented on GitHub (Apr 5, 2023): @rwt-2001 sure.
Author
Owner

@annieappflowy commented on GitHub (Apr 5, 2023):

Thanks @GouravShDev. I think there are other interesting issues to take :D
Assigned it to @rwt-2001

<!-- gh-comment-id:1497549617 --> @annieappflowy commented on GitHub (Apr 5, 2023): Thanks @GouravShDev. I think there are other interesting issues to take :D Assigned it to @rwt-2001
Author
Owner

@Xazin commented on GitHub (Apr 23, 2023):

The PR linked to this does not change any behavior inside the link menu, are we sure this is completed?

<!-- gh-comment-id:1519031357 --> @Xazin commented on GitHub (Apr 23, 2023): The PR linked to this does not change any behavior inside the link menu, are we sure this is completed?
Author
Owner

@Chandan-CV commented on GitHub (May 24, 2023):

@annieappflowy @Xazin, if no one working on this, can I take it up? 😃

<!-- gh-comment-id:1560482018 --> @Chandan-CV commented on GitHub (May 24, 2023): @annieappflowy @Xazin, if no one working on this, can I take it up? :smiley:
Author
Owner

@Chandan-CV commented on GitHub (May 31, 2023):

@annieappflowy Can you assign me this issue?

<!-- gh-comment-id:1569477797 --> @Chandan-CV commented on GitHub (May 31, 2023): @annieappflowy Can you assign me this issue?
Author
Owner

@annieappflowy commented on GitHub (Jul 11, 2023):

Sorry for the late response. Are you still keen to work on it? @Chandan-CV

<!-- gh-comment-id:1630877254 --> @annieappflowy commented on GitHub (Jul 11, 2023): Sorry for the late response. Are you still keen to work on it? @Chandan-CV
Author
Owner

@AdiAr11 commented on GitHub (Jul 16, 2023):

Hi @annieappflowy , can i work on this? Also what all changes are required? Like should i change the title from 'Add your link' to 'Type a valid URL to link' and change the functionality by first checking if the link is valid before clicking entering or either of the two?
Thank you!
Screenshot 2023-07-16 at 3 13 13 PM

<!-- gh-comment-id:1637038441 --> @AdiAr11 commented on GitHub (Jul 16, 2023): Hi @annieappflowy , can i work on this? Also what all changes are required? Like should i change the title from 'Add your link' to 'Type a valid URL to link' and change the functionality by first checking if the link is valid before clicking entering or either of the two? Thank you! <img width="894" alt="Screenshot 2023-07-16 at 3 13 13 PM" src="https://github.com/AppFlowy-IO/AppFlowy/assets/54585121/4a8b0087-55be-4298-8e9c-3cf57eb0274c">
Author
Owner

@annieappflowy commented on GitHub (Aug 11, 2023):

Hi @AdiAr11 , are you available to work on this?

<!-- gh-comment-id:1674230659 --> @annieappflowy commented on GitHub (Aug 11, 2023): Hi @AdiAr11 , are you available to work on this?
Author
Owner

@AdiAr11 commented on GitHub (Aug 11, 2023):

@annieappflowy Yes, I am available!

<!-- gh-comment-id:1674231306 --> @AdiAr11 commented on GitHub (Aug 11, 2023): @annieappflowy Yes, I am available!
Author
Owner

@AdiAr11 commented on GitHub (Sep 28, 2023):

@annieappflowy I think the following PR fixes this issue.
https://github.com/AppFlowy-IO/appflowy-editor/pull/324

<!-- gh-comment-id:1739417806 --> @AdiAr11 commented on GitHub (Sep 28, 2023): @annieappflowy I think the following PR fixes this issue. [https://github.com/AppFlowy-IO/appflowy-editor/pull/324](https://github.com/AppFlowy-IO/appflowy-editor/pull/324)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#899
No description provided.