Age | Commit message (Collapse) | Author |
|
- Adds Recency to Navigate By and when selected, previous and next
scroll controls navigate the navigation manager stack backwards and
forward.
- Removes the code that makes the Navigation tool bar appear when an
entry is added to the navigation manager stack so the tool bar doesn't
appear unless the user explicilty chooses for it to.
- Changes Navigation toolbar context sensitivity to false.
Change-Id: I246dd442b4bc5191324c35f9b91a9cb2eac8e7f4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108316
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <raykowj@gmail.com>
|
|
Change-Id: Ic27bcaa1c6ca370abc6b1e44eac91c5e6eb41ca7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109167
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
- apparently only SwTextFormatColls are ever in the SwFntCache
- however lots of SwFormat members need to clear the cache
- so for now, only move it down to SwFormat
- this should make SwModify less of a kitchen sink
Change-Id: I389fc9f0a9d9b18b5914f24ae62c5d4119bc468f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109154
Tested-by: Jenkins
Reviewed-by: Bjoern Michaelsen <bjoern.michaelsen@libreoffice.org>
|
|
This is the third patch in the series.
When you start a new document, Writer compat setting
"Tolerate white lines of PDF page backgrounds
for compatibility with old documents" is off.
Now, when saving and reloading it with LoadUserSettings disabled,
it is still the program default of "off", instead of "on".
Change-Id: I06869600f0d75c9d5457372d56e39750ff5b3dae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108547
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>
|
|
- at least in Writer it is a common pattern to switch case on a Which
and then static_cast the SfxPoolItem to the assumed "right" type
- instead, it would much less errorprone to use the TypedWhichId to be
sure to cast to the right, matching type for an id
- also, this allow to assert the dynamic_cast in debug build to
highlight type confusion
- finally, there is some example use added in sw showing how it gets rid
of explicit manual lookup of the type to cast to
- likely, many of the static_cast<TypedWhichId>(SfxPoolItem*) can be
found and replaced with a compiler plugin. This is left as an exercise
to the reader.
- alternatively, this might be a nice easy EasyHack to start ....
Change-Id: I00110c0fe25626bc89d835e3c4e7ff75839756b0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109042
Tested-by: Jenkins
Reviewed-by: Bjoern Michaelsen <bjoern.michaelsen@libreoffice.org>
|
|
This is useful when encountering layout problems, to refresh
the doc layout from an extension.
Change-Id: Ie2472f061ebea203a9d876782b3bb953477de7c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109169
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
|
|
Change-Id: I1fe9311871724ff8b7b8960f5dba6e890198565c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109211
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
tdf#136686 demonstrates the confusion that arises, if one is
not familiar with the HTML convention of "strong emphasis"
* remove "Strong" label in "Styles formatting toolbar"so that
"Strong Emphasis" appears in Toolbar Customization.
* remove "Default" label in "Styles formatting toolbar"so that
"Default Character" appears in Toolbar Customization.
(reflecting change to distinguish Default Character from
Default Paragraph)
* remove "Quote" label in "Styles formatting toolbar"so that
"Quotation" appears in Toolbar Customization.
* add "Table" to "Default Table Style" (reflecting change to
differentiate Default Styles)
* Add tooltips for "Strong Emphasis" and "Quotation" CS
These changes should also result in identical label appearance
in both Menus customization and Toolbar customization.
Change-Id: I588cdcba51ab6b8ab1e8a44de577ebe7d4f4658d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108577
Reviewed-by: Seth Chaiklin <sdc.blanco@youmail.dk>
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
Tested-by: Jenkins
|
|
Change-Id: I5d0256c23c8e06bf3af1fb33edbe5c940b28ed56
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108937
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
|
|
Change-Id: I12326479afb5756729b62ff3676f995d16f64aa0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109045
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
git show -w is empty on this commit
Change-Id: Ifc139cf4557ae9c2664057d2ed87e020330d17c3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109194
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
When typing into a protected section (or other read-only area), the code
goes through SwEditWin::KeyInput(), which checks for HasReadonlySel(),
and calls into sw::DocumentContentOperationsManager::InsertString() in
the read-write case.
When typing via ExtTextInput (e.g. Online), then SwEditWin::Command()
called into sw::DocumentContentOperationsManager::InsertString() without
such a check.
The convention is to do a read-only check in the first Writer function
called by vcl, so handle this in SwEditWin::Command(), to have exactly 1
read-only popup on typing a character.
Change-Id: I7d002e7d76bffeb6f16750de735c5bbf13a7bba9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109186
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Without setting mbSelectAll properly the selection cursor
is converted to a SwShellTableCursor, which can't handle
if a selection is extended outside of the table.
Change-Id: Ibd43097ac782b9fc02ff818d3911ebcd20efbd21
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109185
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109205
Tested-by: Jenkins
|
|
See tdf#94879 for motivation.
Change-Id: I83fc138714ad170fad0465b884d6297134a0d806
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109076
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
this just changes the Get/Set methods, the constructor and internal
representation of Color is not changed.
Change-Id: Idb6e07cc08bbaa5bd55b6bd4b585e648aef507b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109074
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
This is the second patch in the series.
When you start a new document, Writer compat setting
"Expand word space on lines with manual breaks
in justified paragraphs" is on.
Now, when saving and reloading it with LoadUserSettings disabled,
it is still the program default of "on", instead of "off".
So this one is slightly more dangerous because it will prevent
setting old behaviours for MULTIPLE settings which are
not listed in the UI.
(Anything that trusts that ConsiderWrapOnObjPos to indicate a
very old document.)
The fact that we are explicitly not loading ConsiderWrapOnObjPos
from the document should not imply that this document is an old one.
At this far point away from StarOffice 8, it is probably safer
to assume the document is new, not old.
P.S. I'm pretty sure the reset of ExpandWordSpace was a complete
error during a refactor.
(At that point, it was the only item that defaulted to true.)
The only scenario I can imagine is that at one point in time,
ExpandWordSpace was not part of the SvtCompatibilityOptions,
and so this would add it. But now it certainly is one of the
options, and so this pointless pre-init is simply overwritten.
Change-Id: Ibe160099839bcf40d5659f7d5246854c7c4fb5e8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108546
Tested-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Regression from commit d379d18666aa42031359ca8eb34b0021960347ae (oox:
import WPS shape with text as shape with textbox, 2014-06-18), the
problem was that a textbox's shape + textframe are internally 2 sdr
objects, so once GraphicZOrderHelper knows the current shape should be
on top of a shape+frame pair, it should suggest a larger ZOrder.
This is necessary till there is no setter version of
SwTextBoxHelper::getOrdNum(), which would allow import filters to ignore
this complexity, but that would be a larger change.
Change-Id: Ibbb1bcd9301eb369f25f211120f62be7c59b0fd2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109111
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
While the struct is apparently modelled after a specific data format, all the
individual members are accessed by dedicated code, never via something like
memcpy.
Change-Id: I7eff8a251d9fa9f6d771cc425584b9765308d723
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109110
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I62a8410c2f21684d063001825748f8fdbd7727b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109059
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
- ... and ensure caching gets checked in SwFormat this time.
This reverts commit d988b0cdeb54d84a0347a036a277c21c12544ab9.
Change-Id: Iea887f841c47cde4a025a305eb0984392e3bb5cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109037
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Reviewed-by: Bjoern Michaelsen <bjoern.michaelsen@libreoffice.org>
|
|
Make sure to use gtk3 rendering
- Open https://bugs.documentfoundation.org/attachment.cgi?id=149492
- Click F5 to open the navigator
- Go to Images
- Try double click the third and last image at least 15 times
-> crash, here's part of bt:
Thread 1 "soffice.bin" received signal SIGSEGV, Segmentation fault.
SwFrame::IsInTab (this=0x0) at sw/source/core/inc/frame.hxx:938
938 if ( mbInfInvalid )
(gdb) bt
0 SwFrame::IsInTab() const (this=0x0) at sw/source/core/inc/frame.hxx:938
1 0x00007fd7e78caf9c in SwAccessibleDocument::getExtendedAttributes() (this=0x7e4e090) at sw/source/core/access/accdoc.cxx:596
2 0x00007fd7e78cbf9d in non-virtual thunk to SwAccessibleDocument::getExtendedAttributes() () at sw/source/core/access/accdoc.cxx:708
3 0x00007fd7f18f90ad in attribute_set_new_from_extended_attributes(com::sun::star::uno::Reference<com::sun::star::accessibility::XAccessibleExtendedAttributes> const&)
(rExtendedAttributes=uno::Reference to (SwAccessibleDocument *) 0x7e4e180) at vcl/unx/gtk3/a11y/gtk3atktextattributes.cxx:1229
4 0x00007fd7f19113f3 in wrapper_get_attributes(_AtkObject*) (atk_obj=0x7df37d0) at vcl/unx/gtk3/a11y/gtk3atkwrapper.cxx:394
I put "Related" because I don't know if it corresponds to the initial bug
but I used the attached file of the bugtracker
Change-Id: Ief9fda9f7bcf277d18490169eee2e43fb046a1bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109051
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
|
|
numbered list items:
sw/source/core/doc/DocumentRedlineManager.cxx:97: redline table corrupted: empty redline
Change-Id: Ief26248b5cad3967d867c80261c1d51f11e6233d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108940
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
Change-Id: I93c849edbc171baba65cc44422fd396372c6f694
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109058
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I0f1f05af81a25700228c229317c7409e576fa54c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108338
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
|
|
Change-Id: Ic9328cf2a4712cc49c304b1d1e5e34e301ab5312
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109041
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I4884bfb67a061b865e8cf38b2fea6de0cb1bc3d6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109057
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I3792ddadad9582a7e6f4740829c081d9571ddaff
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109049
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
When frames containing comment anchors are removed the annotation window
is hid but the text range overlay belonging to the annotation window
remains visible in the edit window. This patch sets the visibility of
text range overlay of the annotation window to the same visibility as
the annotation window.
Change-Id: I82170b2cab248a2c852a347ebdc6901bed3c83f8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108428
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <raykowj@gmail.com>
|
|
- fixes crash that occurs using Hide All
- fixes logic used for Hide/Show All
- adds comments for Toggle, Hide All and Show All popup menu items ids
Change-Id: I87076c606e3d2bc49f5fe7d07119c6c74237f3e5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109027
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I3a1179947704452e3ffec02be59d0f7bf0b75ab0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109017
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ia284ddc1a2f9e60639a2992f0b091745d9721d09
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109016
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I1aa2dc5eabe29c66584451959e414e5f4d279aad
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109015
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ida02184e2cce87f8cfcf4ad380855175ea958f45
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109014
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I74b1125fff6a63089a5eae4b32bcf30096bfbb29
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108661
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I304963f2200cda41a228290f4caf6033b5c7413c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109023
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
no point in passing an OutputDevice there
Change-Id: I6da9b29a0e6f57ad6a8ebcb6113987ee25419483
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108988
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Better path forward is to use a dockable window for the DevTools - added in
commit 5f5ee992bd8377dc984b576950d468f6407d5c63
This reverts commit c0d770d4feb919cb3306c48d576ac30709154eae.
|
|
Change-Id: Id3b8f424e24bac5164b8b3069580ffee65dd265c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107993
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
- Make SwFormat/SwContentNode explicitly resetting the page desc. before
death instead of SwFormatPageDesc trying to do that on the dying hint:
* the dying hints is send _after_ the SwFormat/SwContentNode dtor are
completed, so the RTTI magic did not work anyway
* simply resetting the attribute in a final dtor and make
SwFormatPageDesc just check to never get a dying hint (because it
should have been destructed by reset) is a lot less errorprone.
Change-Id: I231f1729729491ba7544e5ba93d81192b212e2ae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108648
Tested-by: Jenkins
Reviewed-by: Bjoern Michaelsen <bjoern.michaelsen@libreoffice.org>
|
|
Change-Id: I94875b9fb7ee18edf63ed28902da4ae77eb14bb0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108973
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
the variable value is alpha, not transparency
Change-Id: I87904afcc0fb0fecfeeb4b80b52c5dd62893669c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108972
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ice7904f2531dba98101c5f7a05d14702fa897053
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108971
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
of category axis label, if the rotation is undefined in xml,
to avoid e.g. missing rotation of them.
Regression from commit: 21620f9d2f50e66dffc45a5afb539edb8d54434c
(tdf#138194 Chart OOXML import: set text break to true)
Change-Id: I5b03614eca547a206afc0d8118e3f6c055ed5348
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108792
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
First paragraph of the document lost its formatting
from commit 056933bc55608d0ca061539ae124d7b9386cdb62.
(tdf#137802 sw: fix crash on deleting last paragraph).
This reverts commit 056933bc55608d0ca061539ae124d7b9386cdb62.
Note: last take went wrong so this is a follow up for
commit 25ca25ee25278536c56a2ac4a63af91100ce4d2e
(tdf#138739 sw: fix regression of first paragraph).
Hopefuly it will be fine.
Change-Id: Id8a19c47f172b25cd5559f5823f19b3faf38cfd2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108196
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
|
|
Change-Id: I3ef18a2601a51d56614b5da9b56e871bd33ec79e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108942
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I28edbe108989470885fa5a0fe58190c0f3657f53
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108934
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
|
|
replace single use of deprecated gtk-paste
replace single use of deprecated gtk-play
replace single use of deprecated gtk-stop
replace only two uses of deprecated gtk-clear
replace only two uses of deprecated gtk-find
replace only two uses of deprecated gtk-media-next
replace only two uses of deprecated gtk-media-previous
Change-Id: Ia749c03ad5887f7919b1d0dc2facf8ae7e5fb7a6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108820
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
...that are not interested in the updated *indexUtf16 value
Change-Id: Iaaa0fe45dcb54c4a8a347bcdb6fca6041218d009
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108941
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
I hope I understand this correctly.
In Options - Load/Save - General, there is option
"Load user-specific settings with the document" (default = true).
Turning this off seems to simply mean, don't read the settings in
Options - LibreOffice Writer - Compatibility from the document.
The assumption I guess is that this would instead use system defaults.
(Well, actually the user can "modify" these and "set as default".)
Well, that wasn't quite true in a few cases, like this one.
The general logic says "if the setting is not specified in the doc,
set to some old behaviour". And that makes sense. Of course there
is no setting in old documents - the concept didn't exist yet.
But when we explicitly exclude these user-enforceable
compat items, do we really want to force old behaviour
instead of current behaviour? And if the user has actually modified
these defaults, do we want to ignore the user's settings? I doubt it.
So here is the first patch.
When you start a new document, Writer compat setting
"Use printer metrics for document formatting" is off.
Now, when saving and reloading it with LoadUserSettings disabled,
it is still the program default of "off", instead of "on".
And if the user modifies it to always be on, it will still
turn on - so no regression there.
Change-Id: I13b236852da843ce72be16d79e2fdace1550523c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108545
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Change-Id: Iac7678cd3aaa25dcb66144f53656795f15023b9f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108821
Tested-by: Jenkins
Reviewed-by: Andreas Kainz <kainz.a@gmail.com>
|