[Kmymoney-devel] Re: Review Request: Allow the resizing of the table in the csv importer plugin

[allan]

On 06/01/11 14:28, Cristian Oneț wrote:
> On Thursday 06 January 2011 15:52:11 allan wrote:
>>>> Should I look at the tab approach, or do I give priority to your decimal
>>>> separator change?
>>>
>>> Leave the separator stuff for now. I changed it in locale so I can manage
>>> for the time being. A better organization of the upper part of the
>>> dialog should be the first priority and when you get space after
>>> reorganizing you can add a decimal separator option just like KSpread
>>> does. But first wait for me to commit my changes so we don't conflict.
>>
>> OK.
>>
>> I don't know if you have investment CSV's, but there is a very simple
>> one in Contribs.
>
> The change is committed (revision 1212313). If you plan reorganizing the UI
> your free to do so. Another thing that I recommend looking into is that right
> now there is:
> - csv*
> - invest*
> Aren't the two "modules" a bit similar and could share a big piece of code
> instead of the way it is now using duplication? The differences I've seen so
> far are not that big.
>

Yes, that's been something I've had in mind for a while.  I've also 
wondered if the two windows are necessary, or whether they too might 
become unified, once the tabs are sorted.  In fact just a few minutes 
ago, I wondered if the windows could appear in the tab area.  I'll have 
a play, and get back to you before going in too deeply.

Allan