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

Add option to ignore editor tab key handling #14

Merged
merged 1 commit into from
Jun 13, 2017
Merged

Conversation

alexlande
Copy link
Contributor

This adds a prop to Editor to allow you to disable the tab key handling. The default behavior is nice for a lot of cases, but it's a focus trap for keyboard users navigating through a page with an editor in it.

@alexlande alexlande requested a review from kitten June 12, 2017 23:33
@@ -15,6 +15,10 @@ export function LiveEditor(props: HTMLAttributes<HTMLElement>): JSX.Element
export function LiveError(props: HTMLAttributes<HTMLElement>): JSX.Element
export function LivePreview(props: HTMLAttributes<HTMLElement>): JSX.Element

export class Editor extends Component<HTMLAttributes<HTMLElement>, {}>{}
export type EditorProps = HTMLAttributes<HTMLElement> & {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not familiar with TypeScript at all, so let me know if I screwed this up somehow.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that's the correct place 👍

@kitten kitten merged commit a316c3c into master Jun 13, 2017
@alexlande alexlande deleted the ignoreTabKey branch June 13, 2017 15:57
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.

2 participants