The react-ldavis from sseyren

react-ldavis's People

Contributors

Stargazers

Watchers

react-ldavis's Issues

React 17+ support

We are currently trying to integrate a pyLDAvis visualization into our React frontend using your library, but npm could not resolve the dependencies:

npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR! 
npm ERR! While resolving: [email protected]
npm ERR! Found: [email protected]
npm ERR! node_modules/react
npm ERR!   react@"^17.0.2" from the root project
npm ERR!   peer react@">=16.8.0" from @emotion/[email protected]
npm ERR!   node_modules/@emotion/react
npm ERR!     @emotion/react@"^11.9.0" from the root project
npm ERR!     peer @emotion/react@"^11.0.0-rc.0" from @emotion/[email protected]
npm ERR!     node_modules/@emotion/styled
npm ERR!       @emotion/styled@"^11.8.1" from the root project
npm ERR!       3 more (@mui/material, @mui/styled-engine, @mui/system)
npm ERR!     3 more (@mui/material, @mui/styled-engine, @mui/system)
npm ERR!   17 more (@emotion/styled, @mui/base, @mui/icons-material, ...)
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer react@"^16.0.0" from [email protected]
npm ERR! node_modules/react-ldavis
npm ERR!   react-ldavis@"^1.0.0" from the root project
npm ERR! 
npm ERR! Conflicting peer dependency: [email protected]
npm ERR! node_modules/react
npm ERR!   peer react@"^16.0.0" from [email protected]
npm ERR!   node_modules/react-ldavis
npm ERR!     react-ldavis@"^1.0.0" from the root project

Would it be rather straightforward to add support for higher React versions?

lodash should be a dependency, not peerDependency

Feature Request: Customize Colors

I noticed that the fill colors for all the shapes in the clustering visualization are all hardcoded, but it would be a lot better if it relied on react theme colors since this is a react library. The current colors do not go with my project's brand guidelines, and I wish I could have a way to change the colors of this tool.