CleverRaven / Cataclysm-DDA

Cataclysm - Dark Days Ahead. A turn-based survival game set in a post-apocalyptic world.
http://cataclysmdda.org
Other
10.26k stars 4.11k forks source link

[Android] Mission menu scrolling bug #73606

Open Cupidatis opened 4 months ago

Cupidatis commented 4 months ago

Describe the bug

Scrolling mission menu is always skipping 1 position

Attach save file

sky-trimmed.tar.gz

Steps to reproduce

1.Spawn some NPCs 2.Ask them for missions 3.Try to scroll mission menu

Expected behavior

Normal scrolling without skipping

Screenshots

No response

Versions and configuration

Additional context

Easy to reproduce without mods.
Also can't disable Imgui in options like in 1 month early version

RenechCDDA commented 4 months ago

Can you update and check again? This should have been resolved by #73554. That PR should be in the build you're reporting.

Cupidatis commented 4 months ago

Can you update and check again? This should have been resolved by #73554. That PR should be in the build you're reporting.

73554 was implement in this version

cdda-experimental-2024-05-09-0820 fa8ae46 [64-bit] where I am playing. I will try fresher version, but first need to do backup.

Cupidatis commented 4 months ago

Updated to Cataclysm-DDA experimental build 2024-05-11-0139 . Didn't help. Also new bug. Screenshot_2024-05-11-08-37-07-225_com cleverraven cataclysmdda experimental This line below don't disappear. Screenshot_2024-05-11-09-32-56-444_com cleverraven cataclysmdda experimental And this...

Elshad19 commented 4 months ago

I copied the post from the dublicated post (#73664):

The latest experimental build implemented the new UI to mission selection. However there is a problem with it. While scrolling, it skips a mission and jumps to the other mission. There is no way to select the mission in middle. Check the video below for visual demonstration.

Steps to reproduce

1) Have more than 3 missions at the same time, 2) Press "M" to view your missions, 3) Scroll to reproduce the problem.

Expected behavior

While I support scrolling quickly through the mission list, I think there should be an additional option to manually select one mission at a time instead of jumping over each other when scrolling.

Screenshots

https://github.com/CleverRaven/Cataclysm-DDA/assets/87479123/292e06ad-8c85-45e0-a066-c72a34581d1d

Pay attention to the names of the missions.

Versions and configuration

Version: ef430d2 https://github.com/CleverRaven/Cataclysm-DDA/releases/tag/cdda-experimental-2024-05-11-0139

Platform: Android (10.0) & 4Gb Ram.

RenechCDDA commented 4 months ago

I was finally able to confirm this after loading it on my phone. Tentatively I'd blame the input timeout https://github.com/CleverRaven/Cataclysm-DDA/blob/a5ba2a01bb53855632ae9fc25386a997c2ec2a90/src/mission_ui.cpp#L99

Will be looking into a fix

Cupidatis commented 1 month ago

I was finally able to confirm this after loading it on my phone. Tentatively I'd blame the input timeout

https://github.com/CleverRaven/Cataclysm-DDA/blob/a5ba2a01bb53855632ae9fc25386a997c2ec2a90/src/mission_ui.cpp#L99

Will be looking into a fix

Also new (sub?)bug with new settings screen for npc. Same thing: scrolling more than one option for click.