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

light theme : font remains white #82

Closed
redmuchroom opened this issue May 24, 2019 · 10 comments
Closed

light theme : font remains white #82

redmuchroom opened this issue May 24, 2019 · 10 comments

Comments

@redmuchroom
Copy link

Dear,

I'm trying to use the boom panel with the light theme. It seems that the font remains white...
do you have any advice to correct it?

thanks,

br
image

@yesoreyeram
Copy link
Owner

I am not able to reproduce this. What version of plugin and grafana ?

@yesoreyeram
Copy link
Owner

image

@redmuchroom
Copy link
Author

hi thanks,
grafana: 6.1.6
plugin : 1.0.0

@yesoreyeram
Copy link
Owner

I couldn't reproduce this yet. If you don't mind, post the panel json without sensitive information?

I just tested in brand new installtion with the above versions.

image

@redmuchroom
Copy link
Author

redmuchroom commented May 24, 2019

here it is: (better in a file :))
paneljson.txt

@redmuchroom
Copy link
Author

hi ,
by any chance did you manage to reproduce the issue?
if you need any help from me, feel free :)

thanks

@yesoreyeram
Copy link
Owner

Sorry i couldn't reproduce this. Any help appreciated. Feel free to raise PR if you know the root cause.

@yesoreyeram yesoreyeram added help wanted Extra attention is needed and removed investigating labels Jun 3, 2019
@redmuchroom
Copy link
Author

hi,

I just put the debug mode... and indeed it is confirmed that the text color is white :) .
Do I miss something in the configuration? I think more and more that it is a misused from my side again...

image

thanks

@yesoreyeram
Copy link
Owner

Got it.. There is a piece of code which always return white color irrespecitive of theme.

https://github.com/yesoreyeram/yesoreyeram-boomtable-panel/blob/master/src/app/boom/BoomSeries.ts#L126

Fix coming soon.

@yesoreyeram yesoreyeram removed the help wanted Extra attention is needed label Jun 6, 2019
@yesoreyeram
Copy link
Owner

@redmuchroom Fix available in latest master branch

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

No branches or pull requests

2 participants