Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Android] Mission menu scrolling bug #73606

Open
Cupidatis opened this issue May 9, 2024 · 5 comments
Open

[Android] Mission menu scrolling bug #73606

Cupidatis opened this issue May 9, 2024 · 5 comments
Labels
ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds OS: Android Issues related to Android operating system (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Cupidatis
Copy link

Cupidatis commented May 9, 2024

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

  • OS: Android
    • OS Version: Manufacturer: Xiaomi; Model: M2101K7BNY; Release: 13; Incremental: V14.0.3.0.TKLRUXM;
  • Game Version: cdda-experimental-2024-05-09-0820 fa8ae46 [64-bit]
  • Graphics Version: Tiles
  • Game Language: Русский <color_dark_gray>(89.5%) [ru]
  • Mods loaded: [
    Тёмные Дни Впереди [dda],
    Aftershock [aftershock],
    Магиклизм [magiclysm],
    Разум над материей [mindovermatter],
    <color_cyan>Arcana and Magic Items [Arcana],
    Небесный остров [skyisland],
    <color_cyan>Arcana/Aftershock Patchmod [Arcana_aftershock_Patch],
    <color_cyan>Arcana/Magiclysm Patchmod [Arcana_Magiclysm_Patch],
    Помпезные перки [bombastic_perks],
    Портальные штормы игнорируют НПС [personal_portal_storms],
    Отключить потребности у НПС [no_npc_food],
    Медленный рост фунгалоидов [no_fungal_growth],
    Характеристики за убийства [stats_through_kills]
    ]

Additional context

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

@Cupidatis Cupidatis added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label May 9, 2024
@RenechCDDA
Copy link
Member

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

@Cupidatis
Copy link
Author

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
Copy link
Author

Cupidatis commented May 11, 2024

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
Copy link

Elshad19 commented May 11, 2024

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

Screenrecorder-2024-05-11-15-05-35-621.0.mp4

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 RenechCDDA added (S2 - Confirmed) Bug that's been confirmed to exist OS: Android Issues related to Android operating system ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels May 13, 2024
@RenechCDDA
Copy link
Member

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

ctxt.set_timeout( 10 );

Will be looking into a fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds OS: Android Issues related to Android operating system (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

3 participants