Bug 179136 - kate's plugin autocompletion: small window with proposals has strange length
Summary: kate's plugin autocompletion: small window with proposals has strange length
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-30 14:39 UTC by Cyrill Helg
Modified: 2009-01-15 01:10 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
long window with proposals (9.45 KB, image/png)
2008-12-30 14:40 UTC, Cyrill Helg 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Cyrill Helg 2008-12-30 14:39:54 UTC 
Version:           3.1.86 (using 4.1.86 (KDE 4.1.86 (KDE 4.2 >= 20081221)), Gentoo)
Compiler:          i686-pc-linux-gnu-gcc
OS:                Linux (i686) release 2.6.28-rc9-zen1-IBM-T43

Why is this appearing list so long all the time? It should only be long as needed I think.

See screenshot.
Comment 1 Cyrill Helg 2008-12-30 14:40:23 UTC 
Created attachment 29761 [details]
long window with proposals
Comment 2 Joseph Wenninger 2008-12-31 12:46:57 UTC 
I guess it is this code in katecompletionwidget.cpp.

  if( baseHeight < 300 ) {
    baseHeight = 300; //Here we enforce a minimum desirable height
    m_expandingAddedHeight = 0;
//     kDebug( 13035 ) << "Resetting baseHeight and m_expandingAddedHeight";
  }

Svn blame says it is from zwabel, I guess it is for kdevelop's expanding trees. I'm adding him to the CC list.
Comment 3 Cyrill Helg 2008-12-31 13:26:08 UTC 
Looks easy to change :)
Comment 4 David Nolden 2009-01-02 14:25:51 UTC 
Feel free to reduce to to 80 or something like that.

Else I will probably do it within the next days(I'm sick atm).
Comment 5 David Nolden 2009-01-07 00:42:27 UTC 
I've tried it, and this is not the right place for the fix.

The completion-list currently does not adapt its height to the content.
Comment 6 David Nolden 2009-01-15 01:10:18 UTC 
Fixed in r911171