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

Allow tooltip rotation and improve alignment of rotated elements #679

Merged
merged 6 commits into from
Jun 8, 2021

Conversation

josh-ksr
Copy link
Contributor

With these changes it is possible to rotate tooltips (#260).

I also changed the alignment of rotated elements, so that all rotated elements have the same distance to the chart.

Rotated Tooltips:
1

Alignment including these changes:
after

For comparison: Alignment of rotated elements before, without these changes:
before

@josh-ksr josh-ksr closed this May 20, 2021
@josh-ksr josh-ksr reopened this May 20, 2021
@martinkong0806
Copy link

image

The background colour when rotate still seems off

To reproduce
example/lib/bar_chart/samples/bar_chart_sample3.dart line 25 - 27
image

@josh-ksr josh-ksr closed this May 20, 2021
@josh-ksr
Copy link
Contributor Author

Thanks for testing, I'll reopen the pull request when that's fixed.

@josh-ksr josh-ksr reopened this May 26, 2021
@josh-ksr
Copy link
Contributor Author

josh-ksr commented May 26, 2021

@martinkong0806 the background problem when rotating the tooltips should be fixed with my latest changes:

@imaNNeo
Copy link
Owner

imaNNeo commented May 28, 2021

@martinkong0806 Thanks for testing!

@imaNNeo
Copy link
Owner

imaNNeo commented May 28, 2021

@josh-ksr Everything looks good and amazing. I think everyone is going to love it.

@imaNNeo imaNNeo merged commit c24e6b1 into imaNNeo:dev Jun 8, 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.

None yet

3 participants