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

PerformanceDisplay styled by CSS instead of JS. #2779

Merged
merged 4 commits into from
Jun 3, 2015
Merged

Conversation

emackey
Copy link
Contributor

@emackey emackey commented Jun 3, 2015

Moved some JS to CSS. You can turn this display on from the URL bar with ?stats=true.

### 1.11 - 2015-07-01

* Breaking changes
* PerformanceDisplay (private API) no longer takes style options. It is now styled with app-overridable CSS. Use `scene.debugShowFramesPerSecond` to enable it as before.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would change this to something like "The performance statistics displayed by setting scene.debugShowFramesPerSecond to true can now be styled using the cesium-performanceDisplay CSS classes in shared.css.

@hpinkos
Copy link
Contributor

hpinkos commented Jun 3, 2015

@emackey everything else looks great! thanks!

@emackey
Copy link
Contributor Author

emackey commented Jun 3, 2015

Thanks both. Ready.

@hpinkos
Copy link
Contributor

hpinkos commented Jun 3, 2015

Great! thanks @emackey

hpinkos pushed a commit that referenced this pull request Jun 3, 2015
PerformanceDisplay styled by CSS instead of JS.
@hpinkos hpinkos merged commit f6257a5 into master Jun 3, 2015
@hpinkos hpinkos deleted the performance-style branch June 3, 2015 18:23
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

Successfully merging this pull request may close these issues.

3 participants