ailon / markerjs2

Add image annotation to your web apps.
https://markerjs.com
Other
143 stars 39 forks source link

Request: add change event for MarkerAreaState #109

Closed tarngerine closed 2 years ago

tarngerine commented 2 years ago

I'm implementing in a React context and as per most React APIs, an onChange prop that is fired every time the MarkerAreaState changes (whatever we read in getState()) would be super handy to help with syncing/persisting changes live and activating any necessary side effects. In particular, markerupdate is missing from the events today, but i think a basic statechange event might also be nice.

ailon commented 2 years ago

Good points on both events. I will look into adding them.

ailon commented 2 years ago

Add both statechange for the whole area status change and markerchange for individual marker state change. Available now in v2.23

tarngerine commented 2 years ago

Thank you!

On Aug 15, 2022 at 8:21:15 AM, Alan Mendelevich @.***> wrote:

Add both statechange for the whole area status change and markerchange for individual marker state change. Available now in v2.23

— Reply to this email directly, view it on GitHub https://github.com/ailon/markerjs2/issues/109#issuecomment-1215007970, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMN5TL476XBUYXFXPLT5CDVZI74XANCNFSM56I6NRCQ . You are receiving this because you authored the thread.Message ID: @.***>