Skip to content

Add spacebar functionality#2087

Closed
jasonchangxo wants to merge 2 commits intoJedWatson:masterfrom
jasonchangxo:fix/open-on-spacebar
Closed

Add spacebar functionality#2087
jasonchangxo wants to merge 2 commits intoJedWatson:masterfrom
jasonchangxo:fix/open-on-spacebar

Conversation

@jasonchangxo
Copy link
Contributor

Added spacebar functionality to align with html controls. toggle open state on spacebar (if not searchable) toggle closed state & select focused option on spacebar (if not searchable)

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.0%) to 91.871% when pulling 02f4403 on jasonchangxo:fix/open-on-spacebar into b6fd3e7 on JedWatson:master.

@gwyneplaine
Copy link
Collaborator

@jasonchangxo, thanks for this, can you add a test case for this as well please?

@jasonchangxo
Copy link
Contributor Author

Sure thing @gwyneplaine ! Added test case for open on spacebar

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) to 92.791% when pulling a4058d3 on jasonchangxo:fix/open-on-spacebar into b6fd3e7 on JedWatson:master.

@gwyneplaine
Copy link
Collaborator

@jasonchangxo, thanks for this! I've merged in a separate body of work that includes the functionality you've supported in this PR. You can find it here #1428

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.

3 participants