Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented May 16, 2024

The same as the previewer page. A default image was also missing.
This first checks whether svg or png exists, if this is not there then a default image is loaded.

Copy link
Member

@qqmyers qqmyers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thanks

@qqmyers qqmyers merged commit 62d3352 into gdcc:main Jun 4, 2024
@qqmyers
Copy link
Member

qqmyers commented Jun 5, 2024

@BenediktMeierUIT - I think I merged this too quickly. It looks like the change to add internationalization #19, made the fileupload2.js script a module which interferes with being able put your handleImageError method in as an onerror function. I pulled that method out into a separate js and avoided sending the siteUrl variable to it to make things work again in the main branch ( see 3be8983 and 01935a8 ). If you know of a cleaner way to do this, please feel free to add a new PR.

@qqmyers qqmyers mentioned this pull request Jun 5, 2024
@ghost
Copy link
Author

ghost commented Jun 6, 2024

Okay, I'll take a closer look. I will change something there. Thanks for the reference.

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.

1 participant