-
Notifications
You must be signed in to change notification settings - Fork 94
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
URLs in heading do not work in readonly mode #3822
Comments
This issue also exists in readonly shares of Text app, thus transfering it to nextcloud/text. |
For some reason, |
The reason for this bug is that our heading anchor links span over the whole headline in readonly mode. That's why they overlay possible links in the heading. So I see two options here: a) Limit width of heading anchor to I'm in favour of option a. What do others think @susnux @max-nextcloud @juliushaertl? Also @jancborchardt and @nimishavijay? |
I am also in favor of option A: Users will not be disappointed by not working links in headers (as they can be added they should work). |
In readonly mode, the heading anchor links spanned the whole width so far, overlaying potential links in headings. Fixes: #3822 Signed-off-by: Jonas <[email protected]>
In readonly mode, the heading anchor links spanned the whole width so far, overlaying potential links in headings. Fixes: #3822 Signed-off-by: Jonas <[email protected]>
In readonly mode, the heading anchor links spanned the whole width so far, overlaying potential links in headings. Fixes: #3822 Signed-off-by: Jonas <[email protected]>
Describe the bug
URLs in headings do not work in read-only mode. If you click at them nothing happens. If you open the same file with write permissions in the Text-App via Files everything works as expected.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I would expect a new browser tab to be opened with the URL as it ist if you open it from within the Text-App.
Client details:
Server details
Collectives app version: (see Nextcloud apps page)
2.2.1
Operating system:
Web server:
Database:
PHP version:
reproduced the bug on different servers.
Nextcloud version: (see Nextcloud admin page)
25.0.3
The text was updated successfully, but these errors were encountered: