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 #51

Closed
pixelzoom opened this issue Jan 8, 2019 · 3 comments
Closed

non-standard energy colors #51

pixelzoom opened this issue Jan 8, 2019 · 3 comments
Assignees

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Jan 8, 2019

From phetsims/scenery-phet#455 (comment):

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,

EnergySkateParkColorScheme:

    kineticEnergy: new Color( '#00cc1a' ),
    potentialEnergy: new Color( '#3282D7' ),
    thermalEnergy: new Color( '#FF5500' ), // red colorblind
    totalEnergy: new Color( '#B4B400' ), // dirty yellow

@arouinfar is there any reason why EnergySkateParkColorScheme should not be using PhetColorScheme?

@pixelzoom pixelzoom changed the title use PhetColorScheme for energy colors? non-standard energy colors Jan 9, 2019
@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

@jessegreenberg, in phetsims/scenery-phet#456 the colors used for energy were updated in PhetColorScheme.js. ESP should be using PhetColorScheme for the energy colors in EnergySkateParkColorScheme. I'd imagine that ESPB uses the same color scheme, but to be clear, the change should apply to both sims.

@jessegreenberg
Copy link
Contributor

Alright, this sim (and therefore basics version) is using PhetColorScheme for the energies. Colors before (published version):
before

Colors after (master):
after

Closing.

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

3 participants