ericgio / react-bootstrap-typeahead

React typeahead with Bootstrap styling
http://ericgio.github.io/react-bootstrap-typeahead/
MIT License
1.01k stars 408 forks source link

Bootstrap 5: close button missing btn class #750

Closed carstenblt closed 1 year ago

carstenblt commented 1 year ago

Version

react-bootstrap-typeahead: 6.0.0 react-bootstrap: 2.6.0 bootstrap: 5.2.2

Steps to reproduce

Use Typeahead with the multiple option.

Expected Behavior

Bildschirm­foto 2022-11-18 um 12 39 13

Actual Behavior

Bildschirm­foto 2022-11-18 um 12 34 29

Adding the btn class to the button element fixes the X. Removing line-height from rbt-token or changing rbt-token-label to padding: 0.5rem; fixes the label padding.

ericgio commented 1 year ago

Hey @carstenblt, thanks for the report. Have you included the package CSS with your project?

carstenblt commented 1 year ago

@ericgio yes, both:

@import "react-bootstrap-typeahead/css/Typeahead.css";
@import "react-bootstrap-typeahead/css/Typeahead.bs5.css";
carstenblt commented 1 year ago

I'm sorry, it's all me. I was missing the close css from bootstrap.