<div dir="ltr">Hi,<div><br></div><div>I'm preparing to deliver a kind of monthly packaging of git master on Windows so I finally got a look into this and can tell you the following:</div><div><br></div><div>Style has absolutely nothing to do with this bug. The problem is the fact that the CSV importer layout contains a bunch of fixed sizes (mostly vertical) which prevent the buttons from being visible if the actual height that is needed to render the data does not fit into these fixed sizes. </div>
<div><br></div><div>Once I've removed some fixed vertical sizes from the layout the importer started to look as expected. I will provide a patch for this.</div><div><br></div><div>Regards,</div><div>Cristian</div><div>
<div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-11 20:59 GMT+03:00 Allan <span dir="ltr"><<a href="mailto:agander93@gmail.com" target="_blank">agander93@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On 11/04/14 16:13, Chris wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I tried it in kubuntu and it seems fine to me.everything works as expected.<br>
<br>
On windows the problem is with the aero theme. If I change the style to<br>
classic. It works as expected.<br>
<br>
I added the following code to csvdialog.cpp<br>
<br>
   m_wizard = new QWizard;<br>
   m_wizard->setWizardStyle(<u></u>QWizard::ClassicStyle); <== added this line<br>
   ui->horizontalLayout-><u></u>insertWidget(2, m_wizard, 0);<br>
<br>
I don't know why the aero theme does not work.<br>
<br>
Chris<br>
</blockquote>
<br>
<br></div>
Well done, that man!  It definitely had the smell of a style issue, but I hadn't sussed that I could dictate it.<br>
<br>
I tried the different styles and in fact your Aero worked without trouble although no other looked right.  The MacStyle was a no-go.<br>
<br>
Glad that's sorted.  I just need to do the finishing off with the scrolling and resizing now.<span class="HOEnZb"><font color="#888888"><br>
<br>
Allan</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 11/04/2014 7:39 AM, Allan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 10/04/14 13:52, Chris wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You are spot on!<br>
<br>
It was the whitespace on windows that was causing the issue. I added<br>
--ignore-whitespace and the patch applied flawlessly<br>
<br>
Unfortunately it made very little difference in windows. I haven't tried<br>
kubuntu yet, I will do so tomorrow.<br>
<br>
One thing that I have found. if I click and drag the "title bar" just<br>
above the introwizard I can drag the introwizard into a better position<br>
and the select file and cancel buttons become accessible.<br>
<br>
My guess based on that observation is the title bar which is supposed to<br>
be hidden is not being hidden and this is pushing the introwizard down.<br>
</blockquote>
<br>
The title bar isn't supposed to be hidden.  It shouldn't be there at<br>
all. The introwizard is a QtWidget, which has a title bar, but the<br>
widget itself doesn't get loaded into the importer dialog widget.<br>
Instead, it has a horizontal layout which does not include the title<br>
bar.  The layout instead gets inserted into another layout in the<br>
importer.<br>
<br>
Also, your screen-shot shows that the title is CSV Importer, which is<br>
not the name of the widget, but the importer dialog's name. So, I can<br>
only see it being a Windows 'feature'.<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I have also noticed the "extra" title bar just below the main dialogue<br>
title bar, disappears when you change the radio button selections and<br>
the small notifier dialog pops up.<br>
<br>
This also appears to be a problem with that dialogue not being properly<br>
hidden. Once it is moved to the center and brought to the front the<br>
extra graphic then hides properly.<br>
<br>
Does that make sense. If not I can include some images to show you what<br>
I am talking about<br>
<br>
<br>
Chris<br>
</blockquote>
<br>
I think a couple of images might help me visualise it.<br>
<br>
Allan<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
On 10/04/2014 7:56 PM, Allan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 10/04/14 09:50, Chris wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Allan Thank you for that<br>
<br>
I get errors when I tried to apply the patch. I thought I might be<br>
behind so I pulled in the latest master but it results in the same<br>
errors it seems you might be patching a commit or branch I don't have.<br>
<br>
git apply --check<br>
0001-Modify-CSV-importer-<u></u>settings-and-UI-to-correct-<u></u>appea.patch<br>
error: patch failed: kmymoney/plugins/csvimport/<u></u>bankingwizardpage.ui:6<br>
error: kmymoney/plugins/csvimport/<u></u>bankingwizardpage.ui: patch does not<br>
apply<br>
error: patch failed: kmymoney/plugins/csvimport/<u></u>csvdialog.cpp:105<br>
error: kmymoney/plugins/csvimport/<u></u>csvdialog.cpp: patch does not apply<br>
error: patch failed: kmymoney/plugins/csvimport/<u></u>csvdialog.h:168<br>
error: kmymoney/plugins/csvimport/<u></u>csvdialog.h: patch does not apply<br>
error: patch failed: kmymoney/plugins/csvimport/<u></u>introwizardpage.ui:48<br>
error: kmymoney/plugins/csvimport/<u></u>introwizardpage.ui: patch does not<br>
apply<br>
error: patch failed:<br>
kmymoney/plugins/csvimport/<u></u>investmentwizardpage.ui:6<br>
error: kmymoney/plugins/csvimport/<u></u>investmentwizardpage.ui: patch does<br>
not apply<br>
<br>
I also was doing some testing last night and I found the layout errors<br>
change between builds. I have seen this sort of error before. Often it<br>
is the result of a un-initialised variable. I also realised I may have<br>
another issue. for no particular reason I have been building<br>
kmymoney as<br>
32bit app perhaps the behaviour I am seeing is an improperly<br>
initialised<br>
int or a pointer problem due to the change in word length.<br>
<br>
Just guesswork so far I haven't committed to recompiling the entire<br>
library as 64bit yet to test it<br>
<br>
Chris<br>
</blockquote>
<br>
Hi Chris<br>
<br>
That's strange.  All five files failed?!!  I can't see it being a<br>
commit problem.  Nobody else has worked on them, at least for a couple<br>
of years, so they should be in step.<br>
<br>
I wonder if you tried on Windows rather that Ubuntu?  I produced them<br>
under Ubuntu, and for all files to fail, I'm wondering if you were on<br>
Windows, and whether it's a line-ending problem, which would affect<br>
all the files, of course.<br>
<br>
I hadn't really thought about that previously, but I guess the file<br>
will need to be edited for Windows.  There are only three lines which<br>
affect the immediate performance of the plugin, those at line 149-151<br>
in csvdialog.cpp, if you wanted to see to what extent, or if, they<br>
might help or change your issue.<br>
<br>
Allan<br>
<br>
<br>
Allan<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 8/04/2014 9:48 PM, Allan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Chris<br>
<br>
Here's the patch for you to try.  For me, it looks OK on Ubuntu now,<br>
so should be OK for you, too.  Windows, we'll see if it helps, but I<br>
can't see it making any difference so far as the Title bars are<br>
concerned.<br>
<br>
Let's know how it goes.  There are some outstanding issues with<br>
scrolling and resizing.  I'm pretty close with those but didn't want<br>
to make more changes until this first lot is tried.<br>
<br>
Allan<br>
</blockquote>
<br>
<br>
</blockquote>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
</div></div></blockquote></div><br></div></div></div>