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

chart backgound and text colors #435

Open
benstoltz opened this issue Jun 27, 2018 · 1 comment
Open

chart backgound and text colors #435

benstoltz opened this issue Jun 27, 2018 · 1 comment

Comments

@benstoltz
Copy link
Member

@tomwayson Starting this conversation up because we need to make decisions on this. This will be covering text, background, and chart colors. Chart colors will need to be thought of in the context of multiseries as well.

Previously I'd come up with something like: styles: { background: { color: '#xyz', alpha: 0.3 } }
alpha is needed for background color because it defaults to 0, completely transparent.

That being said AmCharts docs say: Background color. You should set backgroundAlpha to >0 value in order background to be visible. We recommend setting background color directly on a chart's DIV instead of using this property. <--- makes me think we should re-consider how we handle this.

General thoughts @tomwayson ? (previous discussion is located at: #382

@tomwayson tomwayson changed the title Style Colors chart backgound and text colors Jun 27, 2018
@tomwayson tomwayson added Medium and removed High labels Jun 27, 2018
@tomwayson
Copy link
Member

Good to know about setting on the div, that would seem to clear up some of the sticking points: #382 (comment)

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

No branches or pull requests

2 participants