On Wed, 2011-09-21 at 21:13 +0400, Ivan Timofeev wrote:
> 2011/9/21 Caolán McNamara
> > Did I just break this, or were they always so broken ?
>
> No, I had tested on OOo 3.3 just now - same devilry.
Ah good :-)
> > And do you want to fix it ? :-)
>
> Yeah, I want. But since I'm newbie to sour
2011/9/21 Caolán McNamara
> Did I just break this, or were they always so broken ?
No, I had tested on OOo 3.3 just now - same devilry.
> And do you want to fix it ? :-)
Yeah, I want. But since I'm newbie to source code, I'm working slowly.
However, if this is not time-critical, I will try.
___
On Wed, 2011-09-21 at 16:56 +0400, Ivan Timofeev wrote:
> See results of my small test (in normal mode) for docs with 20 pages:
>
> Range Draw&Impress CalcWriter
>
> No pages (!) 1-201-20
> 1-51-51-5 1-5
> q1-5 1-5 (!)
2011/9/21 Caolán McNamara :
> Isn't it a good thing that the previous range isn't the one selected
> anymore when an invalid range has been entered ?
Maybe it is. But other apps have different behavior on processing invalid
range.
See results of my small test (in normal mode) for docs with 20 pa
On Wed, 2011-09-21 at 14:53 +0400, Ivan Timofeev wrote:
> Ok! Problems with incorrect pasting eliminated. But one problem still
> exists...
> If in brochure mode we input range, that is incorrect at all (e.g.
> "foo") or starts with illegal characters (e.g. "foo1-4"), previous
> range will be lost
2011/9/21 Caolán McNamara :
> So I reckon we might be better off modifying
> StringRangeEnumerator::getRangesFromString to always clear its page
> vector argument, and then its probably worth being more forgiving and
> output what valid ranges in the text can be extracted before the
> invalid chars
On Tue, 2011-09-20 at 22:10 +0400, Ivan Timofeev wrote:
> 2011/9/20 Caolán McNamara :
> > Hmm, if in normal mode I type in 3-4 I get that range previewed, change
> > it to "foobar" and it stays previewing 3-4, which isn't great either
> > really.
>
> In normal layout in Writer and in any modes in
2011/9/20 Caolán McNamara :
> Hmm, if in normal mode I type in 3-4 I get that range previewed, change
> it to "foobar" and it stays previewing 3-4, which isn't great either
> really.
In normal layout in Writer and in any modes in other apps, i.e.
brochure layout in Writer is the only dissimilar pl
On Tue, 2011-09-20 at 14:25 +0400, Ivan Timofeev wrote:
> I'm glad to see my patches in master! :-) Thanks!
>
> 2011/9/20 Caolán McNamara :
> > I don't *think* we need to move the code around, the check for empty is
> > sufficient so I shrank it down to that bit.
>
> Hmm, when in "brochure" layou
> I think this lines should (?) be moved below the newly introduced check.
*This* lines:
rPagePairs.clear();
rValidPagesSet.clear();
rValidStartFrms.clear();
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mail
I'm glad to see my patches in master! :-) Thanks!
2011/9/20 Caolán McNamara :
> I don't *think* we need to move the code around, the check for empty is
> sufficient so I shrank it down to that bit.
Hmm, when in "brochure" layout, try to input incorrect range and you
will see "No pages" on the pri
On Sat, 2011-09-17 at 03:33 -0700, Ivan Timofeev wrote:
> This patch fixes crash when you input incorrect range of pages in "Print"
> dialog, "General" tab, "Pages" textfield and layout is set to "Brochure".
> http://nabble.documentfoundation.org/file/n3344034/Fix-for-fdo-35513.patch
> Fix-for-fdo
12 matches
Mail list logo
