Skip to content

xmp: Added support for additional metadata #41

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

Closed
wants to merge 1 commit into from
Closed

Conversation

LNA-DEV
Copy link

@LNA-DEV LNA-DEV commented Mar 29, 2025

This adds support for a couple of additional metadata properties in XMP. All of these are children of the RdfDescription

XMP subject not supported #40

This adds support for a couple of additional metadata properties in XMP. All of these are children of the RdfDescription

XMP subject not supported bep#40
@bep
Copy link
Owner

bep commented Mar 29, 2025

I'm going to close this for now, as I think this goes beyond the scope of this project (at least it goes beyond the scope of mt free time).

See https://pkg.go.dev/github.com/bep/imagemeta#Options and HandleXMP for how to handle this on the caller side,.

@bep bep closed this Mar 29, 2025
@LNA-DEV
Copy link
Author

LNA-DEV commented Mar 29, 2025

Okey I can absolutely understand that. But this implies that some tags won't be supported downstream in Hugo am I right?

@bep
Copy link
Owner

bep commented Mar 30, 2025

I'll reopen this and have a look at it later (but no promise) I was a little bit grumpy last night ...

@bep bep reopened this Mar 30, 2025
@LNA-DEV
Copy link
Author

LNA-DEV commented Mar 30, 2025

Thanks for reconsidering^^ I appreciate it a lot!

To make my usecase clear: I want to use the metadata saved by my photoeditor (Darktable) to build the gallery section of my webpage. (So I can set an alt text and "hashtags" based on the Darktable metadata.) This PR together with gohugoio/hugo#13539 on the Hugo side would enable that use. (I am currently testing it with a custom build of this two PRs. Looks good so far.)

After all I can totally understand if it is too much of an edge case but I think this could be useful for image centered pages.

@bep
Copy link
Owner

bep commented Apr 7, 2025

I couldn't push to this PR, so I replaced it with #43

@bep bep closed this Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants