fix(NcRichText): show links title for desktop client
☑️ Resolves
- Fix https://github.com/nextcloud/talk-desktop/issues/320
🖼️ Screenshots
| 🏚️ Before | 🏡 After |
|---|---|
🏁 Checklist
- [ ] ⛑️ Tests are included or are not applicable
- [ ] 📘 Component documentation has been extended, updated or is not applicable
- [ ] 3️⃣ Backport to
nextrequested with a Vue 3 upgrade
Should we do it all the time (without props)? Or if href doesn't match content?
Should we do it all the time (without props)? Or if href doesn't match content?
Then it duplicates already existing web-browser feature
Or if href doesn't match content?
I say we keep the same for all links
Then it duplicates already existing web-browser feature
True. I couldn't find any related options or preferences in Electron app, but that anyway should be done in talk-desktop repo, I believe. Another point of doing that is that we have links outside of NcRichText (as conversation links, settings redirects)
Maybe we can implement it with a custom solution, like:
- add listener on hover
- if it's a link, draw a preview box in the corner
Then it duplicates already existing web-browser feature
Sounds like we do not want this? If I am wrong please reopen