Slate: Cannot focus inline void elements

Created on 27 Feb 2020  Â·  11Comments  Â·  Source: ianstormtaylor/slate

Do you want to request a _feature_ or report a _bug_?

_bug_

What's the current behavior?

When arrowing through the text in the mentions example, as soon as a mention is highlighted, focus is lost and user can no longer use arrow keys.

2020-02-26 19-54-03 2020-02-26 19_55_56

Slate: 0.57.1
Browser: Chrome
OS: Mac

What's the expected behavior?

The mentions should become highlighted and arrow keys should continue to work.

bug ♥ help

Most helpful comment

Hi @cj1128 , the examples that are currently hosted should have functional input voids. Please double check and let me know if that's not the case. The live examples work for me in FF and Chrome.

A version of slate that fixes the problem with the input voids has not yet been released.

By any chance, do you have a timeline for the release of the fix @CameronAckermanSEL ?

All 11 comments

I think this is related to one of the PRs I merged in that was attempting to fix inputs in void nodes.

Is this related to Issue #2685 and #2019 ?

Examples with input, editable-voids and embeds are all broken now, You just can't type anything in this input element.

Slate version: 0.57.1

Hi @cj1128 , the examples that are currently hosted should have functional input voids. Please double check and let me know if that's not the case. The live examples work for me in FF and Chrome.

A version of slate that fixes the problem with the input voids has not yet been released.

Hi @cj1128 , the examples that are currently hosted should have functional input voids. Please double check and let me know if that's not the case. The live examples work for me in FF and Chrome.

A version of slate that fixes the problem with the input voids has not yet been released.

By any chance, do you have a timeline for the release of the fix @CameronAckermanSEL ?

Does anyone know if there is a previous version I can rollback to in the meantime where this bug is not present?

Hey @kilrain, the version of the library that has this bug should not have been released yet, it should only impact the live examples right now. Are you able to reproduce this on the latest release in npm?

Ah - that's good news. I'm in the middle of upgrading from .47 and haven't yet migrated the inline-void parts of my codebase. Thanks for the quick response @CameronAckermanSEL !

Hi @cj1128 , the examples that are currently hosted should have functional input voids. Please double check and let me know if that's not the case. The live examples work for me in FF and Chrome.

A version of slate that fixes the problem with the input voids has not yet been released.

Hi @CameronAckermanSEL, this is my bad. I used the release version (0.57.1) to test the master examples. All the examples work well except editable-voids and embeds. I retest them using the master version and they work as expected.

Thanks for all the hard work.

@cj1128 I'm using 0.57.1 version on my local development and input on void element don't work, the onChange event doesn't fire and the left and right arrows don't work.

You mention you fixed this with the master version, what version is that?

Thank you for this awesome library btw.

I think I fixed this problem here. https://github.com/ianstormtaylor/slate/pull/3572

There's a preview deployment under the automated PR checks. Would appreciate a (careful!) second glance

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ianstormtaylor picture ianstormtaylor  Â·  3Comments

markolofsen picture markolofsen  Â·  3Comments

varoot picture varoot  Â·  3Comments

chriserickson picture chriserickson  Â·  3Comments

ianstormtaylor picture ianstormtaylor  Â·  3Comments