Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[charts] Divide CartesianProvider to use logic in Pro package #13531

Merged
merged 16 commits into from
Jun 19, 2024
Merged

[charts] Divide CartesianProvider to use logic in Pro package #13531

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
4a3fedc
Divide cartesianContextProvider
JCQuintas Jun 18, 2024
765499e
Use new useCartesianContext hook
JCQuintas Jun 18, 2024
4575461
Fix import
JCQuintas Jun 18, 2024
67c106f
Fix import
JCQuintas Jun 18, 2024
2f03e6c
Fix missing component
JCQuintas Jun 18, 2024
9a3a6f1
Add exports
JCQuintas Jun 18, 2024
7341f89
Add .nice back to prevent issues
JCQuintas Jun 18, 2024
a5ee2ef
Both scale and domain should be nice
JCQuintas Jun 18, 2024
f9a953e
Add ticknumber to nice()
JCQuintas Jun 18, 2024
2c750e2
Update packages/x-charts/src/context/CartesianProvider/CartesianConte…
JCQuintas Jun 18, 2024
ca5319a
Dont error as MUI Y
JCQuintas Jun 18, 2024
f0040c9
Simplify computeValue for X
JCQuintas Jun 18, 2024
6971548
computeValue for Y
JCQuintas Jun 18, 2024
5062469
Update packages/x-charts/src/context/CartesianProvider/computeValue.ts
JCQuintas Jun 19, 2024
b606028
Update packages/x-charts/src/context/CartesianProvider/computeValue.ts
JCQuintas Jun 19, 2024
a45bbc3
Move consts back out
JCQuintas Jun 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Both scale and domain should be nice
  • Loading branch information
@JCQuintas
JCQuintas committed Jun 18, 2024
commit a5ee2ef9df3165d35d852cec5ad7db8eeda42d51
8 changes: 4 additions & 4 deletions packages/x-charts/src/context/CartesianProvider/computeValue.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,8 +85,8 @@ export const computeValue = (
const extremums = [axis.min ?? minData, axis.max ?? maxData];
const tickNumber = getTickNumber({ ...axis, range, domain: extremums });

const scale = getScale(scaleType, extremums, range);
const [minDomain, maxDomain] = scale.nice().domain();
const scale = getScale(scaleType, extremums, range).nice();
const [minDomain, maxDomain] = scale.domain();
const domain = [axis.min ?? minDomain, axis.max ?? maxDomain];

completedXAxis[axis.id] = {
Expand Down Expand Up @@ -157,8 +157,8 @@ export const computeValue = (
const extremums = [axis.min ?? minData, axis.max ?? maxData];
const tickNumber = getTickNumber({ ...axis, range, domain: extremums });

const scale = getScale(scaleType, extremums, range);
const [minDomain, maxDomain] = scale.nice().domain();
const scale = getScale(scaleType, extremums, range).nice();
const [minDomain, maxDomain] = scale.domain();
const domain = [axis.min ?? minDomain, axis.max ?? maxDomain];

completedYAxis[axis.id] = {
Expand Down