Age | Commit message (Collapse) | Author |
|
Change-Id: I227e304c6868afe7a98dbd8e96487270d06ce1eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92067
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
(cherry picked from commit 95460224f4f8443547dbf5cedbea008ea2639337)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92019
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
search and replace went wrong in:
commit b94214350e6e3d0252c09f203a3ce7807a179f90
Date: Thu Jul 25 14:09:09 2019 +0100
weld SdPublishingDlg
Change-Id: I10af85633758a6e4c354f6fc957624122dd01f3c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91901
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: Ie064632667b40201e0d266c64e3068c4d1983c7d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92283
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit db677c854a37031235b0f42b4f4f48560149754f)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92289
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
28b77c89dfcafae82cf2a6d85731b643ff9290e5 changed at-char anchored fly
selection.
WollMux calls setString("") to remove one character in the first
paragraph and a fly anchored at the start of the body text is deleted
due to the new IsAtStartOfSection() check.
It would be possible to treat deletion via API differently than via UI,
as there is already a flag ExcludeFlyAtStartEnd but it would require
passing the flag through 10 functions and also to store it in
SwUndoDelete...
The main intent of the IsAtStartOfSection() check was that Ctrl+A should
delete every fly; this can be achieved by checking that everything was
selected, so that selections only inside the first/last paragraph don't
delete the flys at the edges.
Change-Id: Id62e7d99a10b42eaecea5564372d29b6c43e3f7e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91993
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 971205dc2110c1c23ff1db1fc4041e2babf6fa9f)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92169
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Idbe742cdedc497ed9e0aec72cc69842b12d94aa6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92182
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
|
|
Change-Id: I8016e4c0ad97bf13934540d14697967d93e6d07e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92164
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit 5431e8e25c6668480ebac11248f2aecbc0bda431)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92167
|
|
we get a GError of G_IO_ERROR_FAILED_HANDLED which is documented
as "Operation failed and a helper program has already interacted with the user.
Do not display any error dialog." PENDING seems to describe this state
better than a generic error case and results in no error dialog.
Change-Id: Iee382e49edaa0a734526659971e0339991e2c03e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92171
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Copying redlines to frame text was failed in tables inside
frames. Skip these redlines temporarily.
Regression from commit e8bae67b3dbcc90ace8264b6b1aefaf0ce459aba
(tdf#125894: DOCX: import tracked changes in frames).
Change-Id: I4f3ca2e95fb2e7637f8cf8dca1088a7727bcf98d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91985
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
(cherry picked from commit 1350832be533ce6627607b1aaabd2b3565e6e7b3)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92015
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
regression from
commit 0ef5c47547bec6319b853326603f3b807407fe78
sc: rowcol: tdf#50916 convert core/tool
Change-Id: Ib8207f60d34e4257fc9a854f4e4d358951e24718
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92062
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit c076b99dd21edae0364cf9319a221a1d6a4bd487)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92170
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
Saving some arrows from ODT to DOCX scattered them around
their correct position. This happened because of a function that
recalculates the position of drawing objects when they are rotated,
according to the rotation. It turns out we don't have to do this
with lines and such.
Co-Author: Balázs Regényi
Change-Id: Iea6a34d15003cacc27a8030cb73511aba39225f6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90989
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
(cherry picked from commit 08a11f8fe19560b000c62da00d7425b4f500d605)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92166
Tested-by: Jenkins
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|
|
- Update line icons
- Update Font Color icons
Change-Id: I2bd9970c62669dc97a528c61df10566e83edb39a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92140
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit b8c2e08b68722972d65f08107d74ba4a10f713df)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92024
|
|
Change-Id: Ib4702627578e06f209c9c58021971eb4e1ed39a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92082
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: I71a532667d4ef6b2db223fddc8afcf7384bfe7fd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92018
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
|
|
Change-Id: I292350c4f6cb886bb21404744a73dfd904aa9fb0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92040
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit b332452f01f19ca1065a2bef0b390ad5e92f4105)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92016
|
|
Change-Id: Ie4cb7d2c9e5fb7bca88287fcaff03d5c9ca04b8b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91899
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
- tdf#131974: Flipped Undo & Redo button in RTL UI
- tdf#131975: Wrong icons for "Undo: Data Input" (.uno:RecUndo)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91944
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit 5b56db7dcf471dc837fc05e92636579ef80a205c)
Change-Id: I7c6e667df20cbe125da020f246d264626c48528c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91896
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
|
|
- elementary: Update Chapter Numbering icons
- elementary, KJ, Sifr: Revert Good, Neutral and Bad Cell Styles icons
Change-Id: I610b17896841d84b964afbde994590633312e0f0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91959
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit 1bc3f2660d1df6aedb0560f5ff8cff2624ce9fe6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91903
|
|
Document is special and contains deletion that contains only a table, no
paragraphs before or after it. The assertion is wrong in this case.
It doesn't matter if the atParaEndNode is incremented because it's going
to be on an end node either way.
(regression from 91b2325808a75174f284c48c8b8afc118fad74e4)
Change-Id: I3fe82f0ed0b7b213df036bb8026dc209c00123de
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91873
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 6b2e925aa0dd389a613e182545de6550057374d4)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91890
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
The problem is that in SwUndoDelete::UndoImpl(), at the start there is
only one content node with a frame in the layout, and there is a special
case with m_nReplaceDummy being set so this SwTextFrame gets deleted by
the MoveNodes() call in line 969 that moves it into the newly inserted
section.
As a result of that, MakeFrames() won't do anything because it can't
find an existing frame in the same section.
In this case, MoveNodes() should leave the frame alone, as it is beyond
the end of the deletion and excluded from MakeFrames() - if it wasn't
merged before then it doesn't need action, if it was merged before (in a
different document maybe) the RecreateStartTextFrames() ought to fix it.
(regression from 723728cd358693b8f4bc9d913541aa4479f2bd48
which disabled immediate creation of frames in Copy_())
Change-Id: Iabe2fb1810b0df898b1f57b3d850c3503c79c5d0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91842
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 6c7245e789f973cf6dad03f7008ab3f9d12d350c)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91886
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Teach starmath how to deal with surrogate pairs
see https://en.wikipedia.org/wiki/Universal_Character_Set_characters#Surrogates
+
comments from:
https://bugs.documentfoundation.org/show_bug.cgi?id=129372#c6
Here's the culprit character:
𝜕 %uD835%uDF15
I didn't dig why these 2 (found in reduced example attached to the bugtracker)
didn't need this patch
𝑋 %uD835%uDC4B
𝑢 %uD835%uDC62
Change-Id: I3bf2322a9e7f1974aa8622a91812aeb11e613ace
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91941
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
(cherry picked from commit 11b57129b53e1e2d71a5f969e2417226b4e2ddd9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91900
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
<http://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf> "CIPA DC- 008-Translation-
2012: Exchangeable image file format for digital still cameras: Exif Version 2.3"
documents that the Orientation tag 0x0112 expects a count of 1 values of type SHORT
(16 bit), and details that values <= 4 bytes are stored in the Value Offset field
always using bytes starting from the left of the field.
This is a regression introduced with 42c0e433aca68c669bc0f55af404b6bae1655fba "Avoid
-fsanitize=misaligned-pointer-use". That commit had wondered why the original code
had used OSL_SWAPWORD instead of OSL_SWAPDWORD when reading and writing such
orientation values. It turns out that that original code had happened to work
correctly when processing either little or big endian data on a little endian
machine. (Though it would have worked incorrectly when processing either little or
big endian data on a big endian machine.) And with
42c0e433aca68c669bc0f55af404b6bae1655fba, the code worked when processing little
endian data on a little endian machine, but failed when processing big endian data
on a little endian machine, as is the case for tdf#131669 on e.g. x86_64.
(read32 has become unused and is thus removed.)
Change-Id: I7992629048ac44c00ee703c75164f3d094773244
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91881
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
(cherry picked from commit fd5961cb0e2ebc2f5797f76a2b1f9fd52ca4b3ab)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91889
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Change-Id: If39b5034896998a3137b913afadd05e6f4173331
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91882
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit ab751b456d334464b01ae6a6b25ebfd1f8e2fc91)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91891
|
|
Change-Id: I3516d55f08c3bdd857994db7704c2af0a20cc260
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91924
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit 6209ac8094e3578bb2dfcbf5f7e2f302bd760926)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91895
|
|
Expander added
Change-Id: Iea41f9cf335b75210de0acf5688fddd5e3dd3dbb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86478
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
(cherry picked from commit 96b4bf352b1dc43637080719c91eef61fef74bf8)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91725
|
|
See https://bugs.documentfoundation.org/show_bug.cgi?id=131594#c0
for more info
Change-Id: Ic57826eb5a440e83cea1d9bde5e9144727e3b6df
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91141
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
(cherry picked from commit 910197b8cf9b653c1b39b35b73424a36b7c1d1ae)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91512
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
return of false means we didn't change page for some reason, the idea is
that it might be blocked to go to another page, in which case sync with
what page we ended up on, but don't bother with that if the dest page
would be the same as the current page
Change-Id: I280128240601413fb6d027d001b2ecc9a4efa76f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91718
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Latin text goes from l->r, then t->b. If we rotate to the right, then we
get t->b, then r->l. Vertical text in vcl's Font means the individual
glyphs are painted in a way that looks "non-rotated" in the tbrl case.
btlr is not symmetric to this: if you rotate to the left, then Latin and
vertical text is handled the same way, i.e. there is no compensation at
a glyph level.
This means that as far as vcl is concerned, the Font's vertical flag has
to be true in the tbrl case, but no in the btlr one. Fix
SwFont::SetVertical() to do this, which means that rotating at a
character level or using the btlr text direction will result in the same
rendering for a one-liner text.
Regression from commit 89e5b431d468745da3a1eff14d48296107b9101b (sw btlr
writing mode: implement DOC filter, 2019-03-28).
(cherry picked from commit a8d26a0bb40c101394ded8061d1b58048153631b)
Change-Id: I2619c77a3b2597dbf9feab6c7042e8d8c7454197
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91820
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Change-Id: I2c0e9c70d9b25d8e7cabaf890cd6107dcdd7834d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91863
Tested-by: Jenkins
Reviewed-by: Rizal Muttaqin <riz_17_oke@yahoo.co.id>
(cherry picked from commit 3168a1d790323c51765655d9da273d51db94b707)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91724
|
|
This is a combination of 2 commits.
tdf#131190 Make custom page styles deletable again in Calc
Here is a patch to return the ability to delete custom page styles in
Calc. This was lost in commit 30c9bc76d0718f0c01d34f81845d88413645b42c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90547
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <raykowj@gmail.com>
(cherry picked from commit bb844eb299b614c1fa56e140630db070cf709a02)
tdf#131190 Make custom page styles deletable again in Calc
Return the ability to delete custom page styles in Calc. This was lost
in commit 30c9bc76d0718f0c01d34f81845d88413645b42c
ae402cc048fde7ffd3ca2696c08d5e54cc30d1be
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91672
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
Change-Id: I18936585cfbe45e0254beb882adc8709781f57e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91404
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
|
|
Change-Id: I889c3ba38776a63587a68c7827956b6747c08e38
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91715
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Change-Id: I54aa5f399f8c1a7117d0f206752a25a02bb3819a
(cherry picked from commit 990a9bbc3121a12c7300955e9b7566f6065bd434)
|
|
* Update translations from branch 'libreoffice-6-4'
to d9d8ae075592448276e014cfcab9864b570c0c5d
- update translations for 6.4.3 rc2
and force-fix errors using pocheck
Change-Id: I21921dd81e58fbd2c0f8ce1fd44cdd0ea27d7b58
|
|
Previously, the SfxPrinterChangeFlags::OPTIONS flag was
only set for the case that the printer itself was considered
unchanged ('else' case below).
However, this also needs to be done when the printer is
considered to have changed, so the flag is properly
passed in the call to 'SetPrinter' below, to make
sure that changes in document-specific options are
properly propagated.
Change-Id: Id91ebbbf40888e52149b40ac25f188af4be425f3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91626
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
(cherry picked from commit 21fc54777d5dedc33d6bfa23b9f1072a6667eebe)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91519
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
If you Select All, Cut then move the cursor around a bit then paste, then this happens:
sw/source/core/text/txtfrm.cxx:3461: void SwTextFrame::CalcAdditionalFirstLineOffset(): Assertion `pTextNode->IsNumbered(getRootFrame()) == pTextNode->IsNumbered(nullptr)' failed.
The reason is this borked MergedPara, which really only has 1 node it's listening on, but pLastNode and pParaPropsNode point to a node that is actually part of a different MergedPara now.
$6 = {
listener = {
m_rToTell = @0x6c02700,
m_vDepends = std::__debug::vector of length 1, capacity 256 = {{
<SwClient> = {
m_pRegisteredIn = 0x6af5f20
},
}}
},
extents = std::__debug::vector of length 0, capacity 0,
mergedText = "",
pParaPropsNode = 0x6d1c390,
pFirstNode = 0x6af5f20,
pLastNode = 0x6d1c390
}
The reason is that SwContentNode::DelFrames() doesn't update these members properly; at the time when it's called for the previous pLastNode, the offending node has Merge::NonFirst set already in the call of CheckParaRedlineMerge() so it sets the wrong new pLastNode.
Fix this by iterating DelFrames() loop backward.
(regression from sw_redlinehide)
Change-Id: I508fd25af385a25ba9ed78d71aa3d1f02a7ac7a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91597
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 520b0e1679d7a97aa3d91cfc95ca647339da7e84)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91524
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
seen when dragging last animation in animation pane up one position
Change-Id: I2c5163a58bb67fc7982c41546559207977e8b7c5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91714
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
Change-Id: Ib396ef9f5b019b08a181eeb5a8eb983bdb401db4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91710
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
... to the change to use the last node - not the first node - for
paragraph properties.
The problem is that if you move the cursor around at some step of reproducing the bug, this happens:
sw/source/core/crsr/crsrsh.cxx:544: bool SwCursorShell::LRMargin(bool, bool): Assertion `sw::GetParaPropsNode(*GetLayout(), GetCursor_()->GetPoint()->nNode) == pTextNd' failed.
For a fully deleted range, it looks more plausible to map index 0 to the end of the last node instead of the start of the first node now.
This means that text will be inserted at the end of the last node too.
(regression from fa5eb82b398e29ae033f7b7c8c8195dfc10cf5b0)
Change-Id: Ica891aea658b85211397320a44c9c10b48ca2659
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91598
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 58353884dc86bdb3c1464f8bbf8c3e131584b78a)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91525
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Change-Id: I565129dcd077ddbfccedb37dfbf1b9e1acf4eb95
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91532
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
(cherry picked from commit 8666dfecbaac69b9dc9a9c23a05654d847608907)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91521
|
|
In DocumentContentOperationsManager::CopyWithFlyInFly(), first
CopyNodes() also creates all layout frames, then SaveRedlEndPosForRestore
fixes the end position of all redlines that were moved by CopyNodes()
(they were moved not by changing their position but by inserting
new nodes before their end position).
Of course this means that the layout frames are created with redlines
that have only a temporary end position, and then things go wrong when
the end positions are adjusted, so add something similar to
SwUndoDelete::UndoImpl() to recreate the frames in CopyWithFlyInFly().
This hit the assert:
sw/source/core/text/redlnitr.cxx:94: std::unique_ptr<sw::MergedPara> sw::CheckParaRedlineMerge(SwTextFrame&, SwTextNode&, sw::FrameMode): Assertion `pNode != &rTextNode || &pStart->nNode.GetNode() == &rTextNode' failed.
(regression from ... sw_redlinehide)
Change-Id: I82e0f5b320cab201e762f58800f83e08f4f01048
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91596
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 166b5010b402a41b192b1659093a25acf9065fd9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91523
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
... lcl_DeleteRedlines(), if there are redlines immediately consecutive
that get deleted; merge the SwPaM ring in that case which should help.
Change-Id: I07b67413ad6ee52bc81b89facc40202d0ca75e2a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91595
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 27aa4b16bf704d0246595750daf57b57ff2577b3)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91522
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Otherwise this is the usual case of possible quadratic mdds slowness.
Change-Id: I5b33df65e127702c011517d79b9cd41a7e820ca0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91553
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 55220e7bc6576a7575c675ba425e757a3aefa243)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91517
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
Change-Id: I865d6b3dcb7f3bff037a4015aa98db2fa2578672
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91593
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
(cherry picked from commit 28ccea8b7feed20d33e2ca7d6197d947dbbad848)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91516
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Under linux, the version is now also checked for changes at startup
of libreoffice.
Change-Id: I572f718cf5afc83a70d98a6897f1d3d6877644f7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91037
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 6b6af620ff53209487a146aa3977db328c3a6300)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91405
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Wrong placeholder is returned due to iteration not stopping
when a match is found. It causes a text element position
to be incorrectly set.
Change-Id: I58be6825cec7a61f48b46418bdf28964c1afe4ce
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91023
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
Tested-by: Jenkins
Signed-off-by: Xisco Fauli <xiscofauli@libreoffice.org>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91472
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
instead of using CXML which might return either the desired writer8 odt filter
or the StarOffice_XML__Writer filter which both have CXML set as userdata
Change-Id: I3da7e7ee9a0069642c09766e8778bcaf247a4575
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91509
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
The problem is that if Word reads a w:sectPr that is inside a w:textbox
and has a w:headerReference, then Word throws a confusing error
reporting a location inside the headerN.xml file and refuses to open the
file.
It looks like Word doesn't actually support sections inside text frames,
although it doesn't complain if the section break doesn't contain a
header/footer reference.
The WW8 export appears to avoid this by checking that
TXT_MAINTEXT == m_nTextTyp and skipping sections otherwise, but the
m_nTextTyp doesn't change when exporting a text frame in DOCX case,
so let's change that.
Possibly this makes m_bFlyFrameGraphic variable redundant, not sure
about that.
Change-Id: If862b226254983bb608bbce180f4aa2f41721273
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91421
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 0dd48d1a9a716456ff1ebe67e19881ad2f56939b)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91397
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Follow-up to 0dd48d1a9a716456ff1ebe67e19881ad2f56939b - in another
document, the sectPr is written from
DocxAttributeOutput::StartParagraphProperties(), which lacks a check
that it's in the body text.
Change-Id: Ia3b56f40a7457f072735a0e09205089a0c5f4584
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91429
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
(cherry picked from commit 38be440dbe8a706052182d06bb1ae95abdd06fcc)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91399
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
|
Flips along both the y- and x-axis weren't imported,
resulting wrong direction of arrow and other shapes.
Co-Author: Balázs Regényi
Change-Id: Iac222ac2a6a6110289969c32b40828b83da0aefd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90646
Tested-by: Jenkins
Reviewed-by: Balazs Varga <balazs.varga991@gmail.com>
(cherry picked from commit cb441c4d0adf698e6af9073c6c3285a66b76871e)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91391
Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
See bt here:
https://bugs.documentfoundation.org/attachment.cgi?id=159006
cause of invalidation of pOLV here:
https://bugs.documentfoundation.org/show_bug.cgi?id=131571#c4
Change-Id: Ibf79e1fa4ebe94470afb2042cdc007bc05010b28
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91078
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit 83ae778d2e7350a15db2f8fbfde7521ab905d0aa)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91115
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
|
|
this is similar to the problem of tdf#130623 "Base: Empty Field Properties"
so take that solution and move it down the the shared component
includes: tdf#131576 overlapping windows, move bottom one down a little
Change-Id: I567c5a2519edd5921984a27405cddd6a4904fbba
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91370
Tested-by: Jenkins
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
|