Project

General

Profile

Emulator Issues #11743

Qt: Move CPU Emulation Engine options to the Advanced tab

Added by 8times9 11 months ago. Updated 6 months ago.

Status:
Fixed
Priority:
Low
Assignee:
-
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
Yes
Relates to performance:
No
Easy:
Yes
Relates to maintainability:
No
Regression start:
Fixed in:
5.0-10996

Description

Currently these options are placed awkwardly inside of two boxes at the bottom of the General config window, first inside one called "Advanced Settings" (with no other options inside except the CPU Emulation Engine), and then inside of a respective CPU Emulation Engine box. (See "Before" screenshot).

This Advanced Settings box is unnecessary when we already have a tab called Advanced, so I made a mockup of what it would look like if the CPU Emulation Engine radio buttons were moved to there (See both "After" screenshots).

Not only does this look much cleaner and have a more logical layout, but it leaves more space available in the General tab should more options need to be added there in the future.

Unlike #11736 this is a change I am unable to make on my own (I tried and failed haha), so an actual developer will need to address this.

Before.png (16.3 KB) Before.png 8times9, 05/28/2019 11:00 PM
Advanced after.png (23.4 KB) Advanced after.png 8times9, 05/28/2019 11:00 PM
General after.png (12.7 KB) General after.png 8times9, 05/28/2019 11:00 PM
Dolphin 3.0.png (13.2 KB) Dolphin 3.0.png 8times9, 05/28/2019 11:16 PM
7445
7446
7447
7448

History

#1 Updated by 8times9 11 months ago

7448

Here's a screenshot all the way back from Dolphin 3.0.

Back then there wasn't an Advanced tab, so the Advanced Settings box actually made sense and housed several other options. But later the addition of options like Custom RTC resulted in the Advanced tab being added, and I guess the CPU Emulation options just never got moved there.

#2 Updated by Helios 11 months ago

  • Easy changed from No to Yes
  • Relates to usability changed from No to Yes
  • Priority changed from Normal to Low

I agree. I don't see why JIT options should remain there these days, and it is definitely a function that nobody should ever touch unless they're specifically trying to help identify an issue with the JIT.

#3 Updated by JMC4789 9 months ago

I initially disagreed, but, I can see why. Is there something else more useful we could put in the main page or do we need a much bigger restructuring at this point?

#4 Updated by 8times9 9 months ago

I can't really think of any options that would be better suited in General, all of them already fit well in their current locations in Dolphin

Though, I'm probably not the best person to be answering this question.

#5 Updated by JosJuice 6 months ago

  • Fixed in set to 5.0-10996
  • Status changed from New to Fixed

Also available in: Atom PDF