Comments (22)
I'll try to contribute a vision for the future.
Current state: Viewer/Photos does provide a timeline view of my photos and videos, covering all files I have access to. There is already a world map plugin, but no exif information panel.
Vision: I'll envision a basic (!) Google Photos alternative. The mobile Nextcloud client already allows me to upload my photos/videos automatically and the Nextcloud Photos app provides a timeline and world map view.
- The timeline view should be tied to a specific folder (optionally all folders), to not show all my media files (as in: do not show my work stuff)
- There should be a panel/information menu, providing basic exif information (like location, date/time, camera, etc.)
- There should be basic image editing features (like rotation, cropping, ...)
- Possibly extended by additional image editing features (like brightness, saturation, contrast, filters, ...)
As far as I can see the image editing tool TUI, recommended in #65 (comment) , doesn't need a server-side part, so the only things to code would be a matching theme (optional), a way to open existing images and a way to save the resulting image (either by overriding the existing file or by saving a copy).
This would be truly amazing, as there is no good (open-source) alternative to Google Photos. The only okayish tool I found so far is Synology Photos, which is okay (good design, timeline view), but is proprietary and doesn't provide any image editing features, like cropping. ... to quickly edit an image is really handy and this is what I miss the most, compared to Google Photos
from viewer.
Should be permanent, that is save the image when it's rotated
This is out of scope though.
I would say making the change persistent is most important, as otherwise it’s quite confusing. Non-persistent rotation doesn’t really make sense and would clash with expectations.
So I’d say until we figure out the way to make this persistent, no need to invest in putting in elements to rotate, as user expectations will be that it’s persistent. Better not having it than not having it up to expectations. ;)
from viewer.
I see that this is set as milestone, which means that it is not done. Why is the issue closed?
I suspect that feature from this issue was just implemented by image editor so it may be closed and marked as complete.
from viewer.
I see that this is set as milestone, which means that it is not done. Why is the issue closed? I see that another issue was opened, but most people interested in this feature who monitor this issue will not get notified when the new issue will be completed.
Look again, #1317 is not another issue but the implemented feature.
This will be part of Nextcloud 25, as the milestone says, and this being closed as "completed"
from viewer.
@jancborchardt shall we?
I feel like this is a bit out of scope!
@ncgu thanks for the suggestion. :)
from viewer.
I wouldn't say light image editing is out of scope, and rotation is really basic.
Of course some points:
- Rotation only makes sense for images
- It does not need to be so present, in the 3-dot menu is enough
- Can be one entry/row, or even just "Rotate right"
- Should be permanent, that is save the image when it's rotated
from viewer.
- Should be permanent, that is save the image when it's rotated
This is out of scope though.
The viewer is not made to have any kind of actions on the file really. 🤔
This would also means we basically implement a full image processing in the front or the back and upload/update the file again. I'm not sold 😁
Ok for the rest :)
from viewer.
* It does not need to be so present, in the 3-dot menu is enough
OK. Is it possible to put the rotate function (+90/-90) on the r and l keys?
from viewer.
@ncgu are those keys usually bound to rotate?
Any software in mind where they do it?
from viewer.
https://www.irfanview.com/
I've been using it for years.
really cool
from viewer.
And any popular ones? Because I never heard of irfanview :)
The idea when adding keyboard shortcut is to have universally known keys binding. 😉
from viewer.
Regararding persisting the rotation, please also see nextcloud/gallery#490 for some details on how that could be achieved. But I agree with @skjnldsv That the viewer should really be about viewing, if we add any editing features, we should have that handled in a spearate app, that can maybe register their editing actions in the viewer somehow.
from viewer.
that can maybe register their editing actions in the viewer somehow.
definitely :)
When #7 is in, any app that register the actin in files will appear here !
from viewer.
Btw @ncgu looks like l & r keys were also going to be used on the gallery app:
https://github.com/nextcloud/gallery/pull/490/files#diff-5e4ddda58d290ec339ec9e57444801ddR271 :)
Let's go for them!
from viewer.
Refs:
- https://github.com/nhn/tui.image-editor https://ui.toast.com/tui-image-editor/
- https://github.com/SeregPie/VuetifyImageInput https://seregpie.github.io/VuetifyImageInput/
https://github.com/imgly/imgly-sdk-html5 https://staging.photoeditorsdk.com/html5-demo/
from viewer.
Just to add my 2¢, along with the standard rotate, I'd also like the the ability to pixilate/blur selections; rotate and pixelate/blur are the 2 most common things I use/need to do when doing quick edits for posting online. A drop-in toolbar type of editor like mentioned in #925 would be perfect in my case. =)
from viewer.
@rmeissn I do get @skjnldsv technical point of view: this is not a vision for Nextcloud Viewer/Photos, this is a vision for Nextcloud!
I do share your desire, yet it is true that basic image editing capabilities should be available also in other apps (for example: nextcloud/contacts#1662 ).
In addition your vision integrates functionality that already exists or has been requested in other apps:
- we have face recognition and more thanks to Recognize (https://apps.nextcloud.com/apps/recognize ) and Face Recognition
- metadata reading, writing and displaying ability has been discussed since a bit (nextcloud/photos#226 ) and it could/should integrate in the server with other types of metadata (audio, video, bibliographic, ...)
- an interface to better search and display information that Nextcloud already has (like e.g. the geographic position of photos using the Maps app) was asked here: nextcloud/photos#445
I do hope that this recap will help the discussion
from viewer.
Unfortunately the viewer does not seem to respect the EXIF information. Rotated the photo in GIMP or GWENVIEW for example and the Nextcloud viewer still sees it not rotated. GIMP and GWENVIEW agree with each other
from viewer.
hi,
so rotation of the images still no avail? nextcloud/gallery#490
also assuming there is no working metadata reading, writing and displaying ability has been discussed since a bit?
thanks
from viewer.
Feature implemented/completed in #1317
from viewer.
I see that this is set as milestone, which means that it is not done. Why is the issue closed?
I see that another issue was opened, but most people interested in this feature who monitor this issue will not get notified when the new issue will be completed.
from viewer.
Thanks. Perhaps adding "Feature implemented/completed in #1317" would have made things clearer.
from viewer.
Related Issues (20)
- Opening a video on mobile prevents switching to fullscreen HOT 4
- Editor UI is broken on 29/stable28 HOT 4
- [Bug]: Can't play video since last update HOT 1
- disable previous / next video preloading HOT 8
- Do not save edited photos as new files
- Sandwich menu is not working when editing
- Partial localisation of the app
- Wrong tile of the edited file when saving if contains Cyrillic symbols
- Broken font CSS style
- Close photo viewer when clicking outside the photo itself.
- Sync timecode with server and add a function to resume playback where we left it
- Unable to edit photos; redirected back to the file list (NC 29 & 30) HOT 10
- [App Photo] Lag on rotate animation in editor HOT 2
- No "Edit" button on mobile HOT 2
- [Bug]: Cannot open video files shot on phone camera on web
- Add rename action to file details page HOT 1
- Slideshow continues to slide files while editing
- Version name can't be set via fullscreen+sidebar HOT 1
- Download of images fails
- [RFC] Viewer 3.0 API HOT 16
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from viewer.