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 puffin_egui to egui 0.24 #166

Merged
merged 6 commits into from
Nov 24, 2023
Merged

Update puffin_egui to egui 0.24 #166

merged 6 commits into from
Nov 24, 2023

Conversation

emilk
Copy link
Collaborator

@emilk emilk commented Nov 20, 2023

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

I also made puffin_egui even better with a new function show_viewport_if_enabled. On supported backends (eframe), it will open the puffin profiler flamegraph in its own viewport:

Screenshot 2023-11-20 at 21 11 04

On other egui backends it will fall back to open it in a window.

@emilk emilk changed the title Update to latest egui master Update to egui 0.24 Nov 23, 2023
@emilk emilk marked this pull request as ready for review November 23, 2023 16:48
@emilk
Copy link
Collaborator Author

emilk commented Nov 23, 2023

When merged we should release a new puffin_egui (but only that crate; and maybe puffin_viewer)

@emilk emilk changed the title Update to egui 0.24 Update puffin_egui to egui 0.24 Nov 23, 2023
@mergify mergify bot merged commit 949b0c3 into main Nov 24, 2023
5 of 6 checks passed
@mergify mergify bot deleted the emilk/update-egui branch November 24, 2023 08:24
@TimonPost
Copy link
Member

I'll make a release of puffin_egui and puffin_viewer

@TimonPost
Copy link
Member

Released 0.24 puffiin egui and 0.18 of the viewer

plasticbox pushed a commit to alt9github/puffin that referenced this pull request Nov 29, 2023
### Checklist

* [x] I have read the [Contributor Guide](../CONTRIBUTING.md)
* [x] I have read and agree to the [Code of
Conduct](../CODE_OF_CONDUCT.md)
* [x] I have added a description of my changes and why I'd like them
included in the section below

### Description of Changes

I also made `puffin_egui` even better with a new function
`show_viewport_if_enabled`. On supported backends (`eframe`), it will
open the puffin profiler flamegraph in its own viewport:

<img width="844" alt="Screenshot 2023-11-20 at 21 11 04"
src="https://github.com/EmbarkStudios/puffin/assets/1148717/7e06f433-b9d1-4b6f-bfc5-eeae4bef5986">

On other egui backends it will fall back to open it in a window.
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.

2 participants