Emulator Issues #5725
closedUse native full screen mode on OS X
0%
Description
Name of clone:
?
Purpose of code changes on this branch:
Switch Mac to use native full screen mode rather than taking over the current space and (really?) changing the system screen resolution. This introduces a dependency on OS X 10.7 Lion. Tested configuration options (automatically enter full screen mode; render to main window; others) and also hotkeys, and all work as expected. This also removes the now-superfluous "Fullscreen Resolution" video option (on OS X only).
When reviewing my code changes, please focus on:
- Is depending on OS X 10.7 Lion or later acceptable? The statistics I've seen leave it and later with just over 50% market share, but it is only increasing. More technical users (e.g. those likely to be using Dolphin) likely have a higher adoption rate.
- Is removing the "Fullscreen Resolution" option acceptable? This is now handled by the main resolution setting, as entering fullscreen is just equivalent to resizing the window.
- Are there any other configurations or edge cases that I don't handle?
After the review, please pull into branch:
master
Updated by mdeletrain almost 12 years ago
I find excluding 10.6 users a bit rough ! As you said they still represent a lot of Mac users (in fact, this is the bigger part of mac users, according to this site : http://www.netmarketshare.com/operating-system-market-share.aspx?qprid=10&qpcustomb=*2).
Lot of people keep using 10.6 because of 10.7+ introducing some nasty behaviors they don't want.
Migration from 10.5 to 10.6 with an upgrade price around 50$ was done in only a few months. Migration to 10.6 to 10.7+ with an upgrade price at 20$ is still not done 16 months later... guess why ?
Updated by mdeletrain almost 12 years ago
At least, why not make this behavior conditional, depending of the system it us run on ?
Updated by skidau almost 12 years ago
Issue 5754 has been merged into this issue.
Updated by stefan.moonen almost 12 years ago
I've played a bit with the build you supplied yesterday. Mainly SMG and NSMB, and they both run great with this patch, no perceivable speed difference.
Also, the native fullscreen is a nice touch as it provides the option to easily switch between spaces. Though I think the main question that remains is if it's desirable that you can only play in the native desktop resolution now. I don't think it has to be a problem because the actual rendering resolution makes all the difference, and that is a separate configuration option. Maybe make the native fullscreen functionality an option in the configuration? (On by default for Lion and ML.)
Updated by pauldacheez almost 12 years ago
I vote we remove the fullscreen resolution option entirely and just use whatever res the OS is using. Poof, one less useless option in the Graphics settings. I'm sure Neobrain approves, he loves removing things he deems useless.
Updated by xuzz almost 12 years ago
Support OS X 10.6 and OS X 10.7+. If built on OS X 10.7+, then it will include native full-screen support when running on OS X 10.7+ and will dynamically fall back to the old mode when running on OS X 10.6. If built on OS X 10.6, it will work as before.
Updated by xuzz almost 12 years ago
Fix dangling else eating next block when building on 10.7+.
Updated by xuzz almost 12 years ago
(Who moved the binaries folder? I was testing the wrong thing.)
Updated by parlane almost 12 years ago
PLease update your patch file for latest master.
Updated by xuzz almost 12 years ago
This has been merged in, although with an incorrect commit message ("support retina display on mac").
Updated by NeoBrainX almost 12 years ago
Why have DSPDebuggerLLE::Refresh and TASInputDlg::TextBoxHasFocus/HasFocus been changed to be virtual?
Updated by xuzz almost 12 years ago
Fixes a few dozen clang warnings. Feel free to remove it, though; it's not important.
Updated by Billiard26 almost 12 years ago
- Category set to gfx
- Operating system OS X added
Updated by bradenb95 over 11 years ago
- Status changed from New to Fixed
This issue is now fixed. Your patch was a tad bit more complicated than needed.
Updated by Dentomologist over 3 years ago
I can't make builds that old without installing WX, but this looks to have been fixed by these commits:
https://dolphin-emu.org/download/dev/befe6e696212ea7f997302f3233442c4f18cc890/
https://dolphin-emu.org/download/dev/39965e894d2d014cd4b872e071ea57fd1a69e11c/.