-
Notifications
You must be signed in to change notification settings - Fork 53
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
Unable to edit photos; redirected back to the file list (NC 29 & 30) #2323
Comments
confirm, memories works |
This comment has been minimized.
This comment has been minimized.
confirm it. 😀 |
the errormessage from above is also applied in NC 27.5.1 but there the fotoapp reacts correct |
i took the viewer app from a nextcloud 27 , edited the info.xml in the appinfo-folder (max-version="29") and then (after an updatemessage) it worked ok. so it seam to be the viewer app maybe it is time to contact the author of the viewer app the content of folder js in the app looks strange |
i replaced all in js folder with the content of the js folder from the nc27 version after an update message everything runs ok in nc 29 in a fresh installation nc 29.0.1 , all apps deactivated, a click on a jpg shows it but will not load the editor if clicked on the pencil. |
the image editor (*) fails to load in nc 29. implementatin of filerobot image editor https://scaleflex.github.io/filerobot-image-editor/ read here https://nextcloud.com/blog/how-we-got-to-the-new-nextcloud-photos/ |
Reproduced. Editing is provided by Viewer not the Photos app. Transferred to that repo. |
Still broken in NC 30.0.0 beta 5 (tested on c.nc.com) Weird also: after unsuccessfully trying to edit and getting thrown back to files list, and clicking an image again, you see the error "no plugin avialable to view this file type" (German "Es ist kein Plugin zum Anzeigen dieses Dateityps verfügbar") |
Without more in-depth investigation: the problem seems to be related to the bundling/module-loading. The error is not reproducible in a dev environment with In production one can observe that I assume that the viewer code loads and executes a second time, this time with no handlers being registered through Vite does not seem to be aware of the added cachebuster when lazy-importing (Edits: clarifications, formatting) |
I'm not familiar enough with the setup and Vite's bundling internals, and I am happy to attempt fixing it if someone could give me some hints here. |
This comment has been minimized.
This comment has been minimized.
At the moment it does not look like this error will be fixed. I therefore suggest removing the pencil icon (edit). |
Description
When accessing the Photos app at http://[nextcloud]/apps/photos/ and clicking on the viewer to view images, the interface allows for normal interaction. However, attempting to edit an image by clicking the pencil icon results in being redirected back to the file list screen instead of opening the editing interface.
Error Message
Console in Browser : [Deprecation] Listener added for a 'DOMNodeInserted' mutation event. This event type is deprecated, and will be removed from this browser very soon. Usage of this event listener will cause performance issues today, and represents a large risk of future site breakage. Consider using MutationObserver instead. See https://chromestatus.com/feature/5083947249172480 for more information.
This issue occurs consistently across different browsers including Edge and Chrome on Windows, and Safari on iPad.
Steps to Reproduce
Expected behavior
The expected behavior is to transition to the photo editing interface as was the case in previous version 28.0.3.
Actual behavior
Clicking the edit icon redirects back to the photo file list, and a deprecation warning is logged in the console.
Environment
Additional context
This issue prevents any editing functionality within the Photos app, which was functional in the previous version 28.0.3. The same behavior is observed across multiple platforms and browsers indicating a possible issue with the app's handling of deprecated browser features.
The text was updated successfully, but these errors were encountered: