justgage icon indicating copy to clipboard operation
justgage copied to clipboard

Support for refresh(options) where options will extend current options

Open pbowyer opened this issue 8 years ago • 4 comments

I was really excited to see config parameters can now be passed to refresh(), hoping this would let me change the colour of the text when the dial was completely full (it's hard to tell on a small gauge if you've got there or are a couple of items off being 'full').

However, all my attempts to change settings have failed. For example:

gauge.refresh(100, null, {titleFontColor: '#ffcc00'});
gauge.refresh(105, false, {labelFontColor: '#ffcc00'});
gauge.refresh(115, null, {valueFontColor: '#00ccff'});

This is using the current GitHub version of justgage.js (commit: ce021d6). Stepping in with a debugger the config settings are updated on line 870, but there appears no code to render these changes after it?

pbowyer avatar Aug 23 '16 07:08 pbowyer

You are correct, there is no code to render changes. This remains to be done.

toorshia avatar Aug 23 '16 09:08 toorshia

Yea, I just ran into this myself. I am trying to update the min configuration during a refresh. Any ETA on fixing the rendering?

javaday avatar Aug 23 '16 19:08 javaday

I'd like to roll this out in a month or so.

toorshia avatar Aug 23 '16 19:08 toorshia

In development...

deezone avatar Sep 19 '19 16:09 deezone