fedeorlandau / chakra-ui-simple-autocomplete Goto Github PK

There is a small bug on the package.json where the peerDependency is react v17 when it should be v18

We should allow Option to be generic. Cheers,

Love the component but for our use case we don't want to render badges as we are only allowing a select limit of 1. The rederBadge function could return null but the with the margin would still render. I think a simple prop to turn off badge rendering would be a great addition. I can open a PR if you are open to allowing this.

Cheers

I'd love to make this component more accesible. For this reason, we need to provide keyboard navigation on the option list.

It would be nice if we can put some controls on storybook to show different autocomplete configurations (like a placeholder for instance)

Simply add a new property isInvalid to be passed to the Chakra Input component.

I installed the latest package via npm install and been using it within component
instead of seeing chakra ui styled input, I am seeing vanilla (no style) component

import { Autocomplete, Option } from 'chakra-ui-simple-autocomplete';
....
            <FormControl>
              <FormLabel>Team 1</FormLabel>
              <Autocomplete
                options={availablePlayers}
                result={team1Options}
                setResult={(availablePlayers: Option[]) => setTeam1Options(availablePlayers)}
                placeholder={'Team 1'}
              />
            </FormControl>

Please remove the Travis ci references from the readme file :)

Hey there, I'm running into the following error:

SyntaxError: Named export 'chakra' not found. The requested module '@chakra-ui/system' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@chakra-ui/system';

It looks to be the same thing as chakra-ui/chakra-ui#7170. It looks like updating @chakra-ui/react and @chakra-ui/icons should fix it.

It should run pnpm test on every PR