Bug 444026 - LO Presentation mode broken after update to Plasma 5.23.0
Summary: LO Presentation mode broken after update to Plasma 5.23.0
Status: RESOLVED DUPLICATE of bug 443787
Alias: None
Product: plasmashell
Classification: Plasma
Component: generic-multiscreen (show other bugs)
Version: 5.23.0
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Aleix Pol
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-19 09:46 UTC by Axel Braun
Modified: 2021-11-03 11:23 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Axel Braun 2021-10-19 09:46:11 UTC
I want to do a presentation with LO 7.2 using a Laptop (ThinkPad X1 Extreme, Optimus Graphic with proprietary nvidia driver) with openSUSE Tumbleweed/KDE and an external monitor

‘Normally’ the presentation was on the external monitor, and the Laptop showed the presenter mode with notes pages etc.

Now I have presentation and presenter page on the external monitor - and no way to get the presenter page back to the laptop screen.

See: https://ask.libreoffice.org/t/lo-7-2-2-impress-presentation-mode/69416

Cross-checking with a second laptop running Tumbleweed and Plasma 5.22.5, that worked as expected. After upgrade to 5.23.0 it is broken as well.

This is really a shitty situation if you have to give a presentation in 3 days!

Operating System: openSUSE Tumbleweed 20211016
KDE Plasma Version: 5.23.0
KDE Frameworks Version: 5.86.0
Qt Version: 5.15.2
Kernel Version: 5.14.11-1-default (64-bit)
Graphics Platform: X11
Processors: 12 × Intel® Core™ i7-9750H CPU @ 2.60GHz
Memory: 15.4 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 1650 with Max-Q Design/PCIe/SSE2
Comment 1 David Edmundson 2021-10-19 09:49:06 UTC
As a workaround, meta + drag on the relevant windows.
Comment 2 David Edmundson 2021-10-19 09:51:46 UTC
I don't understand what Plasma is supposedly doing wrong. We don't position those windows on X11. Libreoffice does.

Please check the output of `xrandr -q` looks correct with the correct primary monitor set correctly. If so please report to libreoffice.
Comment 3 Axel Braun 2021-10-19 10:37:16 UTC
Thanks for the workaround, helped (meta key was one of the few options I did not try....)

X1E:/home/docb # xrandr -q
Screen 0: minimum 8 x 8, current 4076 x 1080, maximum 32767 x 32767
DP-0 disconnected (normal left inverted right x axis y axis)
DP-1 disconnected (normal left inverted right x axis y axis)
HDMI-0 connected 1920x1080+0+0 (normal left inverted right x axis y axis) 600mm x 340mm
   1920x1080     60.00 +  60.00    59.94    50.00* 
   1440x900      59.89  
   1280x1024     60.02  
   1280x800      59.81  
   1280x720      60.00    59.94    50.00  
   1152x864      75.00  
   1024x768      70.07    60.00  
   800x600       60.32    56.25  
   720x576       50.00  
   720x480       59.94  
   640x480       59.94    59.93  
eDP-1-1 connected primary 1920x1080+2156+0 (normal left inverted right x axis y axis) 344mm x 193mm
   1920x1080     60.00*+  59.97    59.96    59.93  
   1680x1050     59.95    59.88  
   1400x1050     59.98  
   1600x900      59.99    59.94    59.95    59.82  
   1280x1024     60.02  
   1400x900      59.96    59.88  
   1280x960      60.00  
   1440x810      60.00    59.97  
   1368x768      59.88    59.85  
   1280x800      59.99    59.97    59.81    59.91  
   1280x720      60.00    59.99    59.86    59.74  
   1024x768      60.04    60.00  
   960x720       60.00  
   928x696       60.05  
   896x672       60.01  
   1024x576      59.95    59.96    59.90    59.82  
   960x600       59.93    60.00  
   960x540       59.96    59.99    59.63    59.82  
   800x600       60.00    60.32    56.25  
   840x525       60.01    59.88  
   864x486       59.92    59.57  
   700x525       59.98  
   800x450       59.95    59.82  
   640x512       60.02  
   700x450       59.96    59.88  
   640x480       60.00    59.94  
   720x405       59.51    58.99  
   684x384       59.88    59.85  
   640x400       59.88    59.98  
   640x360       59.86    59.83    59.84    59.32  
   512x384       60.00  
   512x288       60.00    59.92  
   480x270       59.63    59.82  
   400x300       60.32    56.34  
   432x243       59.92    59.57  
   320x240       60.05  
   360x202       59.51    59.13  
   320x180       59.84    59.32  
  1280x1024 (0x1ef) 108.000MHz +HSync +VSync
        h: width  1280 start 1328 end 1440 total 1688 skew    0 clock  63.98KHz
        v: height 1024 start 1025 end 1028 total 1066           clock  60.02Hz
  1280x800 (0x1f9) 83.500MHz -HSync +VSync
        h: width  1280 start 1352 end 1480 total 1680 skew    0 clock  49.70KHz
        v: height  800 start  803 end  809 total  831           clock  59.81Hz
  1024x768 (0x200) 65.000MHz -HSync -VSync
        h: width  1024 start 1048 end 1184 total 1344 skew    0 clock  48.36KHz
        v: height  768 start  771 end  777 total  806           clock  60.00Hz
  800x600 (0x20f) 40.000MHz +HSync +VSync
        h: width   800 start  840 end  968 total 1056 skew    0 clock  37.88KHz
        v: height  600 start  601 end  605 total  628           clock  60.32Hz
  800x600 (0x210) 36.000MHz +HSync +VSync
        h: width   800 start  824 end  896 total 1024 skew    0 clock  35.16KHz
        v: height  600 start  601 end  603 total  625           clock  56.25Hz
  640x480 (0x21c) 25.175MHz -HSync -VSync
        h: width   640 start  656 end  752 total  800 skew    0 clock  31.47KHz
        v: height  480 start  490 end  492 total  525           clock  59.94Hz

Looks OK to me
LO bug report: https://bugs.documentfoundation.org/show_bug.cgi?id=145220
Comment 4 David Edmundson 2021-10-19 11:27:17 UTC
Thanks, please do reopen and comment if anything happens on that libreoffice bug report that pushes things back our way.
Comment 5 Jan-Marek Glogowski 2021-10-20 19:52:04 UTC
FYI: the xrandr output suggests the layout is - from left to right: external HDMI (width: 1920) + 236 pixel nothing (the 2156) + laptop screen (width: 1920). In the LO bug report, Axel provided additional input from a little QScreen debug program for 5.22.5 and 5.23.0. The working setup with 5.22.5 doesn't have this gap, but since it's on a different machine, it's not yet clear it's related to the update.

Still one idea is, that the LO window / screen positioning code might somehow mix up width and offset; Axel's monitor setup is uncommon. FWIW gtk and direct xlib show the same behavior for him.
Comment 6 Michael Weghorn 2021-11-03 11:14:37 UTC
(In reply to David Edmundson from comment #4)
> Thanks, please do reopen and comment if anything happens on that libreoffice
> bug report that pushes things back our way.

FWIW, it turned out this is not a LibreOffice problem, but a KWin regression, already fixed in the 'Plasma/5.23' branch by the following commit:

    commit 2958881264caf8d3bd83a34411e9586f8fcb7211
    Author: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
    Date:   Fri Oct 15 17:18:43 2021 +0300

        Restore old behavior of Workspace::clientArea(clientOpt, Toplevel)
        
        When geometry updates are blocked, the output doesn't get updated. This
        breaks Workspace::clientArea() overload that takes only the window.
        
        Previously, clientArea() would look up the output where the window is
        every time it's called, so the fact that the screen id or AbstractOutput
        is unsynchronized with the frame geometry was irrelevant.
        
        This change restores the old behavior as 5.23 is affected by the
        output() being out of sync with the frameGeometry(). Specifically, when
        kwin starts managing an X11 window, it will block geometry updates,
        setup the window, e.g. make it fullscreen, and unblock geometry updates.
        
        Since Workspace::clientArea(clientArea, Toplevel) uses the output(),
        X11Client::setFullScreen() will most likely put the X11 window at a
        wrong output if it's called inside X11Client::manage().
        
        BUG: 443787
        
        
        (cherry picked from commit 6d5fc9fd3000cf32ecb63a8252a6f50368f3604d)

     src/workspace.cpp | 2 +-
     1 file changed, 1 insertion(+), 1 deletion(-)

Marking as a duplicate of bug 443787 accordingly.

*** This bug has been marked as a duplicate of bug 443787 ***
Comment 7 Axel Braun 2021-11-03 11:23:33 UTC
Plasma 5.23.2 seems to contain the fix already!
Thanks!