Age | Commit message (Collapse) | Author |
|
... for `QtInstanceMessageDialog`.
This fixes the crash reported in tdf#162351 due to a nullptr
deref in `OSQLMessageBox::impl_addDetailsButton` that was seen
with the base class implementation
`QtInstanceDialog::weld_widget_for_response` that
just returns nullptr.
Clicking the "More" button in that dialog doesn't
yet show the corresponding dialog, but that will
be implemenated separately.
Change-Id: I8cd4fac4fd57be993dba1713bb2e19860dfc334b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171551
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
... for response code.
The new helper method
`QtInstanceMessageDialog::buttonForResponseCode`
will be reused elsewhere in an upcoming commit.
Change-Id: Ie7ee065fb68f936b79ac96ba70dc85ffa18c9859
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171550
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
Add a very basic native Qt implementation for `weld::Button`.
It currently doesn't really implement many methods and
just triggers an assert when they get called.
Change-Id: I7a15266e978ca3bb030c847822a82d8d58c4c189
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171549
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
* Update helpcontent2 from branch 'master'
to 4f15160e46f503beb4c8d0f6442be2a2bc6cf036
- Don’t localize file extensions
Change-Id: I73b2200c8a2a2069b2fe799181b524674c6a6442
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/171576
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
* Update helpcontent2 from branch 'master'
to 1544cc6acf3c32f80d5cb76c90aeef04e1e5265f
- Remove unnecessary comma and add missing article
Change-Id: Ie6d2eba45357e55d3b8c564213b43661d5f4afff
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/171574
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
+ Add Help button
+ Add extended tips
+ reword labels
Change-Id: Ic1b24aedc3eb4c43fc1cf19f6bbcb8b3b72c94e5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171210
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
|
|
and
cid#1555849 COPY_INSTEAD_OF_MOVE
cid#1555936 COPY_INSTEAD_OF_MOVE
cid#1555951 COPY_INSTEAD_OF_MOVE
cid#1555955 COPY_INSTEAD_OF_MOVE
cid#1555960 COPY_INSTEAD_OF_MOVE
cid#1555964 COPY_INSTEAD_OF_MOVE
cid#1555965 COPY_INSTEAD_OF_MOVE
cid#1555975 COPY_INSTEAD_OF_MOVE
cid#1555979 COPY_INSTEAD_OF_MOVE
cid#1555987 COPY_INSTEAD_OF_MOVE
cid#1555990 COPY_INSTEAD_OF_MOVE
cid#1555991 COPY_INSTEAD_OF_MOVE
cid#1556002 COPY_INSTEAD_OF_MOVE
cid#1556008 COPY_INSTEAD_OF_MOVE
cid#1556011 COPY_INSTEAD_OF_MOVE
cid#1556015 COPY_INSTEAD_OF_MOVE
cid#1556017 COPY_INSTEAD_OF_MOVE
cid#1556023 COPY_INSTEAD_OF_MOVE
Change-Id: I8ab99d8e52a1780173a4272c59d408432c29de9f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171572
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
Change-Id: Iffa09b68794ff7942c1d2780097f0021c070ed96
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171571
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
Change-Id: Ifa89ef67ffd7f5842c60a34f6705365f3de94770
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171570
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
Number format codes allow arbitrary repeat characters. These repeat
characters are inserted while formatting numbers to pad strings to the
width of the containing cell. For example, this is used in some default
format codes to left-align currency symbols with right-aligned digits.
Previously, Calc determined the number of repeat characters to insert
based on the assumption that the pixel width of a string containing N of
the repeat character would be N times the pixel width of a string
containing 1 of them. This is not a safe assumption.
This change updates the algorithm to instead look at the average width
of a repeat character when used in context as part of a padding string.
This approach is still not entirely correct - it doesn't account for
ligatures around the padding sequence, for example - but in practice it
is a reasonable compromise between cost and accuracy.
Change-Id: I240cf38ec0376ee2d2896e7a7da6760a374b3034
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171548
Tested-by: Jenkins
Reviewed-by: Jonathan Clark <jonathan@libreoffice.org>
|
|
Change-Id: I92dd8fb7212bbf8935ed6f312ef7bd3775b5b8e3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171569
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins
|
|
This may make short lived progress bars somewhat more 'blocky' and less
smooth - but inevitably make loading faster, particularly for loaded
machines with ultra-beautiful, slow to render themed progress-bars.
Change-Id: I5a7e43690f5e7eff35c980cae4991376204eb5fb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171503
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
0xfff is a special value for a style slot on MS formats export: see
MSWordStyles::GetSlot and BuildGetSlot. The slot in nNext must be
checked the same way as nBase and nLink around it.
Change-Id: I94836bc3f624eb99655e005217707e0ea7210c8c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171547
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Ifa00a230ad88d49ccda7c7f47b3d87870d4dfc3c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171489
Reviewed-by: Armin Le Grand <Armin.Le.Grand@me.com>
Tested-by: Jenkins
|
|
Change-Id: I17189845ba273f1843a922e5b8aa521a8580dcb1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171467
Reviewed-by: Hossein <hossein@libreoffice.org>
Tested-by: Jenkins
|
|
* Update helpcontent2 from branch 'master'
to c289e0b3649626a7b88aa88c0106f2f9b2251c8a
- Place find toolbar in 02 folder.
(folder 02 has most of toolbars help pages)
Change-Id: Ib5c4dadc3c2be386ff105e60081912904f25a7ce
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/171488
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
|
|
Change-Id: Ifce16dd9ec093c515e61f65b58e0414b1a157a5e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171126
Reviewed-by: Hossein <hossein@libreoffice.org>
Tested-by: Jenkins
|
|
Change-Id: I468f3b4b6e3ccf2221ad1f5088f2b4a009966c0d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171469
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Iffc22c60572f230e3c13d6e4de89856d50244eec
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171521
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
|
|
Change-Id: I8423be9fdcd0c224d5d7993a4d982176c404035d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171520
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
which is hit quite hard during import of documents with lots of marks,
mostly because of the resorting that occurs in
ContentIdxStoreImpl::Restore
Change-Id: I1e5e3d8cfc0e65f9bf74449df8e048cd061022ec
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171522
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
|
|
Change-Id: I74a7349ece014f7bf26dd8d75723751b344f1b87
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171499
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Thorsten Behrens <thorsten.behrens@allotropia.de>
Tested-by: Jenkins
|
|
How to reproduce the synchronous dialog:
- Build in debug mode
- Click on 'Format' menu, then sub-menu 'Slide Properties'
- The sync-dialog warning shows up
This commit updates the dialog from synchronous to asynchronous processing.
How to manually test the asynchronous dialog:
- Apply patch
- Build in debug mode
- Click on 'Format' menu, then sub-menu 'Slide Properties'
- The 'Slide Properties' modal dialog box opens and is functional.
Tested manually within COOL 24.04 (online) with
- Changing aspect-ratio and background color
FuPage's lifecycle is extended by wrapping itself into rtl::Reference<FuPage>
and passing a copy to the async-action lambda-capture,
as it is being called from FuTemporary and managed as rtl::Reference<FuPoor>.
Otherwised resources are passed to the async-action simply using std::shared_ptr<>.
Signed-off-by: Sven Göthel <sven.gothel@collabora.com>
Change-Id: I692204403e8a6ee526e7b463c8bf8a41c1416489
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171393
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171508
Tested-by: Jenkins
|
|
The original DOCX bugdoc has 150 sections on 88 pages, which means that
copyHeaderFooter() in the domain mapper is a significant cost.
Setting a property on a page style involves making a copy of the
SwPageDesc, working on that copy and then copying it back. Which means
that setting UNO properties one by one vs working via
setPropertyValues() is a big difference.
- old cost: 11984 ms
- new cost: 7764 ms (65% of baseline)
Change-Id: Iea99fb3ad9d3e61494c9f012c4c7fa10cc7ab09d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171519
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
|
|
Change-Id: Ia32b0224be6cda50ece367b9bc30813ce62f5655
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171504
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: If91c4db8e1180f6408ef72031299f0b88e3c9a64
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171510
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
apparent regression from:
commit c403c86c6da4db0a6f2864ad4e13def9a3898cd4
Author: Caolán McNamara <caolanm@redhat.com>
AuthorDate: Sun Mar 17 19:22:08 2019 +0000
weld SwFieldRefPage
Change-Id: I5ae3965aa681ea1f5a005fbc8dad4d0ce36af1b3
Reviewed-on: https://gerrit.libreoffice.org/69373
Change-Id: Icb868403d31e1a6b57ae957f4eb98deafcc7f940
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171514
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
and
cid#1555430 COPY_INSTEAD_OF_MOVE
cid#1555497 COPY_INSTEAD_OF_MOVE
cid#1555542 COPY_INSTEAD_OF_MOVE
cid#1555543 COPY_INSTEAD_OF_MOVE
cid#1555564 COPY_INSTEAD_OF_MOVE
cid#1555593 COPY_INSTEAD_OF_MOVE
cid#1555602 COPY_INSTEAD_OF_MOVE
cid#1555610 COPY_INSTEAD_OF_MOVE
cid#1555667 COPY_INSTEAD_OF_MOVE
cid#1555693 COPY_INSTEAD_OF_MOVE
cid#1555719 COPY_INSTEAD_OF_MOVE
cid#1555757 COPY_INSTEAD_OF_MOVE
cid#1555795 COPY_INSTEAD_OF_MOVE
cid#1555826 COPY_INSTEAD_OF_MOVE
cid#1555847 COPY_INSTEAD_OF_MOVE
cid#1555851 COPY_INSTEAD_OF_MOVE
cid#1555863 COPY_INSTEAD_OF_MOVE
cid#1555865 COPY_INSTEAD_OF_MOVE
cid#1555867 COPY_INSTEAD_OF_MOVE
cid#1555869 COPY_INSTEAD_OF_MOVE
cid#1555879 COPY_INSTEAD_OF_MOVE
cid#1555890 COPY_INSTEAD_OF_MOVE
cid#1555893 COPY_INSTEAD_OF_MOVE
cid#1555900 COPY_INSTEAD_OF_MOVE
cid#1555922 COPY_INSTEAD_OF_MOVE
cid#1555925 COPY_INSTEAD_OF_MOVE
Change-Id: I5d362a09e80eecd282e63c745bfe26eac2631a0d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171492
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins
|
|
... of FormulaToken that's only created to extract a range from an
expression because that OOXML import part doesn't know to operate
on a ScTokenArray.
Change-Id: I4011723b682c3eae0e1c8d8e768553154c024061
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171516
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
we can declare the fieldmark methods with better type-safety now, and
with much less dynamic_cast
Change-Id: Ia515050967afdee191b96dd81b139d89be9179df
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171474
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
In commit f1121362f3dd4287577dd7952c6f00a524a81111 (tdf#156530: fix
OPreparedStatement::setString, 2024-07-03), I misused the method's
'scale' argument, as the number of decimals in the target column.
It seems that the method was modelled after JDBC's setObject taking
four arguments [1], which use is also unclear to me, but it likely
is intended to describe input data. Anyway, all our implementations
either pass the scale to next implementation, or ignore it.
The problem was when 'x' was a string like '0.1370', and 'scale' was
0. The passed 'scale' was used, instead of the target column's; and
that incorrectly truncated the number to 0.
[1] https://docs.oracle.com/javase/8/docs/api/java/sql/PreparedStatement.html#setObject-int-java.lang.Object-int-int-
Change-Id: Ib507d32706551517cc0a8a3cba351093549d7026
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171506
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Disable selecting the text view's text when receiving
keyboard focus (see `TextWindow::GetFocus`) for the
`SalInstanceTextView` implementation.
This aligns this with the GTK implementation
(`GtkInstanceTextView`) that also doesn't do that.
This e.g. prevents selecting the full license
text in the license dialog when installing a
new extension when the qt6 or gen VCL plugins
are used on Linux.
Change-Id: I2f57848f441387fe03abb6d6163416c830cebc4e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171438
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
Regression after commit af5a1b5925dbc816dced8813282d953fd5adbaaa
Drop uses of css::uno::Sequence::getConstArray in dbaccess, 2024-04-30.
Change-Id: Ifaecdeb2898f2e9089b1f893a1e9ba46352cc353
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171505
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
This patch ensure that resizing the comment sidebar will update the entire layout.
Change-Id: Ia3c805569525c558191b253d694ac1f2e99f2ace
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171039
Tested-by: Jenkins
Reviewed-by: Rafael Lima <rafael.palma.lima@gmail.com>
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
|
|
(cf. <https://curl.se/libcurl/c/CURLSHOPT_UNLOCKFUNC.html>; I noticed this with
an experimental Emscripten --enable-curl --with-webdav build that failed with a
"RuntimeError: null function or function signature mismatch" during the call to
> rc = curl_easy_setopt(m_pCurl.get(), CURLOPT_SHARE, g_Init.pShare.get());
in the CurlSession ctor)
Change-Id: Iec4bd5a263484e2e615d0b7bb08928feb7dc4658
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171487
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
|
|
Change-Id: I225dba56297d8b8022eb02ea296cda72c57cd306
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171496
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins
|
|
and
cid#1616044 Use of auto that causes a copy
Change-Id: Idb8e7abb72c5a3b2492f48bd617c0f297bb200da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171478
Tested-by: Jenkins
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|
|
after
commit 1bbcc3fdf312971ffb681b332f47369739d95dde
Author: Noel Grandin <noel.grandin@collabora.co.uk>
Date: Tue May 14 14:56:47 2024 +0200
O[U]String overloads that return view for rest parameter
Change-Id: Ie1d844f4286a14aa9d7cbb3831460c17103582c0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171477
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I38f2c4a86ac65e3e7014108153b937eb6f755371
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171495
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini <andrea.gelmini@gelma.net>
|
|
Perhaps for desktop 40 updates per second can provide a nice smooth
status experience for a full screen-width progress bar in some cases;
but it seems likely that five per second over a remote protocol
might be better.
Change-Id: Ic77b3b76d680771cb766984bdbcd9c8af4af303d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171407
Tested-by: Jenkins
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
|
|
This CUPS >= 1.7 version check was was introduced back in 2017 in
commit 7fbf98d99737ac916cc2dc374f3139e2ea81e47b
Date: Thu Jul 27 08:11:34 2017 +0530
Add Common Printing Dialog Functionality
and is obsolete by now.
According to [1], our AlmaLinux baseline has CUPS 2.2.6
by now.
Replace by a simple check whether CUPS is enabled for
the build.
(`pwg_media_t` used in that conditional code block
is from a CUPS header and e.g. not available in
the Android CI builds, see [2].)
[1] https://pkgs.org/search/?q=cups
[2] https://ci.libreoffice.org/job/gerrit_android_x86_64/40875/console
Change-Id: I4683c86c2c9e44ff41ee8093d8f045bc955cba99
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171485
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
|
|
Change-Id: I5cef95873cd6a67ba2994dcb6c2c8ed231c0b2ca
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171494
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Reportedly Adobe Acrobat Pro complains if any page is missing "/Tabs"
even though ISO 14289-1:2014 requires it only for a page with
annotations.
Change-Id: I84d750da12ca55519aa61c196d2d93aa33f1abad
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171486
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Tested-by: Jenkins
|
|
This makes a similar exclusion as above in bLoneAsCharAnchoredObj
case. I don't know exactly why should this depend on columns, but
this works (so this is basically a monkey programming here).
Change-Id: I6f15da77a16629d9146076e0a4ef50f82ebdc04c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171481
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
|
|
With tdf#160069 the format paintbrush was disabled outside of text
selection and with tdf#160069 target selection was reduced to
text only. The change of tdf#160069 is mostly undone with this
patch.
Change-Id: I8e23ef56206c6749913e2c95870537c3e378f946
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170672
Tested-by: Jenkins
Tested-by: Gabor Kelemen <gabor.kelemen.extern@allotropia.de>
Reviewed-by: Gabor Kelemen <gabor.kelemen.extern@allotropia.de>
|
|
It's a bit easier to navigate while code-reading when the header is not
full of syntax errors, due to missing headers.
Change-Id: If8417d17e6180b6a9679aade3893733fc47956c8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171490
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
|
|
- sw::FindContentFrameHint send by SwTOXType is handled by SwTOXBaseSection
- SwTOXBaseSection/SwTOXBase is still an oldschool SwClient to SwTOXType
- thus we need CallSwClientNotify() for now
- once SwTOXBase is switched over to register at the Broadcaster, we can
revert this
Change-Id: Ia471a5785f49dc85c9777ae4681616842c1addf1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171440
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Reviewed-by: Bjoern Michaelsen <bjoern.michaelsen@libreoffice.org>
|
|
Regression from 04184aa7e3aada8f4d938d20dfdb54b3a7dd3896
(tdf#157829 sw: Implemented line break underflow for bidi portions)
Change-Id: If5d550bd6e0f1099bc37532b2172436c12805104
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171432
Reviewed-by: Jonathan Clark <jonathan@libreoffice.org>
Tested-by: Jenkins
|
|
drop the iterator wrapper, no longer necessary
Change-Id: I41697c062805c3fadd0210cffbc8f7beffe01f2d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171472
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I3b07ff6e208bf33053540548176a06f28138a6a9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171479
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
|