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
Remove "EFB Copies Disabled" option from hotkey toggling cycle. #971
Conversation
@dolphin-emu-bot rebuild I'm +1 for this change, so LGTM |
I'm not sure if this is desired behaviour or not. It might be a better idea to have pressing the toggle key enable EFB copies if it is pressed, or at the very least display an OSD message saying "Please enable EFB Copies in the graphics options" so the user has an idea why the button isn't doing anything. |
I can add some lines so that EFB copies will get enabled if they initially were disabled, and only after enabling EFB copies would they cycle between EFB2RAM and EFB2Tex. How does that sound? |
That sounds potentially slightly confusing. |
I would also think that removing the Disabled from the hotkey binding is clearer. Are there any titles that would benefit from having EFB copies disabled? So that user (or gameini) might end up selecting "disabled" in the first place (and then end up wondering why the he/she can't toggle)? |
I'd go for that. shrug |
Yeah, the message would probably be best. |
Ok done, please instruct if I somehow need to merge these commits to one commit in git. |
@dolphin-emu-bot rebuild |
+1 for this change. No one should be disabling EFB Copies so freely. Besides, it's pretty useless. In every experience I've had with the EFB toggle, the disabled step just means more button presses. |
still LGTM |
Remove "EFB Copies Disabled" option from hotkey toggling cycle.
I would imagine "EFB Copies" hotkey is 99% of the time used to toggle between EFB2RAM and EFM2Texture by users that have less-than-optimal Dolphin hardware and can't use EFB2RAM all the time in games that require it (often only for short periods). So the toggling workflow could be streamlined if there wasn't "EFB Copies = Disabled" option in the cycle.
This commit removes the "Disable EFB Copies" option from the hotkey toggling behaviour. It can still be chosen from the GUI if someone enjoys looking at broken graphics. And if Disabled is chosen, the hotkey won't have apparent effect.
And please correct me (and close the PR) if I'm totally off with the initial assumption of the need for the hotkey.