Add touch navigation for game menus by Solessfir · Pull Request #1014 · Try/OpenGothic

Solessfir · 2026-10-12T02:58:20Z

Adds direct touch navigation for game menus, using Tempest's Android touch events and UI scale.

Tap menu entries and toggle choices directly.
Drag sliders in 1% steps and the stock view-distance percentage choices in 10% steps.
Limit mobile UI scaling to fit the screen, including a configured INTERFACE.Scale multiplier.
Keep desktop input behavior unchanged.

Depends on Try/Tempest#127 for Android density-based implUiScale. The view-distance choices remain unhooked from rendering; this changes their interaction, not their graphics behavior.

Journal dialogs and text entry aren't covered yet.

Try · 2026-10-12T21:03:30Z

Up/down swipe moves one entry

But.. why? Any reason not to implement clickable UI?

Solessfir · 2026-10-12T21:23:51Z

On the phone UI is so small that it is near impossible to press what you want.
We'd have to increase game UI by like x3 to make it work. So without it - I found that swipes are the only good option that worked for me.
Can change to touch and increase UI if mobile detected and make it an .ini config option.

Try · 2026-10-12T21:57:35Z

On the phone UI is so small...

Too small? That is likely due to lack of AndroidAPI::implUiScale - maybe if this entry is implemented, then UI will naturally become big enough?

Solessfir · 2026-10-12T21:58:43Z

Will investigate and post updates.

Solessfir · 2026-10-13T01:16:33Z

Implemented AndroidApi::implUiScale in Try/Tempest#127. With density scaling and a screen-fit cap, the menus are large enough for direct taps. Updated this PR to use tap selection, draggable sliders and tappable choices instead of swipe navigation.

Before:

After:

Solessfir · 2026-10-13T01:37:15Z

Not very happy with this percentageRange but I think it is better that hardcode "20%|40%|60%|80%|100%|120%|140%|160%|180%|200%|220%|240%|260%|280%|300%"; values.
IDK, need your opinion on this :)

Solessfir · 2026-10-13T11:12:34Z

Dropped percentageRange. Percentage choices now just use the script's own entries, so no parser and no hardcoded values.

Try

but I think it is better that hardcode "20%|40%|60%|80%|100%|120%|140%|160%|180%|200%|220%|240%|260%|280%|300%"; values.

There are 2 variants of such control (script driven):

Enum like (Range of view) - those have explicit values
Slider - those have step value stored at userFloat[0]
(OpenGothic actually doesn't have case 2 implemented - changes slider by 3% always)

Basically, it's not for engine to really decide - we just draw what UI-script wants :)

Try · 2026-10-14T22:03:12Z

+  if(auto window = dynamic_cast<const MainWindow*>(w)) {
+    const float scale = window->uiScale() * mul;
+#if defined(__MOBILE_PLATFORM__)
+    return std::min(scale,0.85f*std::min(float(window->w())/640.f,float(window->h())/480.f));


AFAIK model resolution for gothic ui is 800x600

Try · 2026-10-14T22:07:04Z

+    const int count = int(strEnumSize(touchSlider->handle->text[0]));
+    if(count==0)
+      return;
+    touchSlider->value = int(std::round(value*float(count-1)));


would it make more sense to have +1 on a tap, rather?

Try · 2026-10-14T22:24:49Z

+      touchStart = event.pos();
+      touchMoved = false;
+      if(current->beginTouch(touchStart-current->pos()))
+        touchId = event.mouseID;


would it be cleaner to reject mouseEvent, if mouseID!=0 ? Basically, if mouseID!=0, that means there is already some other interaction with this menu, initiated earlier by user. And if you not accepting this event (event.ignore()) - it will not cause mouseUp/mouseDrag

Try · 2026-10-14T22:25:32Z

    void setPlayer(const Npc& pl);

    void onKeyboard(KeyCodec::Action k);
+    bool beginTouch(Tempest::Point pos);


nit: onBeginTouch for consistency

Try · 2026-10-14T22:29:29Z

+  if(!choice && slider==nullptr)
+    return;
+  const auto rect = itemRect(*touchSlider);
+  const int knob = choice ? 0 : int(float(slider->w())*float(rect.h/2)/float(slider->h()));


please, don't use texture resolution to decide integer vs float property.

Try · 2026-10-14T22:33:13Z

  }

+#if defined(__MOBILE_PLATFORM__)
+void MenuRoot::mouseDragEvent(MouseEvent& event) {


nit: may you please have one definition of mouseDragEvent/mouseUpEvent, and only disable implementation code, base on platform?

Solessfir · 2026-10-14T22:48:14Z

Thanks, addressed all of them:

Screen-fit cap now uses 800x600.
Percentage choices step +1 on tap (wrapping), only sliders drag now.
Slider drag maps across the item width, no texture size involved.
Menu ignores mouseID!=0, so the touch id bookkeeping is gone.
Renamed to onBeginTouch/onDragTouch/onEndTouch.
Single mouseDragEvent/mouseUpEvent definitions, only the bodies are platform specific.

Tested on the phone.

Add touch navigation for game menus

108e37d

Solessfir mentioned this pull request Oct 13, 2026

Use Android density for UI scaling Try/Tempest#127

Merged

Make touch menus directly interactive

497ab31

Solessfir added 3 commits October 13, 2026 03:21

Update touch menu instructions

e7f7e19

Use menu Back entries for touch navigation

cdba291

Derive percentage choices from menu scripts

242afe1

Keep touch choices on script steps

0685c87

Update Tempest with merged UI scaling

81639a1

Try reviewed Oct 14, 2026

View reviewed changes

Address touch menu review

59ee1c5

Conversation

Solessfir commented Oct 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Try commented Oct 12, 2026

Uh oh!

Solessfir commented Oct 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Try commented Oct 12, 2026

Uh oh!

Solessfir commented Oct 12, 2026

Uh oh!

Solessfir commented Oct 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Solessfir commented Oct 13, 2026

Uh oh!

Solessfir commented Oct 13, 2026

Uh oh!

Try left a comment

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Try Oct 14, 2026

Choose a reason for hiding this comment

Uh oh!

Solessfir commented Oct 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Solessfir commented Oct 12, 2026 •

edited

Loading

Solessfir commented Oct 12, 2026 •

edited

Loading

Solessfir commented Oct 13, 2026 •

edited

Loading