Age | Commit message (Collapse) | Author |
|
Change-Id: I38f9816a2d91eaa5825907a66ba99d98212b91ec
Reviewed-on: https://gerrit.libreoffice.org/31958
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Id9077ea29e7028c12f40abfd4c57756c863e3cd9
|
|
Change-Id: I389d8cb733a5ee24d3119a1ebac5a5b70daff04e
Reviewed-on: https://gerrit.libreoffice.org/31900
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I3a47c0e90e93050419326303c175cdc2adef6796
Reviewed-on: https://gerrit.libreoffice.org/31925
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Also changed variable names for easier understanding of their meaning.
Change-Id: Iab558d7d1d9533f2a0c42e3d5f4acecead2e818e
Reviewed-on: https://gerrit.libreoffice.org/31807
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I5bf8adf86da9d2e9c53341cdbc94aba05557abfc
|
|
Change-Id: I44a03f557d7a7aafeb17e5d662797203f975672f
|
|
Change-Id: I4437484192016db7ce72614ae07c5a7d1bd9ec5b
|
|
This patch caused a regression that on creation of a new sheet
while there's an unfinished edit of a cell, by using "+" in bottom
left corner, that edit is committed to newly created sheet instead
of that in which it was performed.
Also, fixing original problem (tdf#42432), it forced the pending
edit on deleted page to go to an existing page, which could
overwrite existing data without being noticed (dataloss).
Reverting the patch does not reintroduce the original problem that
it intended to fix (a crash) in master.
Change-Id: I696a85ec9d08ebb3621ebdbce4d9f71eadcdb2c2
Reviewed-on: https://gerrit.libreoffice.org/31843
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Ic409bd4977fdf610294bd13b7ea83ef9a4c48667
Reviewed-on: https://gerrit.libreoffice.org/31750
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jaskaran singh <jvsg1303@gmail.com>
|
|
Change-Id: I6ee179f5a4418bd6d03694e26ff24b8e0264a795
Reviewed-on: https://gerrit.libreoffice.org/31828
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Ic62fdcaf358031df87076373fedd3b2cf71943ba
Reviewed-on: https://gerrit.libreoffice.org/31829
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: I0ca06b8d11b23d6a225026e8fd01622d4180201e
Reviewed-on: https://gerrit.libreoffice.org/31831
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Iae68359bb0e17412300db74ffdceed4341c5288e
Reviewed-on: https://gerrit.libreoffice.org/31835
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: I290710ee5242e33c4ce7b46a44068fcd54c245a8
Reviewed-on: https://gerrit.libreoffice.org/31840
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Iecb64ab1e25913180bcb53a133f9dd6777ffb17b
|
|
SvXMLAutoStylePoolP_Impl re-starts with a clean
XMLAutoStyleFamily::maReservedNameSet, so reserving the names in
ScXMLExport::ExportStyles_() is not enough and needs to be redone in
ScXMLExport::ExportAutoStyles_()
In the bug scenario that lead to cell styles with duplicated names for
different styles of which when reloading only the first was taken.
Additionally, a style must be registered after SvXMLAutoStylePoolP::AddNamed()
to prevent duplicated styles with different names.
Change-Id: If3491eb89ab1741887d9a36ecf904a216fb1368e
|
|
Change-Id: I5a721b2b55c003c34e24d83b9b5e09a88dde684b
|
|
Change-Id: I840b28f419137d2ead44dc649a14d742055fb3b9
|
|
Change-Id: I9aa8c807714f32ba5896a9308f36fcc8ab462a3d
Reviewed-on: https://gerrit.libreoffice.org/31749
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jaskaran singh <jvsg1303@gmail.com>
|
|
Change-Id: I10b1e3880e79794b7f748882606d5c4101811d75
Reviewed-on: https://gerrit.libreoffice.org/31739
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Winfried Donkers <winfrieddonkers@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Notes
(*) In SC, BULK_DATACHANGED was or'ed into the hint id. Replaced with a
dynamic_cast check.
(*) In SC, removed the hint id field from ScIndexHint, no point in
storing the hint id twice
(*) Fold the SfxStyleSheetHintId enum into the new SfxHintId enum, no
point in storing two different hint ids
(*) In some cases, multiple #define's used to map to the same SFX_HINT
value (notably the SFX_HINT_USER* values). I made all of those separate
values.
Change-Id: I990e2fb587335ebc51c9005588c6a44f768d9de5
Reviewed-on: https://gerrit.libreoffice.org/31751
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I069838ac529335c7abc02d3a9f0deb6ff545f92a
|
|
...which also removes the need for a user-declared ScHeaderFooterTextCursor copy
ctor
Change-Id: Ida4ad4b3aef0865b40535f1fbf220ce5c44f2d47
|
|
...when aIter.GetNext returned null; found by Valgrind'ing JunitTest_sc_unoapi_4
Change-Id: Ia94928f78d0512755ee6d788b0470dea34b2a9a7
|
|
It needs to be called once anyway so do it beforehand, but a second call for
the previous row is only necessary if no other condition to add a row was met.
A third call is superfluous.
Change-Id: Id104cb608893f4d09a92733cf19342e3cb280dfd
|
|
Though it was only called twice if no other condition led to the row being
added, but at least once anyway, so we can call it beforehand.
Change-Id: Iff8dfa926b840d58b92ffec296febe4c2da8e5a9
|
|
Iterate only 513 equally attributed cells instead of 1024, per row.
This could be further enhanced if we remembered in advance in which
column the last attribution is located and use empty/default for the
rest, or use spans.
Change-Id: Id6172bd6c350a92fda4f1b637e70ff4f19d59c5a
|
|
This reverts commit 5239cefff56875c7bb45c046977f1724aace0cfb.
That lead to including / looping through all formatted rows, which
easily happens if an entire column is formatted.
Also when printing it would print all formatted rows and printing a
million empty coloured rows certainly is not desired, only one sheet of
paper maybe. Which was the original intention of SC_VISATTR_STOP and as
the name GetLastVisibleAttr() suggests the function is for visible
attribution.
|
|
Which gave problems in the unit test on tinderbox build machines, apparently
because they don't have not even the resulting fallback font installed.
Specifically here digit '1' was slightly wider (123) than digit '0' (122),
which the import accounts for but the export didn't, so widths were divided by
122 instead of 123 hence resulted in 24.23 instead of 24, for example.
Change-Id: If8a04383632de674ae2a15a79608661e434aeb54
|
|
Change-Id: I447c46cc3d66c7f482314626826dd708bf1c01c4
|
|
Change-Id: Ic8ccb0a9715ec05182dacddab2c015b0de6a0fba
Reviewed-on: https://gerrit.libreoffice.org/31675
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
tdf#101363"
This reverts commit 39aa5c2c653f36393416a6aadd6e599051bbfe7b.
Turns out the cause is something completely different..
|
|
This reverts commit 8a460cfa1b1ce01c7a479ce2e7f21a92a2c50de3.
Turns out the cause is something completely different..
|
|
Change-Id: Idbe8e393b64f2a151e20c1851d7c14fa161acf97
Reviewed-on: https://gerrit.libreoffice.org/31635
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Iac074bd6f59d2fc890459b45801d0a6143c3eb9e
|
|
It's a long story, but here is a short summary:
When the function wizard dialog launches, it uses an edit engine
instance to process text input, and that edit engine instance comes
from the input window (for whatever reason). And, we are supposed
to get ScTextWnd::MakeDialogEditView() called to instantiate such
edit engine instance for the function wizard dialog. My previous
change, however, changed this and ScTextWnd::InitEditEngine() ended
up getting called to instantiate the edit engine instance. These
two functions create edit engine instances with different settings
intended for different use cases...
The crash we saw is just one manifestation of such differences.
Change-Id: I7fd84c1b1eca2351b9ecc87e325c2fd3787369eb
Reviewed-on: https://gerrit.libreoffice.org/31664
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Kohei Yoshida <libreoffice@kohei.us>
|
|
Change-Id: Iadaeb1cdf8a73cac3308e82caa2e65db07ec0fc1
|
|
Change-Id: Ibae425df118ad222db2740b51040d7a6c65b2aa6
|
|
See http://tinderbox.libreoffice.org/cgi-bin/gunzip.cgi?tree=libreoffice-5-2&brief-log=1480964401.32227
(as long as it is there)
Change-Id: If119a19e2f5dbe53298f0b30b5749997d99bf884
|
|
Change-Id: I22bc4a615295bf43110633e22b5884b511777c2b
|
|
Change-Id: I58a63a0e6f619442f21827064644ecd8ca57b8ff
|
|
Change-Id: I5c56634b1bca8e37fa73d02d2428645301b6c547
|
|
Change-Id: I50ea17c7779c7b5cacddf548f1773fd7d6c8bade
|
|
With commit 16726a1b37df8bdcae02b3c7699df814977222bd the new import algorithm
was introduced. Unfortunately it looks like that some specific column width
are improved, but for other widths it is worse.
More information is available at:
https://bugs.documentfoundation.org/show_bug.cgi?id=91475#c29
In this commit I added unit tests which prooves proper working of old
algorithm and I have switched to old algorithm.
Unit test which was done:
1. After resave .xlsx file, the column width values did not change
2. After export .ods to .xlsx displayed column width did not change
3. After save .xlsx to .ods displayed column width did not change
This commit resolve column width after exporting to .xlsx, and
correctly preserve column width for LibreOffice.
Change-Id: I26d074b203d08a4bf4e9e31e52dbfd7ca7d3beb1
Reviewed-on: https://gerrit.libreoffice.org/27943
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: Ifc44a22738cbd0ebbd81fae5caf1d9792e7746cf
|
|
Change-Id: Ie94c190f53904372aebfa15749b65c807052fcc7
Reviewed-on: https://gerrit.libreoffice.org/31589
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: I5b2c9455da5f700d0c4da59ee3d48e5aba486a55
Reviewed-on: https://gerrit.libreoffice.org/31583
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Change-Id: Ia701f2b0a3c21e19b2de95c8012cc5a37ed88359
Reviewed-on: https://gerrit.libreoffice.org/31580
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
Especially in the oox code there are many...
Change-Id: I16915cc207c274e5bcdb5d1f4f8708db5a0479a1
Reviewed-on: https://gerrit.libreoffice.org/31578
Reviewed-by: Kohei Yoshida <libreoffice@kohei.us>
Tested-by: Kohei Yoshida <libreoffice@kohei.us>
|