Closed DianaTavares closed 5 years ago
In #39 was decided to change the zoom limits to be able to see the curve for all the temperatures in the simulation. The this is added to the polishing list:
[x] For the x-zoom allow 2 extra clics for the (- )zoom for get to 48
[x] For the y-zoom the factor of change in each click is going to be 5. The max value is going to be 700 and in each click in the (+) zoom the max number is going to be divided by 5 rounded to a significant figure. It's going to allow 11 clicks in total. Something like this:
Max value in y-axe | click number |
---|---|
0.00001 | 11 |
0.00007 | 10 |
0.0004 | 9 |
0.002 | 8 |
0.009 | 7 |
0.04 | 6 |
0.22 | 5 |
1.12 | 4 |
5.6 | 3 |
28 | 2 |
140 | 1 |
700 |
140 zoom is going to be the default
All changes implemented in 1.0.0-dev.12 dev version. Label sizes and decimals have been changed to accommodate the scaling, probably something worth looking at in design.
Nice work @arnabp! in the version that dev.12 the line for the labels of the electromagnetic spectrum still bold. Can you make it a little thinner?
thinner like this line (half of side that the line in the Y axe):
And also, the range of values for the electromagnetic spectrum is not correct. Infrared should be from 700 nm (.7 mictrom) to 10^5 nm (100 microm)
Appropriate changes were made, see next round of pixel polishing at https://github.com/phetsims/blackbody-spectrum/issues/62
In the 10/18/2018 sim design meeting, looking the actual version in PhET test for UI adjustments. Here is the TODO list that resulted:
[x] The cues arrows for the Graph Values should be the same size that the ones in the thermometer.
[x] Change the color of the dot of Graph Values to white in the normal view and black in the projector mode.
[x] Hide the coordinate of the dot (the one inside the blue circule in the picture) when the numbers are bigger that the axe value.
[x] increase the size of the text for the intensity value and a bigger space between the text and the red rectangle
[x] make the line of labels thinner