kingyiren / flex-object-handles

Automatically exported from code.google.com/p/flex-object-handles
0 stars 0 forks source link

Upgrades that would be beneficial #40

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Not sure if this is the right place to put this info but anyway.
First of all, thank you very much for developing this, really saved me a lot of 
time with my project.

Your code is very clean and easy to read. Nice work!!

However i do think there are two things that have room for improvement.

Firstly the multiSelectHandles property, setting it simply doesn't work as the 
multi select model is registered in 
the constructor and there is no way to deregister it without hacking your code.

Perhaps a getter and setter here would be good for updating the model's handles.

Secondly, i was having an issue with updating my model's from outside your 
framework, i.e. tweening them into place. 
The handles would not update their position.
I then worked out i can simply dispatch a PropertyChangeEvent event from the 
model, all sorted.
However, if there is more than one item selected updating the model and 
dispatching events will not update the 
handles because it uses the multi select model. I had to hack the code to allow 
events from that model. i cannot 
think of a simple solution here as i think the original architecture needs to 
be taken into account.

Original issue reported on code.google.com by jwbe...@gmail.com on 7 Apr 2010 at 1:59

GoogleCodeExporter commented 8 years ago
sorry about the duplicates, google is playing games with me.

Original comment by jwbe...@gmail.com on 7 Apr 2010 at 2:00