<?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>317068</bug_id>
          
          <creation_ts>2013-03-20 09:12:52 +0000</creation_ts>
          <short_desc>keepInArea does not work if window dimensions match the area dimensions</short_desc>
          <delta_ts>2013-03-25 07:51:53 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>kwin</product>
          <component>core</component>
          <version>4.10.1</version>
          <rep_platform>Gentoo Packages</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></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>4.10.2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alexander Mezin">mezin.alexander</reporter>
          <assigned_to name="KWin default assignee">kwin-bugs-null</assigned_to>
          
          
          <cf_commitlink>http://commits.kde.org/kde-workspace/328775331c11f288d819491b95d5af8ee9e70cf8</cf_commitlink>
          <cf_versionfixedin>4.10.2</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1352799</commentid>
    <comment_count>0</comment_count>
    <who name="Alexander Mezin">mezin.alexander</who>
    <bug_when>2013-03-20 09:12:52 +0000</bug_when>
    <thetext>If I configure KWin to automatically maximize new windows, dialogs, whose parent window isn&apos;t maximized are placed wrongly. They look and behave almost like maximized, but they are offset to the right and sometimes down. The size is correct, however, so part of the window is located off screen.

Steps to reproduce:
1. Set &quot;New window placement&quot; to &quot;Maximize&quot;
2. Open, for example, system settings
3. Unmaximize the window
4. Click on any button which opens new dialog window

Always reproducible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1352870</commentid>
    <comment_count>1</comment_count>
    <who name="Thomas Lübking">thomas.luebking</who>
    <bug_when>2013-03-20 13:16:19 +0000</bug_when>
    <thetext>Bug in Client::keepInArea()

-    if (geometry().right() &gt; area.right() &amp;&amp; width() &lt; area.width())
+    if (geometry().right() &gt; area.right() &amp;&amp; width() &lt;= area.width())
        move(area.right() - width() + 1, y());
-    if (geometry().bottom() &gt; area.bottom() &amp;&amp; height() &lt; area.height())
+    if (geometry().bottom() &gt; area.bottom() &amp;&amp; height() &lt;= area.height())
        move(x(), area.bottom() - height() + 1);</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1352997</commentid>
    <comment_count>2</comment_count>
    <who name="Alexander Mezin">mezin.alexander</who>
    <bug_when>2013-03-20 19:25:17 +0000</bug_when>
    <thetext>Seems that &quot;+ 1&quot; is wrong too</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1353001</commentid>
    <comment_count>3</comment_count>
    <who name="Alexander Mezin">mezin.alexander</who>
    <bug_when>2013-03-20 19:38:39 +0000</bug_when>
    <thetext>Oh, sorry, &quot;by historical reasons&quot; bottom = top + height - 1, and the same for &quot;right&quot;, according to qt documentation, so everything&apos;s right.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1353032</commentid>
    <comment_count>4</comment_count>
    <who name="Thomas Lübking">thomas.luebking</who>
    <bug_when>2013-03-20 21:19:16 +0000</bug_when>
    <thetext>Git commit 380bdf0c567ed98316b208652162fb55c9420145 by Thomas Lübking.
Committed on 20/03/2013 at 20:19.
Pushed by luebking into branch &apos;KDE/4.10&apos;.

catch maximized windows in ::keepInArea
FIXED-IN: 4.10.2

M  +2    -2    kwin/geometry.cpp

http://commits.kde.org/kde-workspace/380bdf0c567ed98316b208652162fb55c9420145</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1353035</commentid>
    <comment_count>5</comment_count>
    <who name="Thomas Lübking">thomas.luebking</who>
    <bug_when>2013-03-20 21:24:37 +0000</bug_when>
    <thetext>Git commit 328775331c11f288d819491b95d5af8ee9e70cf8 by Thomas Lübking.
Committed on 20/03/2013 at 20:19.
Pushed by luebking into branch &apos;master&apos;.

catch maximized windows in ::keepInArea
FIXED-IN: 4.10.2

M  +2    -2    kwin/geometry.cpp

http://commits.kde.org/kde-workspace/328775331c11f288d819491b95d5af8ee9e70cf8</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>