Skip to content
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

Image replace feature #266

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Diesnei
Copy link

@Diesnei Diesnei commented Sep 10, 2024

Add a "click" event to each "imageEl" that when triggered:

  1. Update the state of the container to "empty"
  2. Remove the current image picture (to avoid duplications)
  3. Calls the file selection method "onSelectFile()"

This way, the "fillImage" method will be called again after "onSelectFile" is fired.

Of course I would like to make it customizable to also work in "read only" scenarios and so on. But I'm not sure how to do it, sorry.

That's my first open source pull request though, i really hope it helps.
Thanks for the amazing package <3.

… "onSelectFile" method to allow image replacement.
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