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

Update "About" screen to remove R from PhET #35

Closed
kathy-phet opened this issue Feb 25, 2019 · 3 comments
Closed

Update "About" screen to remove R from PhET #35

kathy-phet opened this issue Feb 25, 2019 · 3 comments
Assignees

Comments

@kathy-phet
Copy link

The PhET says ® on it in the 'About' screen. We still want to indicate that this is our trademark so change that line to read ...

PhET™ Interactive Simulations

I also reviewed the link to Terms, Privacy, and Licensing, and while that says we are a registered trademark, I think it is fine. We are registered, and people should know that. The main issue is that our sims our used in so many places we don't want the R symbol to be viewed as a marketing piece in these countries, and violate local laws. This webpage lives on our website, served from our country.

@samreid
Copy link
Member

samreid commented Feb 25, 2019

I pushed a proposed fix. In context it looks like this:

image

The previous unicode for (R) was also embedded in a <sup> superscript. I also tried that for the TM but it made it very high and tiny:

image

So I've committed without the <sup> since it looks like the TM is already rendered kind of like a superscript.

Leaving assigned to @ariel-phet and reassigning to @kathy-phet for review and feedback. You can see it live on phettest.

@kathy-phet
Copy link
Author

kathy-phet commented Feb 25, 2019

I think I prefer the non-sup version, but could go either way. @ariel-phet - do you have a preference here?

@ariel-phet
Copy link

non-sup is the way to go @samreid

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

No branches or pull requests

3 participants