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.

Sign up for GitHub

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

Jump to bottom

[bug] The URL gets changed on prev / next #2567

Closed
Tracked by #5249
susnux opened this issue Sep 26, 2024 · 0 comments · Fixed by #2605
Closed
Tracked by #5249

[bug] The URL gets changed on prev / next #2567

susnux opened this issue Sep 26, 2024 · 0 comments · Fixed by #2605
Assignees
@artonge
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Milestone
Nextcloud 30.0.2

Comments

@susnux
Copy link
Contributor

susnux commented Sep 26, 2024

  1. Open a folder in files that contains images
  2. Open a file
  3. See URL looks like apps/files/files/123?dir=/review&openfile=true
  4. Click on Next or Previous
  5. See that the URL changes to /apps/files/?openfile=true and in the background the files app navigates to a different folder

On close the correct folder is restored, but this causes unnecessary requests in the background and in unexpected behavior.
@susnux susnux added the 1. to develop Accepted and waiting to be taken care of label Sep 26, 2024
@susnux susnux added this to the Nextcloud 30.0.1 milestone Sep 26, 2024
@susnux susnux added the bug Something isn't working label Oct 16, 2024
@susnux susnux moved this to 📄 To do (~10 entries) in 📁 Files team Oct 16, 2024
@susnux susnux self-assigned this Oct 23, 2024
@susnux susnux mentioned this issue Oct 31, 2024
Closed
4 tasks
@szaimen szaimen mentioned this issue Oct 31, 2024
Closed
34 tasks
@artonge artonge mentioned this issue Nov 4, 2024
Merged
@susnux susnux assigned artonge and unassigned susnux Nov 4, 2024
@github-project-automation github-project-automation bot moved this from 📄 To do (~10 entries) to ☑️ Done in 📁 Files team Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@artonge artonge

Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Projects
📁 Files team
Status: ☑️ Done
Milestone
Nextcloud 30.0.2
Development

Successfully merging a pull request may close this issue.

fix: Usage of pushToHistory function on prev/next nextcloud/viewer
2 participants
@susnux @artonge