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

Video instead of photo for teaser causes fatal error #48

Open
benjitavos opened this issue Jul 16, 2014 · 6 comments
Open

Video instead of photo for teaser causes fatal error #48

benjitavos opened this issue Jul 16, 2014 · 6 comments

Comments

@benjitavos
Copy link

On Arlima Manage list tab, after selecting an article in a list, the user can browse for a teaser photo from the available media library. The problem is that in media library the user can also select a video, and after adding a video file for a teaser the page that is supposed to render the Arlima list is not rendered correctly. All posts up to the post with the video file is rendered, but the rest is failing to render correctly, a fatal error is thrown.

Arlima version deployed: 3.0 beta 34

@victorjonsson
Copy link
Owner

Videos are not supported at the moment. The first simple step we need to take is to only display images in the media library pop-up (if possible). Later on we can discus whether or not this feature (the rendering of videos in arlima articles) should be implemented.

@benjitavos
Copy link
Author

Filtering out videos from the media library pop-up would be a great start, this way we can at least avoid errors if the user is not aware of the restrictions.

@victorjonsson
Copy link
Owner

yes, it should be a quick fix

@benjitavos
Copy link
Author

Will you be able to perform this fix in near future @victorjonsson?

@victorjonsson
Copy link
Owner

Didn't find any easy way to filter out images in the media library but now the user will get an alert saying that only images can be added to the article.

@benjitavos
Copy link
Author

Thanks, that fix is good enough for us at this point. But if the other solution we discussed earlier is possible to implement later on it would be even better.

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

No branches or pull requests

2 participants