VictoryLabel's backgroundStyle functional styles are not provided current props for evaluation

[chacestew]

Bugs and Questions

Checklist

• This is not a victory-native specific issue. (Issues that only appear in victory-native should be opened here)

• I have read through the FAQ and Guides before asking a question

• I am using the latest version of Victory

• I've searched open issues to make sure I'm not opening a duplicate issue

The Problem

When providing a backgroundStyle prop to VictoryLabel, the functional styles are not being passed props for evaluation.

Reproduction

This sandbox shows the problem: https://codesandbox.io/s/basic-victory-example-eck0o?file=/index.js

Open console to see that undefined is returned when logging the argument provided to the fill functional style.

Cause

I believe this is due to the following code:

victory/packages/victory-core/src/victory-label/victory-label.js

Lines 77 to 80 in cc9b9b2

	return Array.isArray(style) && !isEmpty(style)
	? style.map((s) => Helpers.evaluateStyle(s, props))
	: Helpers.evaluateStyle(style);
	};

On line 79, props are missing as the second argument to Helpers.evaluateStyle.

Fix

If you agree with my findings, I would be happy to create a PR altering this one line.

[boygirl]

@chacestew Good catch! Your solution is correct, and a PR would be very welcome. Thank you!