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

[Cristian Oneț]

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.

-- 
Regards,

Cristian Oneț
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kmymoney-devel/attachments/20110106/c54f5b03/attachment.sig