summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2022-08-19Remove code duplication in GraphicDescriptor for DXFofftkp
GraphicFormatDetector and GraphicDescriptor have duplicate format detection code so now GraphicDescriptor uses GraphicFormatDetector functions instead to detect the format for DXF files Change-Id: I13a3c40ee35259b6b61aa323f4fa5af73290e94b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137935 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-19vcl: rename AnimationData fields to more sane namesChris Sherlock
Change-Id: I5b3ae32b562f300fb33ba8daf5ef438268725a3a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/76431 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-19tools: use std::swap() in gen.hxxChris Sherlock
Change-Id: I01df93aa825ab7a023cd4beb385b5fb41c2847ea Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137389 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-19svl: rename SfxRectangleItem::aVal to maValChris Sherlock
Change-Id: I7bc86de644749fb42eba89872828b15be47bb0dd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137380 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-19no need to allocate these separately in SwStyleManagerNoel Grandin
Change-Id: Id2ff4612506861a11b03cd0ab458400c7e278c73 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138505 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-19unique_ptr->optional in SwFontSaveNoel Grandin
Change-Id: I57acbc3f43eed6db49e475377b18c0c497acc8c0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138504 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-19unique_ptr->optional in SwViewShellImpNoel Grandin
Change-Id: I1f1862c846c2dac0a845b1e8035f447b33e487d7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138503 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-19tdf#150331: update index only when text is insertedXisco Fauli
other branches of the if( m_bIsOverwriteCursor && !m_sOverwriteText.isEmpty() ) condition do the same Regression introduced by 218acb48df79101392c2cc3a030ddb5dcee878c7 "tdf#146728 lok: IME: preset formatting in the new paragraph" CppunitTest_sw_tiledrendering passes with this patch in place. I plan to add a test in a follow-up commit Change-Id: I0f63b2c802b1280d16afeb18ad0f1735891e6a33 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138485 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Reviewed-by: Szymon Kłos <szymon.klos@collabora.com> Tested-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-08-19convert more nNode to SwPosition::GetNodeNoel Grandin
in places where we are using the result with operator< Change-Id: I2859b3e45bca141401feb557a27b4643280f8c27 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138502 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-19Some more use of TOOLS_WARN_EXCEPTIONStephan Bergmann
CppunitTest_dbaccess_tdf126268 had oddly failed once for me with > [_RUN_____] Tdf126268Test::testNumbers > Fontconfig error: Cannot load default config file: No such file: (null) > warn:legacy.osl:4121315:4121315:dbaccess/source/core/dataaccess/ModelImpl.cxx:765: ODatabaseModelImpl::getOrCreateRootStorage: no source to create the storage from! > warn:legacy.osl:4121315:4121315:dbaccess/source/core/dataaccess/ModelImpl.cxx:765: ODatabaseModelImpl::getOrCreateRootStorage: no source to create the storage from! > warn:legacy.osl:4121315:4121315:dbaccess/source/core/dataaccess/ModelImpl.cxx:765: ODatabaseModelImpl::getOrCreateRootStorage: no source to create the storage from! > warn:legacy.osl:4121315:4121315:comphelper/source/misc/logging.cxx:63: EventLogger_Impl::impl_createLogger_nothrow: caught an exception! > > > Fatal exception: Signal 6 > Stack: > workdir/LinkTarget/Executable/cppunittester(__interceptor_backtrace+0x5b)[0x5589724dcd5b] > instdir/program/libuno_sal.so.3(+0x62188d)[0x7fb0bba3888d] > instdir/program/libuno_sal.so.3(+0x6210a9)[0x7fb0bba380a9] > instdir/program/libuno_sal.so.3(+0x61eba6)[0x7fb0bba35ba6] > /lib64/libc.so.6(+0x3ea70)[0x7fb0baefea70] > /lib64/libc.so.6(+0x8ec4c)[0x7fb0baf4ec4c] > /lib64/libc.so.6(raise+0x16)[0x7fb0baefe9c6] > /lib64/libc.so.6(abort+0xcf)[0x7fb0baee87f4] > instdir/program/libEngine12.so(+0x3075ac9)[0x7fb029552ac9] > instdir/program/libEngine12.so(+0x3075c32)[0x7fb029552c32] > instdir/program/libEngine12.so(+0x318e53c)[0x7fb02966b53c] [...] and it might have been interesting to learn what that caught exception was. Change-Id: I3597ed22b4f02a8fa08f1cd36aa64d210e48356d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138509 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-08-19tdf#150443 sw: fix crash of rejecting table row deletionLászló Németh
Joining "Delete" redlines of deleted table row with overlapping tracked formatting change with same time stamp resulted bad tree parent in Manage Changes dialog window (tracked formatting change instead of tracked deletion, see its pencil icon before this fix), crashing at its rejection. Regression from commit eebe4747d2d13545004937bb0267ccfc8ab9d63f "tdf#144270 sw: manage tracked table (row) deletion/insertion". Change-Id: Ia0f0b1fdff96f4acbe14025f8016e39ce95c7f09 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138474 Tested-by: László Németh <nemeth@numbertext.org> Reviewed-by: László Németh <nemeth@numbertext.org>
2022-08-18Use combobox for category list on Elements sidebar panelMike Kaganski
And replace GtkPaned with GtkBox: the elements are not expected to have a resize handle between them. Change-Id: I9ba170458754fe28052007e8051d3814dc7d2ea0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138501 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2022-08-18tdf#150462 set mode to keep scaled glyph positions as floating pointCaolán McNamara
and match render settings to best support that Change-Id: I623f9e578ee6c3fe54a966342b3cdd16b31f219a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138450 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18tdf#150462 don't prescale dxarray before using DrawTextArrayCaolán McNamara
setup the device scaling instead and pass in the unscaled dxarray and at least give vcl the chance to retain some accuracy Change-Id: I17660eb77adf8586be6eb54e61bded3a5bcc20a9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138448 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509258 Dereference null return valueCaolán McNamara
Change-Id: Iebe32d16b3a294e60cca59c49de3da4b1f142e01 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138498 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509291 Unchecked dynamic_castCaolán McNamara
Change-Id: I1289113a9ba65a671ba79bca1eca4cf9ecbbb10f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138497 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509286 silence Out-of-bounds accessCaolán McNamara
Change-Id: I35e84803c766e0932899f55d73922236260cb5e6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138477 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18Move tools/diagnose_ex.h to comphelper/diagnose_ex.hxxStephan Bergmann
...so that its TOOLS_WARN_EXCEPTION can be used in comphelper/source/misc/logging.cxx in a follow-up commit. (And while at it, rename from diangose_ex.h to the more appropriate diagnose_ex.hxx. The comphelper module is sufficiently low-level for this immediate use case, so use that at least for now; o3tl might be even more suitable but doesn't have a Library until now. Also, for the immediate use case it would have sufficed to only break DbgGetCaughtException, exceptionToString, TOOLS_WARN_EXCEPTION, TOOLS_WARN_EXCEPTION_IF, and TOOLS_INFO_EXCEPTION out of include/tools/diagnose_ex.h into an additional new include/comphelper/diagnose_ex.hxx, but its probably easier overall to just move the complete include file as is.) Change-Id: I9f3222d4ccf1a9ac29d7eb9ba1530d53e2affaee Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138451 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-08-18Fix typoAndrea Gelmini
Change-Id: Id785575c297d7e3e8867db319ddb4bffa0496426 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138496 Tested-by: Julien Nabet <serval2412@yahoo.fr> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-08-18sw: lok: fix removing redlines annotationsHenry Castro
The redlines annotations has a logical remove and it fails to notify client side. Signed-off-by: Henry Castro <hcastro@collabora.com> Change-Id: Iba228b4d6001b8c9009332f6889b7b78bc847247 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138312 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com> Reviewed-by: Ashod Nakashian <ash@collabora.com> (cherry picked from commit ddc478a9afb89dd24cfa00ca2edef5d7d7fd4a71) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138426 Tested-by: Jenkins
2022-08-18cid#1327197 UwF: Unwritten fieldCaolán McNamara
Change-Id: Ib132101078a67803d21393c875e55f7a7bf8d225 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138471 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18windows packaging: remove unused msp creationChristian Lohmaier
Change-Id: I71903967f89d968d963f66a3e160d6dead7757f2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138011 Tested-by: Jenkins Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
2022-08-18crashreport: __cpuid/__cpuidex are not available on arm64Christian Lohmaier
…and from a quick glance similar features are not exposed to userland applications on arm64: https://docs.kernel.org/arm64/cpu-feature-registers.html "The ARM architecture defines a set of feature registers, which describe the capabilities of the CPU/system. Access to these system registers is restricted from EL0 and there is no reliable way for an application to extract this information to make better decisions at runtime.[…]" Thus the CPU information would have to be gathered from the OS, probably from the Windows registry, at least it should be possible to read the name/model that way… Change-Id: I39e3679a7c4c581d8e3e4f71842d7ea7cdc2ba67 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138380 Tested-by: Jenkins Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
2022-08-18cid#1509305 Dereference null return valueCaolán McNamara
Change-Id: I17175b9ce05b784844e49aed53d85a55e33a0f45 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138484 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509313 Unchecked dynamic_castCaolán McNamara
we can get the ScDocument another way Change-Id: Ie1953b911d34a647fbac1923bc9166f67f346a69 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138483 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1500536 silence Resource leakCaolán McNamara
Change-Id: Ibd626a90136f89af731dc2bfa684d99609d44b90 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138481 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1500387 alternative silence Resource leakCaolán McNamara
This reverts commit d40bfecddcac38006ee989d8072fb2b453cf6b9f for a different approach Change-Id: I8e058efbff214456b91346ce16e7eba7a89a8a53 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138480 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1500586 Resource leakCaolán McNamara
Change-Id: I32d837af7574f6eb4ac7db4759139d0b78f63e75 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138479 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18related tdf#150197 doc import: ANLV - always SetListFormatJustin Luth
Any time there is numbering, we expect to see an optional sListFormat set. This solves a few different cases found by: assert(false && "depricated format still exists and is unhandled. Inform Vasily or Justin"); Change-Id: I127e100a003c46c7a9dfa681014ffc11ceda24ac Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138452 Tested-by: Jenkins Reviewed-by: Justin Luth <jluth@mail.com>
2022-08-18cid#1500645 silence Resource leakCaolán McNamara
Change-Id: I1548017a39f5080dce0731169811fcb0a059c4d9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138478 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509312 Unintentional integer overflowCaolán McNamara
Change-Id: I93ca395f6a74dd6c67d4388512b5de3441b362fd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138476 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1500682 silence Untrusted loop boundCaolán McNamara
Change-Id: I44ae78fe7427d2647e3efe2688be8c124c16743c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138475 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1509254 Uninitialized scalar variableCaolán McNamara
Change-Id: I2d1f0944a069230a435890734eb18e8ee3e275e9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138470 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18Update git submodulesOlivier Hallot
* Update helpcontent2 from branch 'master' to 21c64d4e09d23bf8933f8d513a857d4df3029cfa - Update LO7.4 new features video Change-Id: Ic1290ef2d90aca04f1bf6df1aae5d19f64f5919f Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138486 Tested-by: Jenkins Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-08-18Update git submodulesOlivier Hallot
* Update helpcontent2 from branch 'master' to abecf1a85a2dded5a59c543f275df51d79f649bc - tdf#150445 Change Gear icon to Hamburger icon Change-Id: I7abe1c0d84550baad982d694922119a1af827bd0 Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138377 Tested-by: Jenkins Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-08-18tdf#149768 Do not fix Templates dialog width and height in pixelsRafael Lima
This patch removes the fixed size of the Templates dialog in pixels so it works better in HiDPI displays. This patch also makes the dialog a bit wider and non-resizable. Change-Id: I256d8d5c8b76c8e32a4f008f3a235f80bf59634d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136922 Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org> Tested-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
2022-08-18loplugin:moveitNoel Grandin
Change-Id: I34de7408553e4ca702cab9aa611c03dc60b9b6a1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138472 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18tdf#150434: chart2, XLSX import: strip long legend labelsTünde Tóth
Full text of legend labels could overflow the chart area, if the legend text was too long. If it's longer than 520 characters, strip it at the first space from the 500th character (or if there is no space, at the 500th character). This results better XLSX interoperability, too. Change-Id: I23a94f6baaf620b40e9b2819738eba5c5a921722 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137492 Reviewed-by: László Németh <nemeth@numbertext.org> Tested-by: László Németh <nemeth@numbertext.org>
2022-08-18no need to allocate these on the heapNoel Grandin
Change-Id: Ia5eec87a4b36ac2dd15c3369bd7c630277a177c0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138473 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18use more pass-by-value in SfxItemSetNoel Grandin
which avoids some copying Change-Id: Id774c8947d0c2676425a1a192e0ac28c8f8402db Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138468 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-08-18tdf#150394 sw: fix crash of "Don't hyphenate last word"László Németh
Multiline last words, e.g. long URLs resulted crashing using the new hyphenation option "Don't hyphenate last word". Follow-up to commit 8c018910ae4d8701b1ce2a95727b9baed4016da3 "tdf#149248 sw offapi xmloff: add option to not hyphenate last word". Note: .fodt format is not applicable for unit testing. Change-Id: I8633af1517f09003b40a06825ad14f3ed7f882a5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138296 Tested-by: Jenkins Reviewed-by: László Németh <nemeth@numbertext.org>
2022-08-18cid#1509397 Uninitialized scalar fieldCaolán McNamara
Change-Id: I250a900298f455ce594bf7dd99acbe984efc5ada Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138466 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18cid#1500394 Explicit null dereferencedCaolán McNamara
Change-Id: Ie97b047b793c0d9546a17f9e9f1d20cbeed62626 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138469 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18external/epoxy: Fix -Wint-conversionStephan Bergmann
...with recent Clang 16 trunk since <https://github.com/llvm/llvm-project/commit/7068aa98412ade19a34b7ed126f4669f581b2311> "Strengthen -Wint-conversion to default to an error", causing > workdir/UnpackedTarball/epoxy/src/gl_generated_dispatch.c:114547:77: error: incompatible integer to pointer conversion passing 'uintptr_t' (aka 'unsigned long') to parameter of type 'GLhandleARB' (aka 'void *') [-Wint-conversion] > GEN_THUNKS(glAttachObjectARB, (GLhandleARB containerObj, GLhandleARB obj), ((uintptr_t)containerObj, (uintptr_t)obj)) > ^~~~~~~~~~~~~~~~~~~~~~~ > workdir/UnpackedTarball/epoxy/src/dispatch_common.h:150:40: note: expanded from macro 'GEN_THUNKS' > GEN_GLOBAL_REWRITE_PTR(name, args, passthrough) \ > ^~~~~~~~~~~ > workdir/UnpackedTarball/epoxy/src/dispatch_common.h:95:14: note: expanded from macro 'GEN_GLOBAL_REWRITE_PTR' > name passthrough; \ > ^~~~~~~~~~~ etc. That only hit on macOS because of the different > #ifdef __APPLE__ > typedef void *GLhandleARB; > #else > typedef unsigned int GLhandleARB; > #endif in OpenGL's glext.h, which <https://github.com/anholt/libepoxy/commit/0cfb0a044bc43245409c020e2ca34091c96283d4> "Fix most GLhandleARB warnings on OS X with a big comment in our code" had tried to address by adding "a cast to uintptr_t to shut up the compiler" in its src/gen_dispatch.py (but without stating what compiler diagnostics exactly were supposed to be silenced by that cast that now started to cause the above issues with Clang 16 trunk). It turns out that at least my macOS build with Clang 16 trunk builds fine without that cast, so just drop it for good. (And patching epoxy's src/gen_dispatch.py means that UnpackedTarball_epoxy needs to switch from gb_UnpackedTarball_set_pre_action to gb_UnpackedTarball_set_post_action.) Change-Id: I2c87bca2cc5510d17098028d4532989f48e349a9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138407 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-08-18tdf#148671: sw_ooxmlexport16: Add unittestXisco Fauli
Change-Id: I634a1f31c93c9186be6aea35f4a020cceed6dd9c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138459 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
2022-08-18cid#1485150 Uncaught exceptionCaolán McNamara
Change-Id: I793683bfb59ff6b8751baa14a999b699cc28fdc9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138467 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-08-18tdf#39593 Ruduce copy/paste code in Converter::converterDurationLiu Hao
Change-Id: Ic1b82b244ff614c8d5ab44f32bdc728deee24165 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138465 Tested-by: Jenkins Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2022-08-18Fix typoAndrea Gelmini
Change-Id: Ie0757b332f4ddb0e9d9d6e2c97b7497766221dec Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138310 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-08-18Fix typoAndrea Gelmini
Change-Id: I6daee630eb433998711091780452dc0534398f79 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138460 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-08-18chart2: improve keys (symbols) rendering in a data tableTomaž Vajngerl
- better symbol positioning taking symbol height and cell height into account. - symbol height depends on font size (as with the legend) - take symbol aspect ratio into account - remove hardcoded values Change-Id: I3537d82538035cd31b928d4bb6fc5572914fcd13 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138463 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>