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

Can not go back from the files list anymore #1623

Closed
nickvergessen opened this issue Oct 5, 2016 · 1 comment
Closed

Can not go back from the files list anymore #1623

nickvergessen opened this issue Oct 5, 2016 · 1 comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: files papercut Annoying recurring issue with possibly simple fix.

Comments

@nickvergessen
Copy link
Member

Steps to reproduce

  1. Open activity app or settings or ...
  2. Open files app
  3. Hit the "back" button of your browser multiple times

Expected behaviour

Should be activity again

Actual behaviour

Staying on the files app

The problem is, that you have to "jump over" the http://localhost/index.php/apps/files/ step. However, that one always pushes a new entry http://localhost/index.php/apps/files/?dir=/&fileid=44 to the history, so the next "back" brings you to apps/files/ again.

Fix

The redirect from apps/files/ to apps/files/?dir=/&fileid=44 should not push to history, or at least not when we are coming back.

On Nextcloud 10 you can do the jump when you are fast enough. But on master it looks impossible to me.
@nickvergessen nickvergessen added bug feature: files papercut Annoying recurring issue with possibly simple fix. labels Oct 5, 2016
@nickvergessen nickvergessen added this to the Nextcloud 10.0.2 milestone Oct 5, 2016
@MorrisJobke MorrisJobke removed this from the Nextcloud 10.0.5 milestone Apr 5, 2017
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@nextcloud-bot nextcloud-bot mentioned this issue Jul 24, 2018
Closed
@nextcloud-bot nextcloud-bot mentioned this issue Aug 7, 2018
Closed
@skjnldsv skjnldsv added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Jun 12, 2019
@ghost ghost removed the stale Ticket or PR with no recent activity label Jun 12, 2019
@wiswedel
Copy link
Contributor

closing because super-stale and NC version no longer supported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: files papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@nickvergessen @MorrisJobke @wiswedel @skjnldsv @nextcloud-bot