Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


Hi Pierre-André,

On Sunday, 2011-12-04 22:25:51 +0100, Pierre-André Jacquod wrote:

The first one is my favorite:
0001-fdo42286-strict-use-of-GetDataArea-and-strict-extens.patch
it extends the area down. It takes into account the cells strictly
below the already selected area. It never shrinks / shortens the
selected area. This is the one that implement in my opinion the best
the behaviour of adding data below already active area.

The second one:
0001-fdo42286-extend-down-but-also-shrink-if-cells-are-em.patch
has the same logic, except it allows to shrink the area, if cells
are emptied. This the filter is allowed to extend, it could be seen
as logic that it is also allowed to shrink.

For consistency it makes sense to also shrink the area, as
re-initializing the filter area with only one cell or one row selected
would produce the same result (plus additional columns). Also, if the
area was never shrunk and later data added in that area but not part of
the actual data area (not contiguous area) that data would be
erroneously included. Please go ahead with this one.

Please check that a defined data base range did not change behavior with
your previous changes.

Regarding variable names, it may be better to stick with naming
conventions of already existing code, i.e. it should be bShrink and
bNeedExtend, makes the code easier to read. Thanks.


The last one:
0001-fdo42286-extend-down-even-if-last-row-empty-but-a-co.patch
extend down, but also if data is added to the first cell bellow. so
if you have something like (o means empty cell, x cell with data),
initial filter only on B2:D3
ooooo
oXXXo
oXXXo
ooooX
and add the last X below right, the the last line will be included
in the area and shown with "empty cells" selection. I do not like
this, since it suddenly take into account a column which was not
part of the initial filtered area.

Mainly because it doesn't fulfill the contiguous data area aspects, for
which at least E3 or D4 would also have to contain data.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD

Attachment: pgpwKZ94eIjEW.pgp
Description: PGP signature


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.