.focus() doesn't work

Bug Report

What’s the current behavior?

firing .focus() doesn’t revert to previous selection, or even focus the editor.

code sample: https://codesandbox.io/s/13r8qmv0l3 (see: line 113)

Made a simple change to slate example code, removing event.preventDefault() from buttons (to lose focus) and then run .focus(). Found while trying to set link hrefs from a custom dialog popup, with no success.

What’s the expected behavior?

Previous selection/highlight should be retained after firing .focus()

[ianstormtaylor]

Hey, thanks for using Slate! Unfortunately, we can’t offer support for usage questions in the issues here because it becomes overwhelming to maintain the project if the issues are filled with questions.

However, we do have a Slack channel and people are constantly asking and answering questions in there. So I’m going to close this issue, but I definitely recommend joining the Slack channel if you want to find people that might be able to help.

Thanks for understanding!

[davidchang]

@afrankel-sfdo @Slapbox

editor.focus(), being an imperative command that does not directly affect the state, ends up being in conflict with the React lifecycle, which renders the Editor and its focus based on what’s in the state. in the code sample given originally, if you call this.editor.focus() outside of the React lifecycle (like if you wrap it in a setTimeout()), you’ll find that it works as expected. but if you try to do so synchronously, React will go ahead and re-render over that, based on the state, which is unfocused