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.

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

remove old link when editing #5690

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

grnd-alt
Copy link
Member

@grnd-alt grnd-alt commented Apr 15, 2024

Fixes #5516

@grnd-alt grnd-alt linked an issue Apr 15, 2024 that may be closed by this pull request
@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Apr 16, 2024
@juliusknorr juliusknorr added this to the Nextcloud 30 milestone Apr 16, 2024
@max-nextcloud
Copy link
Collaborator

The current change works well when only changing a single link in a paragraph.
There are two more scenarios that would be good to account for:

Link in a paragraph

Find more information on https://nextcloud.com as usual.

-> should only change the link.

Link with custom text

More informarion

-> should only change the href.

@max-nextcloud max-nextcloud self-assigned this Apr 17, 2024
@max-nextcloud max-nextcloud force-pushed the fix/5516-updating-the-url-of-a-link-via-toolbar-menu-splits-the-link branch 2 times, most recently from 10bfdeb to 35f1d07 Compare April 17, 2024 14:17
@skjnldsv skjnldsv mentioned this pull request Aug 22, 2024
44 tasks
@blizzz blizzz modified the milestones: Nextcloud 30, Nextcloud 31 Sep 4, 2024
@blizzz blizzz mentioned this pull request Jan 29, 2025
1 task
@blizzz blizzz modified the milestones: Nextcloud 31, Nextcloud 32 Jan 29, 2025
Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I said earlier... this will not work if the link is not the only thing in a paragraph.

@grnd-alt would you be up for picking this up again?

@max-nextcloud max-nextcloud removed their assignment Feb 3, 2025
@grnd-alt grnd-alt force-pushed the fix/5516-updating-the-url-of-a-link-via-toolbar-menu-splits-the-link branch from f9e4df3 to ab4b9d0 Compare February 4, 2025 13:19
@grnd-alt grnd-alt requested a review from mejo- as a code owner February 4, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing bug Something isn't working
Projects
Status: 🏗️ In progress
Development

Successfully merging this pull request may close these issues.

Updating the URL of a link via toolbar menu splits the link
4 participants