Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

MooEditable.js line 387 #19

Closed
ryanmitchell opened this issue May 14, 2010 · 3 comments
Closed

MooEditable.js line 387 #19

ryanmitchell opened this issue May 14, 2010 · 3 comments

Comments

@ryanmitchell
Copy link
Contributor

Hello

After this line you need to call this.checkStates() to update Safari's states as the selected node has now changed.

Thanks
Ryan

@cheeaun
Copy link
Owner

cheeaun commented May 16, 2010

Hey Ryan,

Could you give me a use case where checkStates() is needed to be called after the selectNode?

@ryanmitchell
Copy link
Contributor Author

Yeah - what I'm using it for is editing an img properties - like allowing width/height/alt etc to be changed. So in Safari the image button was never showing as selected, as it would always think the paragraph node was selectedNode using checkStates, but when you use the selectNode after clicking the image you should call checkStates so that anything waiting for an image to be selected gets the right information.

@cheeaun
Copy link
Owner

cheeaun commented May 23, 2010

Fixed in 9432b89

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants