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

Links are not visible anymore #9163

Closed
nickvergessen opened this issue Mar 24, 2023 · 5 comments · Fixed by nextcloud-libraries/nextcloud-vue#3932
Closed

Links are not visible anymore #9163

nickvergessen opened this issue Mar 24, 2023 · 5 comments · Fixed by nextcloud-libraries/nextcloud-vue#3932

Comments

@nickvergessen
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Post a link in chat

Expected behaviour

Link is clearly visible (underlined, mouse effect, etc)

Actual behaviour

grafik

Talk app

**Talk app version:**16.0.1

@nickvergessen
Copy link
Member Author

Even worse, "local links" seem to not link at all anymore, e.g. https://192.168.178.101/index.php/apps/spreed/ and https://127.0.0.1/status.php are not clickable anymore...

@nickvergessen
Copy link
Member Author

Also happens on https://nextcloud-vue-components.netlify.app/#/Components/NcRichText?id=ncrichtext-1 so checking upstream for differences between nextcloud-vue-richtext and the new component in the vue library

@nickvergessen
Copy link
Member Author

nickvergessen commented Mar 27, 2023

Works on stable25 of talk but not on master docs of the lib https://nextcloud.github.io/vue-richtext/

So must be a regression of https://github.com/nextcloud/vue-richtext/compare/v2.0.4..v2.1.0-beta.6

cc @julien-nc

@nickvergessen
Copy link
Member Author

Before the update links are with class "rich-text--external-link" with the features added they have class "rich-text--component" and some links don't resolve anymore.

@nickvergessen
Copy link
Member Author

Even worse, "local links" seem to not link at all anymore, e.g. https://192.168.178.101/index.php/apps/spreed/ and https://127.0.0.1/status.php are not clickable anymore...

Moved to #9187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant