<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>404799</bug_id>
          
          <creation_ts>2019-02-25 10:04:27 +0000</creation_ts>
          <short_desc>Under wayland with two screen the information panel context menu is not at the right position</short_desc>
          <delta_ts>2019-03-12 15:28:20 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>dolphin</product>
          <component>panels: information</component>
          <version>18.04.3</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>multiscreen, wayland-only</keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Méven Car">meven29</reporter>
          <assigned_to name="Dolphin Bug Assignee">dolphin-bugs-null</assigned_to>
          <cc>nate</cc>
          
          <cf_commitlink>https://commits.kde.org/dolphin/40896c02d996f69f7bf4e8adebe28acdbff8d350</cf_commitlink>
          <cf_versionfixedin>19.04.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1840375</commentid>
    <comment_count>0</comment_count>
    <who name="Méven Car">meven29</who>
    <bug_when>2019-02-25 10:04:27 +0000</bug_when>
    <thetext>SUMMARY
Under wayland with two screens, on the secondary screen the context menu for the information panel is not positioned correctly.

STEPS TO REPRODUCE
1. Have two screens, on the left one, open a dolphin window with the information panel
2. Right on the information panel

OBSERVED RESULT

The context menu is opened to the top left pixel of the dolphin window.

EXPECTED RESULT

The context menu is opened at the mouse position.


SOFTWARE/OS VERSIONS
Windows: 
MacOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.15.1
KDE Frameworks Version: 5.54.0
Qt Version: 5.11.1

ADDITIONAL INFORMATION</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1841666</commentid>
    <comment_count>1</comment_count>
    <who name="Méven Car">meven29</who>
    <bug_when>2019-03-04 11:03:28 +0000</bug_when>
    <thetext>I have tested this behavior under X server and it is not affected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1842030</commentid>
    <comment_count>2</comment_count>
    <who name="Méven Car">meven29</who>
    <bug_when>2019-03-05 17:00:25 +0000</bug_when>
    <thetext>Proposed patch
https://phabricator.kde.org/D19535</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1842394</commentid>
    <comment_count>3</comment_count>
    <who name="Méven Car">meven29</who>
    <bug_when>2019-03-07 09:04:02 +0000</bug_when>
    <thetext>https://bugs.kde.org/show_bug.cgi?id=384637 makes it difficult to have a screenshot unfortunately.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1842459</commentid>
    <comment_count>4</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2019-03-07 14:32:42 +0000</bug_when>
    <thetext>Yeah, However you should be able to take a full screen screenshot using a timer, and crop it manually afterwards.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1843714</commentid>
    <comment_count>5</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2019-03-12 15:28:20 +0000</bug_when>
    <thetext>Git commit 40896c02d996f69f7bf4e8adebe28acdbff8d350 by Nate Graham, on behalf of Méven Car.
Committed on 12/03/2019 at 15:25.
Pushed by ngraham into branch &apos;master&apos;.

Correctly position context menu of the information panel under wayland with a secondary screen

Summary:
According to my testing this bug occurs because Qcursor::pos() does not work as expected under wayland on a secondary screen, then it returns inaccurate data.
This could hide bugs elsewhere.
FIXED-IN: 19.04.0

Test Plan:
Under Wayland test the context menu on both screens.
Do the same under Xorg.

Reviewers: #dolphin, elvisangelaccio

Reviewed By: #dolphin, elvisangelaccio

Subscribers: elvisangelaccio, ngraham, nicolasfella, kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D19535

M  +1    -1    src/panels/information/informationpanel.cpp
M  +2    -2    src/panels/information/informationpanelcontent.cpp
M  +1    -1    src/panels/information/informationpanelcontent.h

https://commits.kde.org/dolphin/40896c02d996f69f7bf4e8adebe28acdbff8d350</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>