Project

General

Profile

Actions

Emulator Issues #11292

open

Qt/macOS: A few buttons that don't use the native system UI

Added by 8times9 over 5 years ago. Updated over 1 year ago.

Status:
Accepted
Priority:
Normal
Assignee:
-
% Done:

0%

Operating system:
OS X
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

I found two instances of buttons not using the native system UI on macOS, see attached screenshots.

They are the mapping modification buttons in the advanced mapping window and the dropdown menu in the NetPlay window.


Files

Actions #2

Updated by Techjar over 5 years ago

These are unfortunately a result of MacOS not having a native style for QPushButtons with QSizePolicy::Expanding, and QToolButton in general. Not sure if there's any workaround for this.

Actions #3

Updated by Techjar over 5 years ago

See how this build affects the "Configure Input" window: https://dl.dolphin-emu.org/prs/pr-7279-dolphin-latest.dmg

Actions #4

Updated by 8times9 over 5 years ago

Partially fixed in 5.0-8480

Actions #5

Updated by 8times9 about 5 years ago

A few more have come up:

  • Presets button in Game Config Editor tab of Properties
  • Calibrate buttons in mapping window
Actions #6

Updated by 8times9 about 5 years ago

Presets buttons have been fixed in 5.0-9701

Actions #7

Updated by 8times9 about 5 years ago

Here's a screenshot of what the Calibrate buttons look like on macOS.

Actions #8

Updated by Techjar almost 5 years ago

The only solution for the calibrate buttons would be to make them a QPushButton with a menu, like the game config editor presets menu. This is not an acceptable solution though, as it makes them less intuitive to use, since we can only have a menu and not a default push action.

Actions #9

Updated by 8times9 almost 5 years ago

That's true, but I feel like the buttons being slightly less convenient to use is better than how they currently look. Of course, there's no reason to punish Linux and Windows users for a macOS limitation, so maybe they could be made into QPushButtons only on macOS.

If this were a much less-used window like somewhere in debugging, leaving the buttons as is would be just fine. But the GameCube/Wiimote mapping windows are very frequently visited – I know I use them at least once every time I play games on Dolphin.

Actions #10

Updated by Techjar almost 5 years ago

The particular way we use QToolButton there isn't feasible to change without a complete redesign of it, unfortunately. It's not a simple matter of swapping out the button type. Various behaviors are dependent on that being a button with a default action.

Actions #11

Updated by 8times9 almost 5 years ago

Understandable, thanks for checking on it though.

Actions #12

Updated by spycrab0 almost 5 years ago

Actions #13

Updated by OatmealDome over 1 year ago

While most of these buttons have been removed, there are still two instances of QToolButton being used: the Calibrate button in MappingWindow, and the Export as GCI button in the memory card manager. (The latter is particularly ugly looking because it doesn't even match the size of the other QPushButtons.)

Actions #14

Updated by Billiard26 over 1 year ago

  • Operating system OS X added
  • Operating system deleted (N/A)
Actions

Also available in: Atom PDF