[Bug 262210] email composer: main text entry field adds a vertical scrollbar and grows, so the whole screen scrolls

Volker Krause vkrause at kde.org
Thu Jan 6 15:45:49 GMT 2011


https://bugs.kde.org/show_bug.cgi?id=262210


Volker Krause <vkrause at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #3 from Volker Krause <vkrause kde org>  2011-01-06 16:45:48 ---
commit d306d21f011de10a55fedb792b8f3067f920459e
branch master
Author: Volker Krause <vkrause at kde.org>
Date:   Thu Jan 6 16:43:11 2011 +0100

    Fix off by one error causing the scrollbar to show up.

    BUG: 262210

diff --git a/mobile/mail/composerautoresizer.cpp
b/mobile/mail/composerautoresizer.cpp
index 520c040..8f46cf7 100644
--- a/mobile/mail/composerautoresizer.cpp
+++ b/mobile/mail/composerautoresizer.cpp
@@ -67,8 +67,8 @@ void ComposerAutoResizer::textEditChanged()
   const QRect contentsRect = mEdit->contentsRect();

   // sets the size of the widget dynamically
-  mEdit->setMinimumHeight( qMax( mMinimumHeight, size.height() +
(frameRect.height() - contentsRect.height() - 1) ) );
-  mEdit->setMaximumHeight( qMax( mMinimumHeight, size.height() +
(frameRect.height() - contentsRect.height() - 1) ) );
+  mEdit->setMinimumHeight( qMax( mMinimumHeight, size.height() +
(frameRect.height() - contentsRect.height()) ) );
+  mEdit->setMaximumHeight( qMax( mMinimumHeight, size.height() +
(frameRect.height() - contentsRect.height()) ) );

   const QGraphicsProxyWidget *proxy = mEdit->graphicsProxyWidget();
   QGraphicsItem *proxyItem = proxy->parentItem();

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the Kdepim-bugs mailing list