Skip to content
This repository has been archived by the owner on Apr 26, 2019. It is now read-only.

Define floating menu target #290

Merged

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented Jun 28, 2018

This change adds an attribute to the top-level node in <App>, indicating that floating menus should be put there. Also puts a hidden <input> next to <App> to trap keyboard focus.

Fixes carbon-design-system/carbon#910.

@marijohannessen marijohannessen merged commit df20140 into carbon-design-system:master Jul 3, 2018
@asudoh asudoh deleted the floating-menu-target branch July 3, 2018 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants