Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Select the first cell of the table block when initialised #1755

Merged
merged 1 commit into from
Jul 6, 2017

Conversation

BoardJames
Copy link

This works around the bug that causes #1751 since it could be a while before I get it fixed in TinyMCE.

@BoardJames BoardJames requested a review from nylen July 6, 2017 02:47
@nylen
Copy link
Member

nylen commented Jul 6, 2017

This is a worthwhile change, but it also needs to be fixed in TinyMCE. It's not difficult to select a block without focusing any of its elements.

@nylen
Copy link
Member

nylen commented Jul 6, 2017

Code looks reasonable; I will test this tomorrow.

@nylen nylen changed the title Select the first cell of the table block when initilised Select the first cell of the table block when initialised Jul 6, 2017
@nylen nylen merged commit 313e6c3 into master Jul 6, 2017
@nylen nylen deleted the fix/1751-table-block-select-on-init branch July 6, 2017 21:55
@nylen
Copy link
Member

nylen commented Jul 6, 2017

Thanks for the fix.

@westonruter
Copy link
Member

@nylen @EphoxJames There is a regression here I think. On the Gutenberg Demo page, after the page loads the cursor gets put onto the table halfway down the page:

image

Tug pushed a commit that referenced this pull request Jan 16, 2020
Co-authored-by: Drapich Piotr <drapich.piotr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants