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

non-standard energy colors #209

Open
pixelzoom opened this issue Jan 9, 2019 · 2 comments
Open

non-standard energy colors #209

pixelzoom opened this issue Jan 9, 2019 · 2 comments
Assignees

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Jan 9, 2019

Noted while investigating phetsims/scenery-phet#455.

PhetColorScheme:

var RED_COLORBLIND = new Color( 255, 85, 0 );
var TAN_ORANGE = new Color( 236, 153, 55 );

HEAT_THERMAL_ENERGY: RED_COLORBLIND,
...
KINETIC_ENERGY: Color.GREEN,  // 0,255,0
...
POTENTIAL_ENERGY: Color.BLUE, // 0,0,255
...
TOTAL_ENERGY: TAN_ORANGE,

PendulumLabColors:

    // Energy colors
    KINETIC_ENERGY_COLOR: 'rgb( 31, 202, 46 )',
    POTENTIAL_ENERGY_COLOR: 'rgb( 55, 132, 213 )',
    THERMAL_ENERGY_COLOR: 'rgb( 253, 87, 31 )',
    TOTAL_ENERGY_COLOR: 'rgb( 0, 0, 0 )',

This sim is using totally different colors for energy. @arouinfar is there any reason why this sim should not be using PhetColorScheme?

@arouinfar
Copy link
Contributor

Thanks @pixelzoom. All PhET sims should use the same colors for the various energies, but PhetColorScheme is a bit antiquated. Changes to PhetColorScheme will be discussed in phetsims/scenery-phet#456, so marking this as status:on-hold until that issue is resolved.

@arouinfar
Copy link
Contributor

arouinfar commented Jan 10, 2019

@jonathanolson, in phetsims/scenery-phet#456 the colors used for energy were updated in PhetColorScheme.js. Pendulum Lab should be using PhetColorScheme for KINETIC_ENERGY_COLOR, POTENTIAL_ENERGY_COLOR, and THERMAL_ENERGY_COLOR.

Since the total energy is represented by a stacked bar, rather than a solid one, I do not believe TOTAL_ENERGY_COLOR needs to be changed at this time. The color specified in PendulumLabColors corresponds to the text label below the bar, and black is an appropriate choice here.

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

No branches or pull requests

3 participants