scaleflex / filerobot-image-editor

Edit, resize, and filter any image! Any questions or issues, please report to https://github.com/scaleflex/filerobot-image-editor/issues
MIT License
1.27k stars 322 forks source link

Give more control over the functionality of the editor(details in description) #412

Open KDDeveloper opened 7 months ago

KDDeveloper commented 7 months ago

Hey, I have been using this editor and have had an amazing experience because of its capabilities and features, it would be amazing if we could have more control over the annotations and their behaviour. Example,

  1. An onDelete event.
  2. show a particular set of annotations only based upon a common id added in them by user.
  3. change style of the selected boundary
  4. option to show a string added to the annotation object when hovered over it. something like showLabel = {(annotation)=>{annotation.nameInObject}}.
  5. you have a ref option with which we can control annotations data but it could be made more simpler by making the editor have a controlled state (for react).
AhmeeedMostafa commented 7 months ago

@KDDeveloper thank you for ur suggestions we will consider them while planning for the next major version.

I'd also appreciate explaining the use case of 2nd point and clarifying it more.

KDDeveloper commented 4 months ago

hey @AhmeeedMostafa , Sorry for the late reply. For the 2nd point, lets say there are a 100 of annotations at point of time and i have grouped them on the basis of an Id that i have added in the "annotations" state, then it should be possible for me to just have the annotations grouped by a that id as visible. This enhances the user experience as they can focus on on part at a time and have least amount of confusion. This will also inscrease and better the scope of use. something like groupAnnotsBy={"key in which the id is stored"} showAnnotGroup={"the id value "}. If "showAnnotGroup" is empty show all. and give a small info at a corner that a particular id group is only being shown. if none added, it'll work as is. Thankyou.

AhmeeedMostafa commented 4 months ago

@KDDeveloper Thanks for the clarification, if we have enough time and find this would fit more general use cases we will consider it