From 9b3f4ce3a1a359896f41ca8e549bc6661bc2fa90 Mon Sep 17 00:00:00 2001 From: Jessie Keck <jessie.keck@gmail.com> Date: Wed, 13 Mar 2019 11:27:59 -0700 Subject: [PATCH] Don't wrap LanguageSettings in a List component (so the values are arrow navigable in the NestedMenu) --- src/components/LanguageSettings.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/components/LanguageSettings.js b/src/components/LanguageSettings.js index 29efea8b7..c780eb450 100644 --- a/src/components/LanguageSettings.js +++ b/src/components/LanguageSettings.js @@ -1,7 +1,6 @@ import React, { Component } from 'react'; import ListItemIcon from '@material-ui/core/ListItemIcon'; import ListItemText from '@material-ui/core/ListItemText'; -import List from '@material-ui/core/List'; import MenuItem from '@material-ui/core/MenuItem'; import Typography from '@material-ui/core/Typography'; import CheckIcon from '@material-ui/icons/CheckSharp'; @@ -21,7 +20,7 @@ export class LanguageSettings extends Component { } = this.props; return ( - <List> + <> { languages.map(language => ( <MenuItem @@ -41,7 +40,7 @@ export class LanguageSettings extends Component { </MenuItem> )) } - </List> + </> ); } } -- GitLab