Project

General

Profile

Actions

Emulator Issues #7940

closed

Unified Controller Menu is Too Tall

Added by D.gamefreak.95 about 10 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
UI
% Done:

0%

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

Description

Game Name/Game ID??
No game GUI related

What's the problem? Describe what went wrong in few words.

After the Wii Mote menu and the GCpad Menu were unified into the "Controller Menu" the menu is too long resulting in cut off buttons on lower resolution displays that have higher than normal DPI ( makes display text larger for poor vision) specifically 720p 125% DPI in my case.

What did you expect to happen instead?
A simple scroll bar that appears if the menu is not long enough to be displayed fully i.e on lower resolutions with high DPI, so that menu options remain visible.

What steps will reproduce the problem?

  1. In Windows 7 go to Control Panel\Appearance and Personalization\Display\Screen Resolution
  2. Set display resolution to 720p
  3. Go to Control Panel\Appearance and Personalization\Display, make text larger than 100% for display

Dolphin 3.5 and 3.5-367 are old versions of Dolphin that have
known issues and bugs, so don't report issues about them and test the
latest Dolphin version first.
Which versions of Dolphin did you test on?

Git 4.0-4647

Does using an older version of Dolphin solve your issue? If yes, which
versions of Dolphin used to work?

Any version with separate menus in GUI

What are your PC specifications? (including, but not limited to: Operating
System, CPU and GPU)

Windows 7 x64
720p 55inch display (text magnified via windows to 125% original size)

Is there any other relevant information? (e.g. logs, screenshots,
configuration files)

Image example: http://i.imgur.com/1exD8h3.png
The okay and cancel buttons are missing along with some Wii mote options


Related issues 1 (0 open1 closed)

Has duplicate Emulator - Emulator Issues #7957: dolphin controller configuration issueDuplicate

Actions
Actions #2

Updated by JMC4789 about 10 years ago

caught one of the wrong people.

Actions #3

Updated by MayImilae about 10 years ago

  • Status changed from New to Accepted
  • Category set to ui
  • Relates to usability set to Yes

I changed my desktop resolution to see if I could reproduce this. At 1366x768 (a very common resolution on craptops and 720p HDTVs) there is no trouble at all.

http://i.imgur.com/RNKVh8G.jpg

But on 1280x720 (a resolution you won't find very often) there was indeed an issue as shown by the poster.

http://i.imgur.com/TPZkXUN.jpg

It is definitely too tall. While it isn't a concrete rule anywhere, the Dolphin UI has historically been designed to fit on a minimum resolution of 800x600. This definitely exceeds that. I'm not sure how to fix it though... Three columns wide would be pushing it. We could replace the "configure" text with a gear icon or something to make them square... shrug

Actions #4

Updated by MayImilae about 10 years ago

The title was getting on my nerves.

Actions #5

Updated by D.gamefreak.95 about 10 years ago

I know GUI can be a big pain, so I took the liberty of trying to redesign the menu a bit in paint.

http://i.imgur.com/6r4xoOh.png

I think its better than 3 columns and it's just about 600 pixels tall, if you guys like it feel free to use it

Actions #6

Updated by MayImilae about 10 years ago

The "general settings" are all general -wii- settings, and yet it's under the gamecube settings in your mockup, which is confusing. A two column idea was discussed and passed over during the initial PR due to either having to mix wii settings in with the gamecube settings or having the wii settings be way taller than the gamecube settings.

Actions #7

Updated by rukariosake about 10 years ago

Delete GameCube Adapter box, move checkbox with text "Adapter not detected" to Gamecube Controllers, and rename "Adapter not detected" to "GameCube Adapter (not detected)".

Rename "Wiimotes" to "Wii Controllers" and throw Balance Board into there, delete Balance Board's own box.

Then that will might fit in 720p monitor, only with normal text size.

I'm thinking maybe could use two collapse buttons, for "Real Wiimotes" and "General Settings", they should NOT be collapsed by default.

Actions #8

Updated by MayImilae about 10 years ago

sigh Now I have to respond to all of that >_<.

-The "box" divider is a part of UI conventions for WX. There's a reason for that: having a checkbox immediately below a bunch of dropdown lists makes it easy to miss. It needs that separation for clarity.

-The balance board is in a checkbox instead of the dropdown because the wii doesn't have support for multiple balance boards. However, according to wikipedia...

"Only one Balance Board can be synchronized with the Wii at a time and the board uses the fourth player controller connection, replacing any Wii Remotes that are currently bound to that position. Due to these two limitations there is no ability to use multiple Balance Boards simultaneously."

So with that in mind, it could be added to only the fourth wiimote dropdown... Assuming WX will allow that.

-Collapse buttons is harder than you might think. WX, after all.

Actions #9

Updated by JosJuice about 10 years ago

Issue 7957 has been merged into this issue.

Actions #10

Updated by MayImilae almost 10 years ago

  • Milestone set to Current

This should probably be addressed before 5.0. It's not hard, just... hard to figure out.

Actions #11

Updated by MayImilae almost 10 years ago

  • Status changed from Accepted to Fix pending
Actions #12

Updated by MayImilae almost 10 years ago

  • Status changed from Fix pending to Fixed
Actions #13

Updated by D.gamefreak.95 almost 10 years ago

Thank you very much!

Actions #14

Updated by D.gamefreak.95 almost 10 years ago

Sorry to double post, I forgot to mention, to be clear this has fixed my issue completely even with higher than normal DPI scaling, thanks again

Actions

Also available in: Atom PDF