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

Avoid navigating outside the editor using the arrow keys #2004

Merged
merged 1 commit into from
Jul 27, 2017

Conversation

youknowriad
Copy link
Contributor

closes #1997

In this PR, I'm limiting the scope of the arrow navigation to the editor area only.

@youknowriad youknowriad added the [Type] Bug An existing feature does not function as intended label Jul 25, 2017
@youknowriad youknowriad self-assigned this Jul 25, 2017
@@ -262,7 +262,7 @@ class VisualEditorBlock extends Component {
'*[tabindex]',
'textarea',
'input',
].join( ',' );
].map( ( selector ) => `.editor-visual-editor ${ selector }` ).join( ',' );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we make that a constant?

@youknowriad youknowriad force-pushed the fix/arrow-navigation-outside-editor branch from 6065dcd to 3ac6370 Compare July 27, 2017 09:24
@youknowriad youknowriad force-pushed the fix/arrow-navigation-outside-editor branch from 3ac6370 to 9c348de Compare July 27, 2017 09:33
@youknowriad youknowriad merged commit c8238ca into master Jul 27, 2017
@youknowriad youknowriad deleted the fix/arrow-navigation-outside-editor branch July 27, 2017 09:38
Tug pushed a commit that referenced this pull request May 12, 2020
…left-border

Remove left border of the toolbar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It's possible to lose the cursor in a paragraph block
2 participants