- 28 May, 2019 2 kayıt (commit)
-
-
Tor Lillqvist yazdı
Change-Id: I129bbd9a4c4ed7949fd81a06ebbcfbb6b5d01094
-
Tor Lillqvist yazdı
Based on the corresponding macOS code. Work in progress. The image support ifdeffed out still (because it uses some macOS specific APIs for which I couldn't right away find the equivalent iOS ones). I made it much simpler than the macOS code. I dropped the keeping of a local in-process clipboard completely. Firstly, as far as I see, the iOS clipboard API (UIPasteboard etc) does not even offer the possibility to separately offer some formats and actually provide the data on request. Secondly, we must be prepared anyway that the system can kill an iOS app at any stage while the user is using some other app, so we need to make sure everything that is copied goes onto the system clipboard right away anyway. I had to disable the copying of HTML to the clipboard as that lead to a mysterious assertion failure. See comment in DataFlavorMapper::openOfficeToSystemFlavor(). But RTF seems to work well, too. I assume RTF is what gets used for cross-application copy/paste (and cross-device, even, through Apple's Universal Clipboard thing, where you can copy/paste between your Macs and iOS devices on the same network). I am not sure how relevant the various application/x-openoffice-foo formats are. Change-Id: I174495e33d86fc3990996c229243c05d6cbfcda7
-
- 27 May, 2019 2 kayıt (commit)
-
-
Miklos Vajna yazdı
In preparation of forward-porting the distro/collabora/cp-6.0-only commit 418adc09 (lo: disable under-used hyperlink fields that are buggy online., 2019-05-22), which relaxes the invariant that mpLbForm never returns LISTBOX_ENTRY_NOTFOUND. A signed integer overflow is a problem in itself, but in practice the SID_HYPERLINK_SETLINK handler in sd::DrawViewShell::FuTemporary() didn't insert a field, since the insert mode was a large negative value instead of HLINK_FIELD. [ master -> cp-6.0 backport note: on this branch, this fixes a regression. ] Conflicts: cui/source/dialogs/hltpbase.cxx Change-Id: I238d9f662a74f320d05ea6a6cc97319d2d5b3d5a Reviewed-on: https://gerrit.libreoffice.org/73036Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Michael Meeks <michael.meeks@collabora.com>
-
Tor Lillqvist yazdı
Change-Id: Ib6ab095ea5c4b4a01afad5510208ad352dea5d76
-
- 25 May, 2019 2 kayıt (commit)
-
-
Justin Luth yazdı
and... related tdf#123636 writerfilter: split newline also if PAGE_BREAK ...but only with old MSWord compat flag SplitPgBreakAndParaMark. All of the other cases (COLUMN_BREAK and non-empty runs) split the paragraph, so why not here? This document shows it is needed, but only for SplitPgBreakAndParaMark documents. Note: Word 2003 doesn't display "modern" docx well in this regard. It adds extra paragraphs where it shouldn't. There are already example unit tests that ensure that extra paragraphs aren't written for SplitPgBreakAndParaMark == false. The actual bug's document is not related to the compatibility flag. That will be handled in separate commit. Reviewed-on: https://gerrit.libreoffice.org/70835 Tested-by: Jenkins Reviewed-by:
Justin Luth <justin_luth@sil.org> (cherry picked from commit 89e44da1) tdf#123636 writerfilter: handle deferred breaks on frames ...similar to handling breaks before shapes in lcl_startShape. Three different examples found to create/split a paragraph. Which one to use? (addDummy, m_bIsSplitPara, and lcl_startCharacterGroup). SplitPara is not good because the paragraph properties probably should not be copied to the dummy paragraph (like numbering for example). Slightly modified the lcl_startChar example to ensure that the dummy paragraph doesn't steal a part of the properties, but is only default properties plus page-break. This doesn't export very well, so roundtripping is very poor. Research Note: There exists a compat flag showBreaksInFrames (Display Page/Column Breaks Present in Frames) "This element specifies whether applications should honor the presence of page and/or column breaks which are present within the contents of paragraphs which have been defined as frames using the framePr element." --Currently, LO does nothing with this flag. Probably too exotic and irrelevant (word 2003 era?). No existing unit tests found that have isSet(pg_brk) frames. Reviewed-on: https://gerrit.libreoffice.org/71255 Tested-by: Jenkins Reviewed-by:
Justin Luth <justin_luth@sil.org> Reviewed-by:
Miklos Vajna <vmiklos@collabora.com> (cherry picked from commit f6f53f76) Change-Id: I29f815355401c7af8b347a3ed9d0298bc9b27b93
-
Michael Meeks yazdı
Change-Id: I1afc6278f6853d5e45a305b4f731cb6af0e00cc1
-
- 24 May, 2019 3 kayıt (commit)
-
-
Jan Holesovsky yazdı
In the desktop version, the value that is entered there always ends up applied. In the Online, there was a scenario when it did not: enter a value and switch slide by clicking the slide sorter. There is no harm to apply the value right away when the user edits, because in the desktop version, there was no way to avoid applying the value anyway. Change-Id: I519423d60f682735aeb1d471b3397ff9671c15fd Reviewed-on: https://gerrit.libreoffice.org/72923Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Michael Meeks <michael.meeks@collabora.com>
-
Jan Holesovsky yazdı
This is useful for the Online, where we want to insert the text box directly into the document so that it is visible right away. Change-Id: I09ab2efba758aac077fa37ddfd20b1486b35b3ef Reviewed-on: https://gerrit.libreoffice.org/72911Reviewed-by:
Miklos Vajna <vmiklos@collabora.com> Tested-by:
Miklos Vajna <vmiklos@collabora.com>
-
Miklos Vajna yazdı
The problem was that in case the shape text has an explicit language set, then the shape language will have no effect on shape text language; need to clear those attributes explicitly. The const_cast<> for the EditTextObject should be OK, since we change the shape's item set anyway, so we don't omit any needed notifications. Conflicts: sd/qa/unit/tiledrendering/tiledrendering.cxx Change-Id: I721e3f0aefa103a6bfa320a671da2ee1e039fa38 Reviewed-on: https://gerrit.libreoffice.org/72907Reviewed-by:
Ashod Nakashian <ashnakash@gmail.com> Tested-by:
Miklos Vajna <vmiklos@collabora.com>
-
- 23 May, 2019 7 kayıt (commit)
-
-
Miklos Vajna yazdı
Even on the desktop, in this branch only. Change-Id: Icc203f36ea95812aba6ccf7b3c4d37ad52a86c63 Reviewed-on: https://gerrit.libreoffice.org/72857Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Michael Meeks <michael.meeks@collabora.com>
-
Jan-Marek Glogowski yazdı
After the move the std::pair is invalid. That caused invalid iterators on lookups - hard to debug... Also adds an assertion to warn if size of map and list differ. Change-Id: Ib987d47963d5e1009d64a96dcdd588a0bc27cd77 Reviewed-on: https://gerrit.libreoffice.org/61451 Tested-by: Jenkins Reviewed-by:
Jan-Marek Glogowski <glogow@fbihome.de> (cherry picked from commit e7d5bad5)
-
Tomaž Vajngerl yazdı
In some cases we assume wrong size in svgio filter which we tried to "fix" outside when we render svg to bitmap. This reverts the "fix" and is now practically the same when rendering a svg graphic (might be good to combine in the future). The fix for svgio is fixed in a different commit. Change-Id: I43ee35d3220b3a750b8f72f22085f6a34b9d26b8 Reviewed-on: https://gerrit.libreoffice.org/47497Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Tomaž Vajngerl <quikee@gmail.com> (cherry picked from commit 9707a001) Reviewed-on: https://gerrit.libreoffice.org/72825Reviewed-by:
Tor Lillqvist <tml@collabora.com> Tested-by:
Tor Lillqvist <tml@collabora.com>
-
Miklos Vajna yazdı
Which allows not hardcoding true for LOK. Conflicts: sw/source/uibase/dochdl/swdtflvr.cxx sw/source/uibase/inc/swdtflvr.hxx Change-Id: I644763ba052b148fc34283e361aa02f9bba2c5ee Reviewed-on: https://gerrit.libreoffice.org/72833Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Michael Meeks <michael.meeks@collabora.com>
-
Tomaž Vajngerl yazdı
This adds the ability to define the spinbox orientation that has decrease and increase buttons on the right side and can be changed by adding "orientation="edit-decrease-increase" to the Spinbox "Entire" part in the widget definition. Change-Id: I3601a987a4abb8d998e9cd2d8973d794d3d66f9b Reviewed-on: https://gerrit.libreoffice.org/72662 Tested-by: Jenkins Reviewed-by:
Tomaž Vajngerl <quikee@gmail.com> (cherry picked from commit 3b67ad5f) Reviewed-on: https://gerrit.libreoffice.org/72823Reviewed-by:
Tor Lillqvist <tml@collabora.com> Tested-by:
Tor Lillqvist <tml@collabora.com>
-
Tomaž Vajngerl yazdı
lru_map in master has been fixed in various (unrelated) commits. This brings it to the same state as it is in master as this is needed for the widget draw images and draw command cache. Without this the cache never hits and this will impact performance. Change-Id: If02eff899aa782df68d16a8b6995a59d822f5643 Reviewed-on: https://gerrit.libreoffice.org/72822Reviewed-by:
Tor Lillqvist <tml@collabora.com> Tested-by:
Tor Lillqvist <tml@collabora.com>
-
Tomaž Vajngerl yazdı
The general size of the image should be the same when width/height attributes are present. It is very wrong to assume the size of the image is the area covered by all the primitives in the image. Change-Id: I56f241e84dee37796f9804ce2569c4eb416e83a0 Reviewed-on: https://gerrit.libreoffice.org/54191Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Tomaž Vajngerl <quikee@gmail.com> (cherry picked from commit 7e6dac4e) Reviewed-on: https://gerrit.libreoffice.org/72821Reviewed-by:
Tor Lillqvist <tml@collabora.com> Tested-by:
Tor Lillqvist <tml@collabora.com>
-
- 22 May, 2019 2 kayıt (commit)
-
-
Tor Lillqvist yazdı
This is for COLEAT's internal use. Change-Id: If1ac2a5b251129e4431d3c0bde82529d6bdc7ccc
-
Michael Meeks yazdı
Change-Id: If2069288fac14c6113754288eb9136449626393d Reviewed-on: https://gerrit.libreoffice.org/72781Reviewed-by:
Jan Holesovsky <kendy@collabora.com> Tested-by:
Jan Holesovsky <kendy@collabora.com>
-
- 21 May, 2019 1 kayıt (commit)
-
-
Tamás Zolnai yazdı
.. if the original shape fill is defined with a theme Override the alpha value with the current value get from FillTransparence API attirbute even if the color is defined with a style or a color scheme. Reviewed-on: https://gerrit.libreoffice.org/72596 Tested-by: Jenkins Reviewed-by:
Tamás Zolnai <tamas.zolnai@collabora.com> (cherry picked from commit 259d01a3) Change-Id: I09d26238a9c2b501279e6749687dc535e614bbd6 Reviewed-on: https://gerrit.libreoffice.org/72617Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
- 20 May, 2019 2 kayıt (commit)
-
-
Tor Lillqvist yazdı
Change-Id: Ib0dfb00c6a00640fe35769f21f167bf3eae8769b
-
Tor Lillqvist yazdı
We need to call SfxApplication::GetOrCreate() in lo_initialize() now. Otherwise we will crash in sfx2. No idea why it worked earlier. I think we want to avoid the "VCLXToolkit VCL main thread" ever running in the iOS app. I don't think we had it running earlier, but now for some reason it got started when creating a document from a template, and that seemed to cause a hang. Also, when creating a document from a template, lo_initialize() is called from the app's real main thread (the one that the process starts in). In that case we do want to release the SolarMutex after InitVCL() if this was the first time we called InitVCL(). Awful crack. Not really sure I fully understand all that is going on, but this change does seem to fix recent problems... All this thread and SolarMutex stuff is so incredibly fragile. Change-Id: Ib04498ad199b64f27701f89d2df7003bca316406
-
- 19 May, 2019 5 kayıt (commit)
-
-
Tor Lillqvist yazdı
Change-Id: I34cc6aad6d28c11f0bdad99087cd989c33d7b606
-
Tamás Zolnai yazdı
Write the correct theme path to the InteropGrabBag, the same path what is generated and checked in the export code. Reviewed-on: https://gerrit.libreoffice.org/72500 Tested-by: Jenkins Reviewed-by:
Tamás Zolnai <tamas.zolnai@collabora.com> (cherry picked from commit a8bba60e) Change-Id: I32617c1a11cf3bafb142f7c8839b498aaac49aa0 Reviewed-on: https://gerrit.libreoffice.org/72522Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
Szymon Kłos yazdı
Catch exceptions and hide progress dialog. b2d85f9207bc9bf7489d0a1fd9b35f062502639d Reviewed-on: https://gerrit.libreoffice.org/65815Reviewed-by:
Szymon Kłos <szymon.klos@collabora.com> Tested-by:
Szymon Kłos <szymon.klos@collabora.com> Change-Id: Ie63c8d7e001c90f40cf7504fd8248a6742e9d244 Reviewed-on: https://gerrit.libreoffice.org/72398Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
Szymon Kłos yazdı
Change-Id: Ib652174148e6997c1aabe0251a62612f28243d6e Reviewed-on: https://gerrit.libreoffice.org/67200Reviewed-by:
Szymon Kłos <szymon.klos@collabora.com> Tested-by:
Szymon Kłos <szymon.klos@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/72321Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
Tamás Zolnai yazdı
Change-Id: I326b2803bf2d98d440e14d7f3f75ca61c2c9e49a Reviewed-on: https://gerrit.libreoffice.org/72303Reviewed-by:
Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by:
Tamás Zolnai <tamas.zolnai@collabora.com> (cherry picked from commit aa446591) Reviewed-on: https://gerrit.libreoffice.org/72357Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
- 18 May, 2019 2 kayıt (commit)
-
-
Mike Kaganski yazdı
ViewDocument3 method takes OpenType parameter, which allows to decide if user can choose opening the document read-only or to edit. This backport of commit f60cc89e does not include l10n stuff and dedicated helper executable, using dialog resource strings. Change-Id: I0ad53ba64272fb84728d2221e3dc85d3eefdda68 Reviewed-on: https://gerrit.libreoffice.org/72355Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> Tested-by:
Mike Kaganski <mike.kaganski@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/72518
-
Mike Kaganski yazdı
... in preparation for further changes. Thanks to Noel Grandin for the hint! Change-Id: I2b223322d1d42099b56a74a92e3c39631d6b581c Reviewed-on: https://gerrit.libreoffice.org/72470 Tested-by: Jenkins Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/72501Tested-by:
Mike Kaganski <mike.kaganski@collabora.com>
-
- 17 May, 2019 1 kayıt (commit)
-
-
Mike Kaganski yazdı
Change-Id: Ib7ed7b3d94c04c09debe910a76cd720802f1a6f6 Reviewed-on: https://gerrit.libreoffice.org/72475Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> Tested-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit fa741203) Reviewed-on: https://gerrit.libreoffice.org/72484
-
- 16 May, 2019 2 kayıt (commit)
-
-
Mike Kaganski yazdı
Change-Id: I60f7bbb2921b242396b4620077ca30e12a0d3b4b Reviewed-on: https://gerrit.libreoffice.org/72393 Tested-by: Jenkins Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit 04b055f1) Reviewed-on: https://gerrit.libreoffice.org/72397Tested-by:
Mike Kaganski <mike.kaganski@collabora.com>
-
Tor Lillqvist yazdı
getpwuid_r() returns nullptr. Oddly enough, it does work on actual iOS, though. So use hand-crafted values that match behaviour on actual iOS. Change-Id: Idcc95d330a93495938520229e039f340876c3653
-
- 15 May, 2019 8 kayıt (commit)
-
-
Miklos Vajna yazdı
... still active" This reverts commit 31405b20. This was a cp-6.0-only commit and what the commit messages writes is great, except it seems this was already the case. What the commit does is exactly the opposite: closing the document while LOK is no longer active. (This fixes an assert on VCL shutdown.) Change-Id: I605d9808d993ffbf27db89c03cd2a520a5149c0f Reviewed-on: https://gerrit.libreoffice.org/72344Tested-by:
Aron Budea <aron.budea@collabora.com> Reviewed-by:
Jan Holesovsky <kendy@collabora.com>
-
Jan Holesovsky yazdı
Change-Id: Ibbcb29b411bf413bf110c7d734ff8dc2763b2708 Reviewed-on: https://gerrit.libreoffice.org/72350Reviewed-by:
Miklos Vajna <vmiklos@collabora.com> Tested-by:
Miklos Vajna <vmiklos@collabora.com>
-
Jan Holesovsky yazdı
Change-Id: Ie8640b7179a96cb40fb774ede097ca7b3640edd8 Reviewed-on: https://gerrit.libreoffice.org/72349Reviewed-by:
Andras Timar <andras.timar@collabora.com> Tested-by:
Andras Timar <andras.timar@collabora.com>
-
Mike Kaganski yazdı
Also simplify it, don't issue warnings for now, until we understand clearly what each its option implies. Change-Id: I0d74a42b878991ad84c5c3bba36c8978d920b9be Reviewed-on: https://gerrit.libreoffice.org/72337Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> Tested-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit efca409b) Reviewed-on: https://gerrit.libreoffice.org/72348
-
Mike Kaganski yazdı
Change-Id: Ib03bfd795967ba70333d71d9e5eeec97be90be79 Reviewed-on: https://gerrit.libreoffice.org/72334 Tested-by: Jenkins Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit a11536d5) Reviewed-on: https://gerrit.libreoffice.org/72341Tested-by:
Mike Kaganski <mike.kaganski@collabora.com>
-
Miklos Vajna yazdı
.. hori/vert lines only" This reverts commit f8b4d371. Tomaz considers this ugly and it's no longer needed since commit 93abdf39 (sw lok: assume no windows in SwLayoutFrame::PaintSwFrame(), 2019-05-14). (cherry picked from commit 35ab8387) Conflicts: basegfx/test/B2DPolygonTest.cxx drawinglayer/source/processor2d/vclpixelprocessor2d.cxx Change-Id: Ia9b29921ff3e5d82085e1abf9f39c172357a5e13 Reviewed-on: https://gerrit.libreoffice.org/72343Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Michael Meeks <michael.meeks@collabora.com>
-
Mike Kaganski yazdı
Change-Id: Idf5e77e4122ad4b5b91f25c50fdaaf25e34a76ae Reviewed-on: https://gerrit.libreoffice.org/72336 Tested-by: Jenkins Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit bb62eff4) Reviewed-on: https://gerrit.libreoffice.org/72342Tested-by:
Mike Kaganski <mike.kaganski@collabora.com>
-
Mike Kaganski yazdı
Who was that idiot who initially used wrong values??? Hmm... oh damn, that was me :-( Change-Id: I3bfde7511bcf8adfa38ae68372f21511f21efca1 Reviewed-on: https://gerrit.libreoffice.org/72308Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com> Tested-by:
Mike Kaganski <mike.kaganski@collabora.com> (cherry picked from commit 31cc28ca) Reviewed-on: https://gerrit.libreoffice.org/72332
-
- 14 May, 2019 1 kayıt (commit)
-
-
Ashod Nakashian yazdı
When we destroy the document without LOK we leak the LOK Notifier subsystem, which doesn't get destroyed, since LOK is inavtive by then. Though it's not entirely clear why only this test suffered this issue, I avoided making unwarranted changes without reason (i.e. failing tests). Change-Id: Ib098b8ff3c828cb94a55d1f8cd1f0b010240a749
-