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

[FEATURE REQUEST] Sync action in previews #3155

Closed
jesmrec opened this issue Mar 19, 2021 · 1 comment · Fixed by #3177
Closed

[FEATURE REQUEST] Sync action in previews #3155

jesmrec opened this issue Mar 19, 2021 · 1 comment · Fixed by #3177

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented Mar 19, 2021

There are four kind of files that are previewed inside the app:

  • Images
  • Text (.txt)
  • Video
  • Audio

Actually, the preview mode provides different actions, depending on the kind of file. We should review and unify actions for every kind of file.

This is the current status, that makes sense.

Sync Share Send Open with Av. off Details Delete
Image
Text
Video (streaming)
Video (downloaded)
Audio

The main difference is that the Audio previews allow to sync the file.

Should the sync option be removed from audio? or added in the other ones.

In my opinion, i'd remove it from audio. What do you think?

@michaelstingl
Copy link
Contributor

In my opinion, i'd remove it from audio. What do you think?

Syncing action should be sufficient on the file list entry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants