<font size="4"><font face="georgia,serif"><br></font></font><br><div class="gmail_quote">On Thu, Feb 23, 2012 at 1:06 PM, Jekyll Wu <span dir="ltr"><<a href="mailto:adaptee@gmail.com">adaptee@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Well, that dialog is a little messy in several aspects at the moment:<br>
<br>
1). It uses QTabWidget, which is deprecated according to the krazy<br>
report[1]. Although I'm not clear why QTabWidget is deprecated, the<br>
appearance of that dialog looks really outdated when compared with<br>
most other KDE applications.<br></blockquote><div>Again, it has been a while since I looked at this.  I believe w/ KTabWidget you get all the styles that make it look like the rest of KDE. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
That appearance problem is more obvious now that we have the "Config<br>
Konsole" dialog, which is similar to dialogs in other KDE<br>
applications. I guess users will have similar feeling in next major<br>
release.<br>
<br>
What shall we do ?<br></blockquote><div>I thought perhaps we might try (perhaps a mockup) to put the edit dialog combined w/ configure dialog.  But really, anything would likely be better.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
2). It costs too much code. The file EditProfileDialog.cpp contains<br>
almost 1400 lines of code. This is probably due to that it contains<br>
many customized behavior. But still it contains too much code, and the<br>
codes are just cluttered together within one big file. This is bad not<br>
only for existing developers to maintain the code, but also for random<br>
contributors to figure out where to start with.<br>
<br>
This problem also applies to other files. The best/worst example is<br>
TerminalDisplay.cpp, which contains 3000 lines of code. I have to<br>
admit that number often scares me ...<br>
<br>
What shall we do ?<br></blockquote><div>You're more than welcomed to try to separate the classes.  I rarely have time to do anything but try to keep up w/ bug reports and serious bugs.  Something this big would take serious design and time.  A remote branch would be the way to go here. </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
3). It uses the home made class Konsole::WarningBox to show warning<br>
message. That increase the maintenance burden and provide inconsistent<br>
appearance.<br>
<br></blockquote><div>That's fine.  I look at it a while for the control dialog (ctrl+s/q) and didn't like it since it changed the dimensions of the terminal (like the search bar did).  You're patch is fine.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Again, thanks for your work on Konsole</blockquote><div>   Kurt</div><div> </div></div>