<div class="gmail_quote">On Thu, Mar 19, 2009 at 2:39 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On 19.03.09 14:20:52, David Nolden wrote:<br>
> Am Donnerstag 19 März 2009 14:00:03 schrieb Andreas Pakulat:<br>
</div><div class="im">> > Can someone with some insight into the expandingtree widget please fix that<br>
> > one up so it provides a proper sizeHint? This change is nothing but a<br>
> > workaround for a bug in expandingtree, which simply has a far too small<br>
> > sizehint.<br>
</div><div class="im">> It is nothing more than a QTreeView. And QTreeViews are not good at providing<br>
> size-hints if they have thousands of items in them(At least unfeasably<br>
> inefficient). Also we don't want the quickopen widget to be as wide as the<br>
> widest entry, since that will be _very_ wide. So we should just pick a<br>
> reasonable default.<br>
<br>
</div>Then maybe it should override sizeHint and provide a meaningful value? As<br>
it is this widget doesn't comply with Qt standards (meaning every widget<br>
should provide a reasonable sizeHint).<br>
<br>
Also why is there this huge whitespace area in the quickopen list on the<br>
left of the entries? The first column should be a lot smaller in the<br>
quickopen list, then we wouldn't even need a reasonable default size for<br>
the dialog or the tree.<br>
<div class="im"><br>
> Actually it did have a very reasonable default size, but maybe that got lost<br>
> during the conversion to KDialog(Which I wasn't a fan of, for exactly such<br>
> reasons)<br>
<br>
</div>You'd have the same problem if you'd use the form that we have now, apply<br>
it onto a QWidget and put it into a QDialog with a layout. The reason it<br>
worked with QDialog is that when you resize it inside designer a resize()<br>
call is added to the generated code. So every time someone would resize the<br>
dialog in designer and save that we'd have a different size on next start,<br>
how good is that?<br>
<br>
Oh and btw, 800x400 is _not_ a reasonable size, it spans the whole screen<br>
on small laptops or monitors. Something like 550x350 seems a lot more<br>
reasonable.<br>
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
You'll never see all the places, or read all the books, but fortunately,<br>
they're not all recommended.<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br><sarcasm>yeah, too bad i won't be able to use kdevelop on my phone</sarcasm><br><br>I tend to think more like david, 700x400 would be better.<br><br>Another way to do that would have been to do something like<br>
<br>class MyClass : public KDialog<br>{<br>   public: QSize sizeHint() const { return QSize(700,400); }<br>};<br><br>but how would have benefited us?<br>And does that method imply that this ::setInitialSize() method is just wrong? If it is there it has to be for something...<br>
<br>Thanks,<br>Aleix<br>