react-popover

react-popover copied to clipboard

Missing key attribute warning since 0.5.7 upgrade

1

Not that big of deal at this moment, but would be nice if this can be fixed. ``` warning.js:33 Warning: Each child in an array or iterator should have a...

martinbroos

Popover body is not centered over target

5

The popover body position is not consistent and hard to get it centered over the target (that is not moving).

trinile

feat(Popover): Add check for onOuterMovement

5

Working on a solution for #76

fluke

setState errors on navigation

2

Hi @jasonkuhrt, I keep seeing these errors in the console every time I navigate or the parent component (that contains the Popover) unmounts. ```jsx warning.js:33 Warning: Can't call setState (or...

oyeanuj

Tooltip body as a function

4

Hi @jasonkuhrt, this is a relatively small proposal compared to all the other big things in line for this library. I was wondering if you've considered having the `body` prop...

oyeanuj

Accessibility

2

Hi @jasonkuhrt, Glad to see this project revived again! I am curious about your thoughts on adding more accessibility best practices (like labels, roles, keyboard shortcuts) to the project to...

oyeanuj

Popover with no children

6

This does not work: ``` ``` This does: ``` ``` It'd be nice if both worked =)

jeanbza

Exposing animations via props

11

thoughts? @jasonkuhrt

stephen

Popover background is transparent, how to remove it?

3

Hi, The background seems to be transparent, I tried setting opacity in style as well but it doesnt work. Is this a bug?

naveenreddyin

React Native support

2

Should we support React Native? It seems like there is no logical reason not to. There is an existing [RN Popover](https://github.com/jeanregisser/react-native-popover) by @jeanregisser. We can start this issue by reviewing...

jasonkuhrt