LogosLinuxInstaller icon indicating copy to clipboard operation
LogosLinuxInstaller copied to clipboard

"The Menu Bug": Re-opening tools or layout menus fails the second time

Open ctrlaltf24 opened this issue 1 year ago • 2 comments

This may just be me (my window manager is weird)

Reproduction steps:

  • Open Logos 10 v38.1.6
  • Click on the Tools icon on the toolbar, then close it
  • Try to open the tools icon again, fails after trying to open for a second

It should be noted right clicking on a word in a bible works subsequent times

Let me know if there is more information/how I might further debug this issue

ctrlaltf24 avatar Oct 06 '24 02:10 ctrlaltf24

This bug is known as the menu bug in Telegram/Matrix/FL Forums.

Workarounds include expanding the toolbar and unexpanding it, then clicking again, or double clicking the menu item.

@jg00dman has been working on this one. It is an upstream wine issue.

thw26 avatar Oct 06 '24 12:10 thw26

We've had recent confirmation through testing and discussions with FaithLife that this issue is not a Wine regression, but rather Logos changed their code between versions 9.9 and 9.10, and this is what causes the menu bug. We will have to research how to fix this in Wine to account for Logos's code.

thw26 avatar Oct 08 '24 11:10 thw26

I wonder if they're using https://learn.microsoft.com/en-us/windows/win32/api/wtsapi32/nf-wtsapi32-wtsregistersessionnotification

This log was generated setting WINEDEBUG to +seh,+relay and only contains the time during the tools menu was opened for the first time, then a second time. I've saved a copy of this log for further diagnosis later.

cat ~/.local/state/FaithLife-Community/wine_just_menu_open_twice.log | grep "fixme"
0110:fixme:wtsapi:WTSRegisterSessionNotification Stub 0000000000030158 0x00000000
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE26D8B0) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE269500) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE267F40) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE26B510) :semi-stub
0110:fixme:uiautomation:UiaReturnRawElementProvider Unsupported object id -4, ignoring.
0110:fixme:wtsapi:WTSUnRegisterSessionNotification Stub 0000000000030158
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE27A890) :semi-stub
0110:fixme:wtsapi:WTSRegisterSessionNotification Stub 0000000000040158 0x00000000
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE269DA0) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE266300) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE266E10) :semi-stub
0110:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007FFFFE266130) :semi-stub
0110:fixme:uiautomation:UiaReturnRawElementProvider Unsupported object id -4, ignoring.
0110:fixme:wtsapi:WTSUnRegisterSessionNotification Stub 0000000000040158
01e8:fixme:ntdll:RtlGetCurrentProcessorNumberEx (00007CBA646CF6E0) :semi-stub

ctrlaltf24 avatar Oct 26 '24 04:10 ctrlaltf24

A partial fix is forthcoming in 10rc5 via https://gitlab.winehq.org/wine/wine/-/commit/51b16963f6e0e8df43118deac63f640aee4698b7

This will need reported in Wine HQ.

thw26 avatar Jan 11 '25 17:01 thw26

WineHQ bug updated https://bugs.winehq.org/show_bug.cgi?id=52380

Didn't file the follow-up (the second instance showing window decorations)

ctrlaltf24 avatar Jan 11 '25 18:01 ctrlaltf24

May still be an issue, mixed results when it comes to the bug being resolved

ctrlaltf24 avatar Feb 01 '25 21:02 ctrlaltf24

Closed with Wine 10.5

thw26 avatar May 15 '25 04:05 thw26