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.

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

Line Series Y-Axis Title Alignment Issue #26

Closed
walsha2 opened this issue Sep 15, 2019 · 2 comments
Closed

Line Series Y-Axis Title Alignment Issue #26

walsha2 opened this issue Sep 15, 2019 · 2 comments

Comments

@walsha2
Copy link

walsha2 commented Sep 15, 2019

Description:

There is a minor issue introduced with the latest v4 version of the package. It seems that now AxisTitle alignment is measured as a function of the entire chart container height (possibly?) as opposed to just the y-axis (like it was before in v3).

primaryYAxis: NumericAxis(
  isVisible: true,
  title: AxisTitle(
    alignment: ChartAlignment.center,
    text: "Rate of Return (%)",
  ),
)

The following is for the same exact code, just updating from v3 to v4 of package - note how Rate of Return (%) gets shifted down (no longer centered about the y-axis) in v4.


Here is the output from v3:

Here is the output from v4:

Issue Resolution:

I am not sure why anyone would want AxisTitle alignment relative to the entire rendered chart container and not just the chart axis. The axis title should measure its alignment relative to the axis height.

@JayavigneshwaranG
Copy link
Collaborator

Hi @walsha2 ,

We have analyzed this, but we could not replicate this at our end. Now we have made some general improvements and bug fixes and published to pub. So we request you to use the latest version packages(1.0.0-beta.5). The Y-axis title text is aligned center to the axis.

If you still face the same issue in this version, please revert back with your scenario. We will check and provide the exact solution ASAP.
We are closing this issue. But you can reopen this if you face any concerns.

Thanks,
Jayavigneshwaran

@walsha2
Copy link
Author

walsha2 commented Sep 19, 2019

@JayavigneshwaranG confirmed - this is now fixed with the latest release. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants