Skip to content

Conversation

@jakobroehrl
Copy link
Contributor

@jakobroehrl jakobroehrl commented Jul 15, 2020

Signed-off-by: Jakob Röhrl [email protected]
#277

grafik
grafik
grafik

Signed-off-by: Jakob Röhrl <[email protected]>
@jakobroehrl jakobroehrl added enhancement New feature or request 2. developing Work in progress labels Jul 15, 2020
@jakobroehrl jakobroehrl linked an issue Jul 17, 2020 that may be closed by this pull request
Signed-off-by: Jakob Röhrl <[email protected]>
@jakobroehrl jakobroehrl added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jul 17, 2020
@jakobroehrl jakobroehrl requested a review from skjnldsv July 17, 2020 07:10
@jakobroehrl jakobroehrl added this to the Nextcloud 20 milestone Jul 17, 2020
Signed-off-by: Jakob Röhrl <[email protected]>
Comment on lines +139 to +146
async onlyPhotos() {
// reset component
this.resetState()
// content is completely different
this.$emit('update:loading', true)
this.fetchContent()
},
Copy link
Member

@skjnldsv skjnldsv Jul 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could maybe merge the three duplicate functions into one method?

@jancborchardt
Copy link
Member

With having a specific video view, the issue of the videos not having previews becomes a much bigger problem.

Is it possible to get any kind of preview at all out of the videos here? Otherwise a specific video view doesn’t seem that much of an enhancement, if it’s just all loading filetype icons and then you need to get lucky to find the correct video. ;)

(Actually it makes it a less good UX since it pushes a shortcoming of Photos into the forefront.)

@jakobroehrl
Copy link
Contributor Author

With having a specific video view, the issue of the videos not having previews becomes a much bigger problem.

Is it possible to get any kind of preview at all out of the videos here? Otherwise a specific video view doesn’t seem that much of an enhancement, if it’s just all loading filetype icons and then you need to get lucky to find the correct video. ;)

(Actually it makes it a less good UX since it pushes a shortcoming of Photos into the forefront.)

On my productive system I have no problems with missing video previews. It's only on the dev system.

@skjnldsv
Copy link
Member

Is it possible to get any kind of preview at all out of the videos here?

Nope, it all depends on your setup. If you have video previews properly setup :/
https://www.allerstorfer.at/nextcloud-install-preview-generator/

@jakobroehrl
Copy link
Contributor Author

jakobroehrl commented Jul 21, 2020

Todo:
We want to add a notice on the video pane if not installed:

consider installing the app preview generator for showing video previews here

@jancborchardt
Copy link
Member

@jakobroehrl rather, I would say to only show the split "Your photos" and "Your videos" view when that preview generator is installed and configured.

If it is not, then the videos view is not so useful and we only need the Timeline, and the notice about previews can be shown to admins either:

  • On top of the pictures
  • In the bottom left settings area
  • On the bottom part of the navigation

@tacruc
Copy link

tacruc commented Aug 10, 2020

Note: The video preview might be disabled even if the preview generator app is enabled.
I had quite a fight today getting it to work, even reading all the docs. As everything, which was mentioned in all docs was installed, but still video preview was not available. Therefore I think the hint of just install this one app might lead to more frustration, than it helps.

The underling problem was, that most video codecs are not opensource and therefore not shipped with many distributions. Even installing ffmpeg is not sufficient, if the codecs are missing.
But how many and which codecs have to be installed to consider video previews as enabled? Is webm sufficient? Or webm and H.263.

In short I think it will be hard to check if video previews are available and working on a system.

@skjnldsv
Copy link
Member

In short I think it will be hard to check if video previews are available and working on a system.

@rullzer is this feasible to create a setting that checks each provider and if it's properly working?
Maybe that would be cool to be able to narrow and debug things so we know what preview provider is enabled and which one of them is not working (I also have the same issue, video enabled, but not generating previews, though I did not debug further yet)

@MorrisJobke MorrisJobke mentioned this pull request Sep 10, 2020
13 tasks
@rullzer
Copy link
Member

rullzer commented Sep 10, 2020

In short I think it will be hard to check if video previews are available and working on a system.

@rullzer is this feasible to create a setting that checks each provider and if it's properly working?
Maybe that would be cool to be able to narrow and debug things so we know what preview provider is enabled and which one of them is not working (I also have the same issue, video enabled, but not generating previews, though I did not debug further yet)

Not really. You actually need to encode something and check if the result is correct. Which is almost impossible.

@rullzer rullzer removed this from the Nextcloud 20 milestone Sep 10, 2020
@jakobroehrl
Copy link
Contributor Author

Sad, but then let's close this PR

@jancborchardt jancborchardt deleted the enh/my-videos branch October 16, 2020 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Filter to show only Videos

6 participants