xemle / home-gallery

Self-hosted open-source web gallery to view your photos and videos featuring mobile-friendly, tagging and AI powered image discovery
https://home-gallery.org
MIT License
836 stars 64 forks source link

More Info on CameraSettings #113

Closed cimba007 closed 3 months ago

cimba007 commented 10 months ago

Hello,

it would be nice if the information would show a little bit more from the exif data.

As I don't know how to do a proper PullRequest but I still want to contribute .. here is my try:

https://github.com/xemle/home-gallery/blob/a220ec89802f13618ff0d5327c41ce71a81a3b46/packages/webapp/src/single/Details.tsx#L90

{title:"Camera Settings",value:`ISO ${e.iso}, Aperture ${e.aperture}, FocalLength ${e.focalLength}mm, ExposureTime ${e.exposureTime}`}

https://github.com/xemle/home-gallery/blob/a220ec89802f13618ff0d5327c41ce71a81a3b46/packages/database/src/media/exif.js#L78

...
whiteBalance: exif.WhiteBalance,
exposureTime: exif.ExposureTime

Other Data from Exif can be added the same way it seems. It might require a rebuild of the database to store this information. If you want to make this change locally you just need to do the changes to.

xemle commented 10 months ago

Hi @cimba007

thank you for your input. you are on the right track. To create a proper PR you need to have a development setup with git and nodejs environment. I can recommend to use visual studio code as editor. Than you can check if your changes are working as desired. Some skeleton steps for the development setup is:

xemle commented 10 months ago

Currently I am keeping calm by implementing your suggestion right away by my own. I am rewriting the webapp UI using tailwind and the detail view gets a new layout. Since this work takes time and HomeGallery is a spare time project the progress of the UI rewrite is currently on slow progress. The preview looks like:

Screenshot_20231203-100135

So I do not want to change the current (old) state by my own. I am fine to apply a proper PR if you have one with your changes. This can be the current master without the UI rewrite since the UI rewrite has currently private state.

xemle commented 3 months ago

Closing this issue due inactivity.

Please reopen it, if it becomes active again.