-
Notifications
You must be signed in to change notification settings - Fork 49
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
More Info on CameraSettings #113
Comments
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:
|
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: 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. |
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:
home-gallery/packages/webapp/src/single/Details.tsx
Line 90 in a220ec8
home-gallery/packages/database/src/media/exif.js
Line 78 in a220ec8
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.
The text was updated successfully, but these errors were encountered: