Skip to content

Conversation

@Jack-Valentine
Copy link
Contributor

Please consider the following before submitting a pull request:

old example: http://codepen.io/pen?template=oLjMwR

tooltip titleFontColor, bodyFontColor, footerFontColor not working so
changed core.tooltip.js options

문승찬 added 2 commits June 8, 2016 10:16
bodyColor  -> bodyFontColor
titleColor -> titleFontColor
footerColor -> footerFontColor
Change tooltip font color options
this options changed in tooltip core
bodyColor  -> bodyFontColor
titleColor -> titleFontColor
footerColor -> footerFontColor
@Jack-Valentine
Copy link
Contributor Author

Jack-Valentine commented Jun 8, 2016

I made a new issue. #2731

@etimberg
Copy link
Member

etimberg commented Jun 8, 2016

+1 to merge

@etimberg
Copy link
Member

etimberg commented Jun 8, 2016

@duerahan looks like tests are still failing. Do they work locally for you?

titleColor options is not available
titleColor change to titleFontColor
titleColor change to titleFontColor
fixed global variable name fix
callback variable name fix
- 'titleColor' to 'titleFontColor'
- 'bodyColor' to 'bodyFontColor'
- 'footerColor' to 'footerFontColor'
@Jack-Valentine
Copy link
Contributor Author

@etimberg Sorry. I had some mistakes. I fixed global variable name(29bc3b0). And update test case(99d3f42, d6e5b5e).

@etimberg
Copy link
Member

etimberg commented Jun 8, 2016

Thanks @duerahan

@etimberg etimberg merged commit 958f8ca into chartjs:master Jun 8, 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