Bug 122256

Summary: KDE5: no menu in data provider dialog
Product: LibreOffice Reporter: Xisco FaulĂ­ <xiscofauli>
Component: CalcAssignee: Aleksei Nikiforov <darktemplar>
Status: VERIFIED FIXED    
Severity: normal CC: m.weghorn, samuel.mehrbrodt, serval2412, vercha, xiscofauli
Priority: medium    
Version: 6.2.0.1 rc   
Hardware: All   
OS: Linux (All)   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=120908
Whiteboard: target:6.3.0 target:6.2.2
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 102495    
Attachments: proof

Description Xisco FaulĂ­ 2018-12-21 12:53:10 UTC
Steps to reproduce:
1. Open Calc
2. Data - Data provider

-> No menu on top

Reproduced in

Version: 6.2.0.1
Build ID: 0412ee99e862f384c1106d0841a950c4cfaa9df1
CPU threads: 1; OS: Linux 4.15; UI render: default; VCL: kde5; 
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded

but not in

Version: 6.2.0.1
Build ID: 0412ee99e862f384c1106d0841a950c4cfaa9df1
CPU threads: 1; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded
Comment 1 Michael Weghorn 2018-12-21 19:20:58 UTC
Confirmed with

Version: 6.3.0.0.alpha0+
Build ID: 5df6e8e399a1634c7e8dfe3558fc8fcdac3fcb09
CPU threads: 4; OS: Linux 4.18; UI render: default; VCL: kde5; 
Locale: en-GB (en_GB.UTF-8); UI-Language: en-US
Calc: threaded

The menu holding the "Start" and "Column" top-level entries (and their subitems) is missing with kde5.
Comment 2 Julien Nabet 2018-12-22 12:40:15 UTC
On pc Debian x86-64 with master sources updated today + kde5 rendering, I could reproduce this.
I noticed these logs on console:
warn:vcl:27107:27107:vcl/source/window/menu.cxx:438: Menu::InsertItem(): ItemId == 0
warn:vcl:27107:27107:vcl/source/window/menu.cxx:438: Menu::InsertItem(): ItemId == 0
warn:vcl:27107:27107:vcl/source/window/menu.cxx:438: Menu::InsertItem(): ItemId == 0
warn:vcl:27107:27107:vcl/source/window/accessibility.cxx:272: GetAccessibleChildWindow(): wrong index in border window

I got the same either with X11 or Wayland
Comment 3 Samuel Mehrbrodt (allotropia) 2019-01-17 11:34:27 UTC
Is there a reason why that functionality has to be in a menu there?
Maybe we can just convert it to a toolbar?
Comment 4 Commit Notification 2019-02-22 17:10:39 UTC
Aleksei Nikiforov committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/+/8777fc78ce7913b774d1c6ef660c7668c6c7b526%5E%21

tdf#122256 KDE5: Make menubar work for windows with parents

It will be available in 6.3.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 5 Commit Notification 2019-02-23 09:09:43 UTC
Aleksei Nikiforov committed a patch related to this issue.
It has been pushed to "libreoffice-6-2":

https://git.libreoffice.org/core/+/f184af314ec43fbac5bb22cd3ebc09ccdf865a7c%5E%21

tdf#122256 KDE5: Make menubar work for windows with parents

It will be available in 6.2.2.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 6 Vera Blagoveschenskaya 2019-02-26 06:52:08 UTC
Created attachment 149590 [details]
proof

Not reproduced for

Version: 6.3.0.0.alpha0+
Build ID: 4187b2beaa3d1294cd5c76ec0b662f3f4fadc421
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: kde5; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2019-02-25_07:04:36
Locale: ru-RU (ru_RU.UTF-8); UI-Language: en-US
Calc: threaded

(see screen proof).
Comment 7 Vera Blagoveschenskaya 2019-02-26 06:53:11 UTC
Not reproduced for

Version: 6.2.2.0.0+
Build ID: b17bffe4eb15e4cec969aa2699ce8be60e01b1c1
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: kde5; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:libreoffice-6-2, Time: 2019-02-25_17:41:32
Locale: ru-RU (ru_RU.UTF-8); UI-Language: en-US
Calc: threaded