From ad932cdb74f2073897174835c3890fd7275331c2 Mon Sep 17 00:00:00 2001
From: Marc Durdin <marc@durdin.net>
Date: Tue, 15 Dec 2020 11:46:30 +1100
Subject: [PATCH] fix(windows): menu scroll positions need reset at popup

Fixes #4012.

If the menu previously was scrollable, then it would potentially show
items at the wrong offset because the offsets were not reset when the
menu popup was called if no scroll was needed.
---
 windows/src/engine/keyman/UfrmKeymanMenu.pas | 1 +
 1 file changed, 1 insertion(+)

diff --git a/windows/src/engine/keyman/UfrmKeymanMenu.pas b/windows/src/engine/keyman/UfrmKeymanMenu.pas
index 1a92493189..410c8a57e4 100644
--- a/windows/src/engine/keyman/UfrmKeymanMenu.pas
+++ b/windows/src/engine/keyman/UfrmKeymanMenu.pas
@@ -401,6 +401,7 @@ begin
 
   for i := 0 to Fmnu.Items.Count - 1 do
   begin
+    FItemOffsets[i] := 0;
     if Fmnu.Items[i] is TKeymanMenuItem then
     begin
       kmi := Fmnu.Items[i] as TKeymanMenuItem;