phetsims / keplers-laws

"Kepler's Laws" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
1 stars 1 forks source link

Update screenshots #193

Closed arouinfar closed 1 year ago

arouinfar commented 1 year ago

@DianaTavares would like to update the screenshots on the website. Unfortunately, we forgot to do this while the sim was in RC, and we will need to publish a maintenance release for the screenshots to update on the website.

@DianaTavares will retake the screenshots and then assign this issue to me to commit them to main and 1.1.

We'll need:

DianaTavares commented 1 year ago

Keplers-laws-screenshot-alt1 Keplers-laws-screenshot-alt2 Keplers-laws-screenshot-alt3 keplers-laws-screenshot-screen1 keplers-laws-screenshot-screen2 keplers-laws-screenshot-screen3 keplers-laws-screenshot-screen4 Keplers-laws-screenshot

DianaTavares commented 1 year ago

This is the link to the files in SharePoint. I call keplers-laws.screenshot to the screenshot of the second screen because I think that the second law is the most famous Kepler law and is better to use that pictures as a representation of this simulation (principal screenshot). Then alt1 is the first screen, alt2 is the third, and alt3 is the four screen

arouinfar commented 1 year ago

Thanks @DianaTavares! I noticed the dimensions of the alt2 screenshot were a bit off, so I retook that one. Otherwise, things looked good and I committed the screenshots to main and the 1.1 branch.

@AgustinVallejo as far as I know, the only way to update the screenshots on the website is to publish a maintenance release of Kepler's Laws. While the screenshots have been committed to the release branch (as is standard), I don't believe the website actually uses the active release branch to populate the sim page, and instead looks to main. Please check in with @mattpen as needed.

pixelzoom commented 1 year ago

I believe that Kepler's Laws was maintaince released on 10/5/23 with new screenshots.

@AgustinVallejo can this issue be closed?

AgustinVallejo commented 1 year ago

Yup, didn't notice this was being discussed here, sorry. Closing!