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

Crafting recipe search UI does not show up for some languages when using tiny game window #48700

Closed
olanti-p opened this issue Apr 29, 2021 · 1 comment · Fixed by #58609
Closed
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Info / User Interface Game - player communication, menus, etc. Translation I18n

Comments

@olanti-p
Copy link
Contributor

olanti-p commented Apr 29, 2021

Describe the bug

Crafting recipe search UI does not show up for some languages when using tiny game window

Steps To Reproduce

  1. Set language to English, set game window to the minimum (80x24)
  2. Enter crafting menu
  3. Hit /. Search window pops up
  4. Get back to settings, choose Russian
  5. Enter crafting menu
  6. Hit /. Nothing happens.
  7. Drag bottom border of the game window down. When window height reaches 26, the search UI shows up.

On step 6, you can still blindly type something and press Enter, and the search will work, so it's likely a display bug.

Screenshots

80x26, English
image

80x26, Russian
image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1909
  • Game Version: 0.E-10395-g4525b4f [64-bit] (aka build 11581)
  • Graphics Version: Tiles
  • Game Language: Русский [ru]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Bionic Professions [package_bionic_professions]
    ]
@actual-nh actual-nh added Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Info / User Interface Game - player communication, menus, etc. Translation I18n <Bug> This needs to be fixed labels Apr 29, 2021
@Qrox
Copy link
Contributor

Qrox commented Apr 30, 2021

My guess is that due to the longer russian text, the search UI becomes higher than the game window, which causes window creation to fail. Limiting the search window height to the game window height in string_input_popup::create_window may fix this problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Info / User Interface Game - player communication, menus, etc. Translation I18n
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants