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

iOS: Support putting UI next to the dynamic island #5211

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

frederik-uni
Copy link
Contributor

@frederik-uni frederik-uni commented Oct 2, 2024

winit::Window::inner_size returns size of safe area on iOS. use winit::Window::outer_size on iOS
The dimensions of outer_size include the title bar and borders, but as far as I know there is no way to actually display the title bar or borders on iOS so it should be fine.

winit::Window::inner_size returns size of safe area on iOS.
use winit::Window::outer_size on iOS
Copy link

github-actions bot commented Oct 2, 2024

Preview available at https://egui-pr-preview.github.io/pr/5211-iOS-black-bar
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

Copy link
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

I think this makes sense. Now we will still have to manually work around the safe area (until this is done: rust-windowing/winit#3890) but at least egui will render on the whole screen

@lucasmerlin lucasmerlin added bug Something is broken eframe Relates to epi and eframe iOS egui-winit porblems related to winit and removed eframe Relates to epi and eframe labels Oct 2, 2024
@frederik-uni
Copy link
Contributor Author

@lucasmerlin I updated my safe area implementation so it is actually usable. it could be merged and the get_safe_area_insets could be replaced while upgrading winit in the future

@frederik-uni
Copy link
Contributor Author

@emilk is this alright?

@emilk emilk changed the title remove black bar on iOS fixes #3547 Remove black bar on iOS Oct 23, 2024
crates/egui-winit/src/lib.rs Outdated Show resolved Hide resolved
@emilk emilk added the eframe Relates to epi and eframe label Oct 23, 2024
@emilk emilk changed the title Remove black bar on iOS iOS: Support putting UI next to the dynamic island Oct 23, 2024
@emilk emilk merged commit e02be45 into emilk:master Oct 23, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken eframe Relates to epi and eframe egui-winit porblems related to winit iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Black bar at bottom on iOS targets
3 participants