nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
27.42k stars 4.07k forks source link

Is scrollto deprecated? #46113

Open mahoneyr opened 4 months ago

mahoneyr commented 4 months ago

⚠️ This issue respects the following points: ⚠️

Bug description

Is the &scrollto function deprecated? I fed the same URL to the default Docker containers for v27 and v28: http://nextcloud/apps/files/files?dir=/&scrollto=Readme.md

Steps to reproduce

In v27 it re-wrote the URL as http://nextcloud/apps/files/files?dir=/&fileid=2 and highlighted it the file Screenshot 2024-06-25 093822

In v28 it didn't re-write the URL or highlight it the file: Screenshot 2024-06-25 093846

Expected behavior

The v27 behavior where the scrollto highlights the selected item would be more useful for apps like fulltextsearch that still employ it.

Installation method

Community Docker image

Nextcloud Server version

27

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

Fresh Nextcloud Server install

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

cblte commented 4 months ago

Damn it. I do not have any of the two. Running Nextcloud 29

mahoneyr commented 4 months ago

Nextcloud 29 seems to follow the 28 behavior.

rokmarko commented 2 months ago

I can confirm the same problem on 29.0.4.