Project

General

Profile

Actions

Emulator Issues #11651

closed

Qt: Iterative Input is very buggy

Added by 8times9 about 5 years ago. Updated almost 5 years ago.

Status:
Fixed
Priority:
High
Assignee:
% Done:

0%

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

Description

See the attached video to see a demonstration of these issues.

  1. The first button clicked to start iterative input mapping stays highlighted blue for the entire iterative input duration, whereas the highlight should move to the current button that is being mapped.
  2. The red indicator text trails behind by 2 input buttons. For example, if I input A, B, and then get to X, A's button text will now be in red, and when I get to Y, now B's button text will be red.
  3. Iterative input maps joysticks differently, leading to problems (at least with the controller I tested). It seems to use -+ axes instead of the normal + or - axes, which makes the joystick off-center (see video for a demonstration of this). For example, when I map my controller normally I get "Axis Zr+" for down on the C stick and "Axis Z+" for right on the C stick. When mapping it with iterative input I get "Axis Zr-+" for down and "Axis Z-+" for right.
  4. The mapping window is very laggy during iterative input mapping. If I only want to map the Buttons column (A,B,X,Y,Z,START) and finish and click Close on the window, it takes a number of seconds for the window to finally respond and close.

Files

Iterative Input.mov (4.12 MB) Iterative Input.mov 8times9, 03/31/2019 02:54 AM
Actions #1

Updated by 8times9 about 5 years ago

I didn't realize the bug tracker makes you download the video file to view it, for easier viewing here it is on YouTube: https://youtu.be/01vMbukfoNI

Actions #2

Updated by JMC4789 almost 5 years ago

  • Status changed from New to Accepted
  • Priority changed from Normal to High
  • Milestone set to Current

We will probably be removing iterative input since it seems to be broken beyond repair currently.

Actions #3

Updated by spycrab0 almost 5 years ago

  • Status changed from Accepted to Fix pending
  • Assignee set to spycrab0

https://github.com/dolphin-emu/dolphin/pull/8031

Setting the removal as a possible fix for now.

Actions #4

Updated by spycrab0 almost 5 years ago

  • Status changed from Fix pending to Fixed
  • Fixed in set to 5.0-10049
Actions

Also available in: Atom PDF