Giter Club home page Giter Club logo

Comments (6)

bretthadley avatar bretthadley commented on June 18, 2024 1

@carbonrobot

I had same issue when doing some package upgrades. It was solved by modifying how I was using "containerComponent" prop.

//Before

<VictoryChart containerComponent={tooltipLabel ? <VictoryVoronoiContainer/> : undefined} >{...}</VictoryChart>

//After

<VictoryChart containerComponent={tooltipLabel ? <VictoryVoronoiContainer/> : <VictoryContainer/>} >{...}</VictoryChart>

It looks like you no longer include VictoryContainer by default if "containerComponent" is undefined.

from victory.

Kerumen avatar Kerumen commented on June 18, 2024 1

I stumbled across this issue too. I confirm what @bretthadley said, if containerComponent is undefined, there is an app crash in React Native.

from victory.

carbonrobot avatar carbonrobot commented on June 18, 2024

Hi @KelvinPortuphy, do you have some sample code so we can reproduce the issue?

Here is the template for submitting a big report. It helps us solve the issue for you quicker. https://github.com/FormidableLabs/victory/blob/main/.github/ISSUE_TEMPLATE/bug_report.md

from victory.

KelvinPortuphy avatar KelvinPortuphy commented on June 18, 2024

@carbonrobot
that would hard to do because we use it in quiet a number places in our app,
and I have moved on because of tight deadlines

but I noticed replacing
var componentStyle = titleComponent.props && titleComponent.props.style || {};
with
var componentStyle = titleComponent?.props && titleComponent?.props?.style || {};

on line 205 fixed the issue.

from victory.

carbonrobot avatar carbonrobot commented on June 18, 2024

Ok, we will close the issue until you are able to provide some information on your usage.

from victory.

carbonrobot avatar carbonrobot commented on June 18, 2024

Thanks for the additional info @bretthadley and @Kerumen.

This component uses default props for that property so undefined would definitely throw an error. I'll review our docs to make sure we cover this use case.

from victory.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.