Bug 95567 - Menu keyboard shortcuts are garbled ( ???? ) in TightVNC server
Summary: Menu keyboard shortcuts are garbled ( ???? ) in TightVNC server
Status: RESOLVED INSUFFICIENTDATA
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: needsDevAdvice, patch
Depends on:
Blocks:
 
Reported: 2015-11-04 13:58 UTC by vrishab
Modified: 2018-05-02 15:51 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
Garbled menu in Libreoffice (62.76 KB, image/png)
2015-11-04 13:58 UTC, vrishab
Details
Tightvnc xdpyinfo command output (1.72 KB, text/plain)
2015-11-04 13:59 UTC, vrishab
Details
TigerVNC xdpyinfo command output (17.10 KB, text/plain)
2015-11-04 14:00 UTC, vrishab
Details
Issue is present in Libreoffice 5. (134.78 KB, image/png)
2015-11-11 08:15 UTC, vrishab
Details
Temporary fix for this issue (1.60 KB, patch)
2015-12-02 10:15 UTC, vrishab
Details

Note You need to log in before you can comment on or make changes to this bug.
Description vrishab 2015-11-04 13:58:42 UTC
Created attachment 120254 [details]
Garbled menu in Libreoffice

This is on latest Debian Jessie ( 8.1 at this time ). I installed TightVNC. Then opened LibreOffice. The menu is garbled. Please refer to the attachment. Note that this problem does not happen on a standard gnome session ( meaning standard X11 server ). It also doesn't happen on another VNC server ( TigerVNC ). I've attached the extensions supported by TightVNC ( where this bug is seen ) and TigerVNC ( where the menu is good ). I'm of the opinion that this is someway related to XInputExtension ( or some VNC extension ), as TightVNC supports very minimal extensions, whereas TigerVNC supports many.

But the question is all other applications have their menu working fine in TightVNC. LibreOffice is the only app which has this menu issue.
Comment 1 vrishab 2015-11-04 13:59:57 UTC
Created attachment 120255 [details]
Tightvnc xdpyinfo command output
Comment 2 vrishab 2015-11-04 14:00:36 UTC
Created attachment 120256 [details]
TigerVNC xdpyinfo command output
Comment 3 Buovjaga 2015-11-10 12:58:12 UTC
What is the LibreOffice version you are using?

Set to NEEDINFO.
Change back to UNCONFIRMED after you have provided the information.
Comment 4 vrishab 2015-11-10 23:00:04 UTC
[user@host]$ libreoffice --version
LibreOffice 4.3.3.2 430m0(Build:2)

Version number specific to my Linux distribution ( debian ) available at: https://packages.debian.org/jessie/libreoffice
Comment 5 Buovjaga 2015-11-11 06:20:18 UTC
Does the problem also occur with LibreOffice 5? You can try it in parallel: https://wiki.documentfoundation.org/Installing_in_parallel/Linux
Comment 6 vrishab 2015-11-11 08:15:05 UTC
Created attachment 120473 [details]
Issue is present in Libreoffice 5.
Comment 7 vrishab 2015-12-02 10:15:42 UTC
Created attachment 120949 [details]
Temporary fix for this issue

Xtightvnc ( http://www.tightvnc.com/ ) seems to behave differently as it has its own way of handling keysyms. So, the XkbKeycodeToKeysym() calls for the modifier keys ( Alt / Ctrl / Shift ) are 0. Hence, the following code in vcl/unx/generic/app/saldisp.cxx returns ??? when it is given a keysym of 0.

OUString SalDisplay::GetKeyNameFromKeySym( KeySym nKeySym ) const

    if( aKeyCode != 0 && aKeyCode != NoSymbol )
    {
        if( !nKeySym )
            aRet = "???";
        else
        {
          ...
          ...
          ...

        }
    }
    return aRet;
}

I have used this patch to fix this issue temporarily for my case. The patch uses meaningful defaults for the modifier keys. Though a better patch is welcome.
Comment 8 tommy27 2017-01-26 10:00:32 UTC
why don't you submit that patch for review to the gerrit LibO website?

https://gerrit.libreoffice.org/
Comment 9 Xisco Faulí 2017-09-24 17:30:03 UTC
As tommy mentioned,
Could you please submit the attached patch to gerrit as described here:
https://wiki.documentfoundation.org/Development/gerrit/SubmitPatch?
I have set the bug's status to 'NEEDINFO'. Please change it back to 'UNCONFIRMED' once it's done
Comment 10 QA Administrators 2018-04-04 13:09:04 UTC Comment hidden (obsolete)
Comment 11 QA Administrators 2018-05-02 15:51:55 UTC
Dear Bug Submitter,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-20180502