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

fix: #8444 axis align option ignored #9273

Closed
wants to merge 3 commits into from

Conversation

okon3
Copy link

@okon3 okon3 commented Oct 24, 2018

As noted in the issue #8444 the align option of the nameTextStyle seems to be ignored.
This could also be the solution to the problem reported in #8046

Copy link
Member

@100pah 100pah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @okon3 .
Would you please correct the code format: the indent should be 4 spaces.
(There is a .eslintrc in this project)

@okon3
Copy link
Author

okon3 commented Nov 4, 2018

@100pah Should be good now.

Copy link
Author

@okon3 okon3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adjusted. See later commit.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2021

This PR has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this PR. We are sorry for this but 2 years is a long time and the code base has been changed a lot. Thanks for your contribution anyway.

@github-actions github-actions bot added the stale Inactive for a long time. Will be closed in 7 days. label Sep 4, 2021
@Ovilia Ovilia removed the stale Inactive for a long time. Will be closed in 7 days. label Sep 5, 2021
@Ovilia Ovilia modified the milestone: 5.2 Sep 5, 2021
@Ovilia
Copy link
Contributor

Ovilia commented Sep 10, 2021

Fixed since 4.3.0.

@Ovilia Ovilia closed this Sep 10, 2021
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

Successfully merging this pull request may close these issues.

3 participants