[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice
Subject:    Re: [PATCH] Fix table column resizing
From:       Sven Langkamp <longamp () reallygood ! de>
Date:       2004-01-19 22:47:59
Message-ID: 200401192347.59811.longamp () reallygood ! de
[Download RAW message or body]

Am Sunday 18 January 2004 12:22 schrieb Thomas Zander:
> On Sunday 18 January 2004 00:37, Sven Langkamp wrote:
> > Am Saturday 17 January 2004 21:50 schrieb Sven Langkamp:
> > > Am Friday 16 January 2004 19:22 schrieb David Faure:
> > > > Do you think this is safe enough to be backported to the 1.3 branch?
> > >
> > > Well, I think it's safe. I have tested it with inline and non-inline
> > > tables (with exception of multi-page tables)
> >
> > I think Thomas Zander should have a look on it too. If their is an issue
> > which I haven't mentioned it would be bad if this gets into the release
> > and he is more used to the code.
>
> I always get depressed when I take a look at it; too many broken things.
I can understand you. I would like to fixe some of the table bugs which are 
annoying me, but fortunately I don't know enough about KOffice and the tables 
are rather big piece of code. :-(
> For instance; the table cells should not be able to scale vertically using
> the mouse.
> Scaling it really big (vertically) and then typing something makes it snap
> back; I believe entering more text is the correct way of making it bigger.
> Scaling it really small (vertically) and then typing something makes KWord
> hang.
I'm confused. Why was the vertical scaling implemented if the user shouldn't 
be allowed to resize rows using the mouse?

> So I guess you really did not want me to look at it afterwards, right :-)
Well, now my patch is nearly useless because it improves a function which 
shouldn't be there.
____________________________________
koffice mailing list
koffice@mail.kde.org
To unsubscribe please visit:
https://mail.kde.org/mailman/listinfo/koffice
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic