Emulator Issues #5725: Use native full screen mode on OS X - Emulator - Dolphin bug tracker

Actions

Copy link

Emulator Issues #5725

closed

Use native full screen mode on OS X

Added by xuzz over 11 years ago. Updated about 3 years ago.

Status:

Fixed

Priority:

Normal

Assignee:

Category:

GFX

% Done:

Operating system:

OS X

Issue type:

Feature request

Milestone:

Regression:

Relates to usability:

Relates to performance:

Easy:

Relates to maintainability:

Regression start:

Fixed in:

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

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by xuzz over 11 years ago

And I forgot to attach the patch:

Actions

Copy link

Updated by mdeletrain over 11 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 ?

Actions

Copy link

Updated by mdeletrain over 11 years ago

At least, why not make this behavior conditional, depending of the system it us run on ?

Actions

Copy link

Updated by skidau over 11 years ago

Issue 5754 has been merged into this issue.

Actions

Copy link

Updated by stefan.moonen over 11 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.)

Actions

Copy link

Updated by pauldacheez over 11 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.

Actions

Copy link

Updated by xuzz over 11 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.

Actions

Copy link