Victor Cazanave

Results 30 comments of Victor Cazanave

Thanks for your comment @eiproject! > I am a bit confused to change the color. And then I try to find the CSS file that control the color. > I...

Hi @deepghuge36, It looks similar to the [demo](https://victorcazanave.github.io/react-svg-map/). Have you looked at the [examples' source code](https://github.com/VictorCazanave/react-svg-map/tree/master/examples/src)? If this is not what you want, please detail your needs and share more...

Thank you for reporting this issue @tgeorgiadis I don't know when I have time to look at it, because currently I'm very busy. If you know how to fix it,...

`CheckboxSVGMap` seems to work with React 17: https://codesandbox.io/s/checkbox-svg-map-with-react-17-wx0pz

Although I couldn't fix it in a Code Sandbox, `RadioSVGMap` worked in a local project with React 17.0.2. However, in strict mode, it throws a [warning](https://reactjs.org/docs/strict-mode.html#warning-about-deprecated-finddomnode-usage), because the deprecated `ReactDOM.findDOMNode()`...

This component was built to handle internally the state of the selected locations (kinda like an [uncontrolled component](https://reactjs.org/docs/uncontrolled-components.html)). I think making `selectedLocationIds` update the internal state would break the "single...

@arslanakhtar61 Since `RadioSVGMap` handles internally the state of the selected location (like `CheckboxSVGMap`), updating `selectedLocationId` won't re-render the component (as written in the [documentation](https://github.com/VictorCazanave/react-svg-map#api-2)). The next major version (v3.0.0) may...

Thanks a lot for your contribution! 😃 I'm still thinking about how to deal with the maps other than countries (i.e. cities, counties, states...) because I'm worried this repository will...

Thank you for the quick fixing @bkrmalick ! There are still few issues to fix, then it's good to be merged. However I won't merge it until Vega publishes a...

Thanks again @mh-mazen ! Could you resolve the conflicts please? Saudi Arabia has already been added