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

[Preview app] Shows images that are not in the current file list #8932

Open
JammingBen opened this issue Apr 27, 2023 · 5 comments · May be fixed by #11960
Open

[Preview app] Shows images that are not in the current file list #8932

JammingBen opened this issue Apr 27, 2023 · 5 comments · May be fixed by #11960
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@JammingBen
Copy link
Collaborator

Steps to reproduce

  1. Upload 3 images to your personal space
  2. Create a public link for one of them
  3. Go to the Shared via link page and open the image with the preview app

Expected behaviour

There should only be this one image.

Actual behaviour

The app lets you switch between all 3 images form your personal space. The same goes for the Shared with others and the Search view.

@JammingBen JammingBen added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Apr 27, 2023
@kulmann kulmann moved this to Bugs Prio 3 or less in Web Support Board Jun 19, 2023
@jacob-nv jacob-nv self-assigned this Oct 9, 2023
@jacob-nv
Copy link
Contributor

jacob-nv commented Oct 9, 2023

@JammingBen Unable to replicate

@dschmidt
Copy link
Member

dschmidt commented Oct 9, 2023

I'm rather sure it should be still an issue.

It was one of the reasons the concept of folder loaders were introduced so the context for images could always be retrieved in especially the files app (and then we never got round to push it forward).

You basically need to find a place were an image is shown not from it's original location but from a "virtual" place like a share, search or maybe public link. Then reload the page so the context needs to be loaded and that should trigger the faulty behavior.

@JammingBen
Copy link
Collaborator Author

Yes, this is still an issue:

Bildschirmaufnahme.2023-10-09.um.17.15.59.mov

@jacob-nv
Copy link
Contributor

jacob-nv commented Oct 9, 2023

Ah, sorry, that's misunderstanding on my side.

@jacob-nv jacob-nv removed their assignment Oct 17, 2023
@kulmann kulmann moved this from Qualification to Prio 3 or less in Infinite Scale Team Board Oct 27, 2023
@fschade fschade self-assigned this Oct 29, 2023
@fschade fschade moved this from Prio 3 or less to In progress in Infinite Scale Team Board Oct 29, 2023
@JammingBen JammingBen moved this from In progress to Feature Requests in Infinite Scale Team Board Nov 7, 2023
@JammingBen
Copy link
Collaborator Author

Solution for this is being described here: #11893 (comment)

@JammingBen JammingBen moved this from Feature Requests to Prio 3 or less in Infinite Scale Team Board Nov 14, 2024
@JammingBen JammingBen self-assigned this Nov 15, 2024
@JammingBen JammingBen linked a pull request Nov 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
Status: Prio 3 or less
Status: Bugs Prio 3 or less
4 participants