nextcloud / text

📑 Collaborative document editing using Markdown
GNU Affero General Public License v3.0
540 stars 87 forks source link

Display links inserted from smart picker as page title/file name #6180

Open NikoKS opened 1 month ago

NikoKS commented 1 month ago

This request has been made in collectives Issues page https://github.com/nextcloud/collectives/issues/949, but was mistakenly addressed by https://github.com/nextcloud/text/issues/5838.

In short, it would be better when links added by smart picker to another pages to have the title/file name of the page picked. This is similar to the behavior when referring to a github page from another text. For example, I want to refer an issue in a sentence like #5838, instead of embedding an entire preview of the page in another line.

Having a preview of the page would make sense when the link is not in the middle of a sentence. But when it is, it would be better to only show the title.

juliushaertl commented 3 weeks ago

I think this makes sense. I looked into inserting inline links automatically with their actual title in the past (still have a wip commit in https://github.com/nextcloud/text/commit/c98c8f498eabe73e022babc1d442ec0e04356321), but back then it was tricky as we needed a separate request to get the metadata from the smart picker API, maybe this works better now, so we would not need the whole logic for a separate node type there.

NikoKS commented 1 week ago

Hello @juliushaertl, it's great to hear that it might be implementable now! Is there any plan on implementing it?