Skip to content

Correct font on axes and functionality to add axes-labels#64

Merged
JelteMX merged 4 commits intomendixlabs:masterfrom
Mcpoowl:master
Feb 23, 2016
Merged

Correct font on axes and functionality to add axes-labels#64
JelteMX merged 4 commits intomendixlabs:masterfrom
Mcpoowl:master

Conversation

@Mcpoowl
Copy link
Copy Markdown
Contributor

@Mcpoowl Mcpoowl commented Feb 18, 2016

Previously the font supplied was only used in the legend. Fixed it so that the font is also used on the Axes.

Furthermore I added the functionality to add a label to the X and Y-axis. Leave empty for no value

Previously only the legend used the font property. This fixed that; both
axes now use the same font as set in the property. Fixed for all
relevant charts (Doughnut, Pie and polar don't have axes)
You can now add a label on both the X-axis as the Y-axis.
Added two properties in XMLs for all relevant charttypes. When a string
is given, that value will be used as a label. Leave empty for no label.
@JelteMX JelteMX self-assigned this Feb 18, 2016
Fix merge conflicts
@Mcpoowl
Copy link
Copy Markdown
Contributor Author

Mcpoowl commented Feb 18, 2016

Had some conflicts @JelteMX. Should be solved with my latest commit!

@JelteMX
Copy link
Copy Markdown
Contributor

JelteMX commented Feb 18, 2016

Cool! Will test it and add it to the widget

Added functionality for a Charttitle. Since the default fontSize of the
title is very small (12px) I added an option to set the size aswell
@Mcpoowl
Copy link
Copy Markdown
Contributor Author

Mcpoowl commented Feb 19, 2016

Added one more commit @JelteMX . This one adds the functionality to give your charts a title. Since the default fontsize is very small (12px ) I added a property to set the size aswell.

@JelteMX
Copy link
Copy Markdown
Contributor

JelteMX commented Feb 22, 2016

I'll review the changes this week and push an update to the AppStore :-)

@Mcpoowl
Copy link
Copy Markdown
Contributor Author

Mcpoowl commented Feb 22, 2016

Awesome!

@JelteMX JelteMX merged commit 78c7cc0 into mendixlabs:master Feb 23, 2016
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.

2 participants