-
-
Notifications
You must be signed in to change notification settings - Fork 186
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
Adding support for images, any maybe other file types? 😄 #754
base: main
Are you sure you want to change the base?
Conversation
Hey Cal! Great to "see" you again 😁 Thank you for this suggestion / start of a discussion. I started (50% finished?) Simple Image Browser application: https://github.com/whyboris/Simple-Image-Browser (SIB) which might be just what people keep asking me about when they say they want images in VHA. Before I add image types to VHA, I'd like to get the dedicated image browser released first 😅 . I've felt rather strongly against mixing image and video browsing inside one application, but I see how someone might have images that are associated with some videos and it may be useful to see those all in one place. Coding it up and making the UI for it feels like would be quite nightmarish for my app. I guess I could create an image-only gallery view which would just show images inside a folder without keeping to the 16:9 video resolution; perhaps, I could just use the functionality of the Simple Image Browser of not even extracting screenshots, but simply displaying images live 🤔 ... but at that point I'd just be making a crappier version of SIB 🤷 I'll definitely consider it once I've released SIB 🤝 |
The way I'm thinking of it, is that images are just like a video, but with a single frame. When you think of it that way, there's very little to change, and you get all the benefits of what VHA already does extremely well. 💡 The code should basically be identical, just with less features available for images. 🤔 And I'm planning on having a toggle for importing/displaying images or videos, so it's totally optional anyway. My current plan for phase one anyway, is just that. Not even image viewing in app - just like videos, click an image, and your default image viewer opens. (This already just works actually 😂 ). Thoughts on this view? |
I've a few more features and improvements I'd like to add to VHA, so if you'd still like to hold off, I'll create and integration branch on my repo for this and future changes, and still try to make pull requests on your main branch. 😄 |
Thumbnails are much more performant, especially when images get large (raw files), and if you access them over a network on a NAS somewhere. And it's super easy to piggy back off ffmpeg for this - I see it's already done in VHA for replacing your own thumbnail! 😄 |
I'm certainly open to this possibility. I guess in the https://github.com/whyboris/Video-Hub-App/blob/main/src/app/components/home.component.html#L691 And then after that, just about nothing else needs to change 👍 So you are probably right -- if the weird 16:9 view is acceptable (with black letterboxing; which in Dark Mode would be invisible anyway) then the number of changes we need to get this working might be pretty small 🤔 Thanks as usual for excellent ideas and actual additions to the app 🙇 I've not thought through all the implications of having images, but it seems like we'll automatically inherit tags 🤷 which I guess some people want (🤷 ?). The one minor worry is about I'm open to a PR that does something like this -- and/or a variation -- I'm open to recommendations 🤝 |
Just a WIP, adding support for images.
Should be super simple, just work out how to deal with image strips and clips (basically don't), and allow for toggling video and images individually! 💡
Thoughts?