* Fix impress slide previewControl misalignments (on desktop)
Change-Id: I009b75ba95870e3bd051b299113c90abb584c189
Reviewed-on: https://gerrit.libreoffice.org/83317
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Change-Id: Ie05b12fa01856ba61b881a27890f5a8942c5908b
Reviewed-on: https://gerrit.libreoffice.org/83295
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
malloc is not signal safe, and must not be called
from signal-safe functions. If malloc itself signals,
calling it in the signal handler can deadlock.
Luckily, we only needed malloc for getting the
backtrace strings. Now we just write directly to
stderr, which is faster, cleaner, and safer.
Change-Id: I54093f45e05f2a0fd3c5cde0cc2104ffe6d81d2a
Reviewed-on: https://gerrit.libreoffice.org/83151
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
There are a few things acceptable/safe in a
signal handler, and taking locks is not one of them.
This replaces the logic with a simple counter that
serves the purpose just as well.
If we get a double signal, we log and ignore.
Change-Id: If589c18492468c120d00c213805467bcbba05d27
Reviewed-on: https://gerrit.libreoffice.org/83150
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
"/hosting/discovery/" added in addition to "/hosting/discovery"
Change-Id: I33f7fda9981d8d4e95ca5d69d3d6b3da1e4b0eeb
Reviewed-on: https://gerrit.libreoffice.org/83169
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
I am personally not agree to use "jquery" functions in src/* source
code, but there are some "jquery" library components (i.e.
"mCustomScrollbar") that it is dependent it, however let's minimize the
use of "jquery" functions.
Change-Id: Ic64c12d966a6215acfa900cdc05e5ddb84026f79
Reviewed-on: https://gerrit.libreoffice.org/82996
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
When mCustomScrollbar fires onScroll then queue the task to run as soon
as the current synchronizing code.
Change-Id: I761937a0ad181a42b592ad953cc7b83ea73a1b21
Reviewed-on: https://gerrit.libreoffice.org/82990
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
Some mobile devices support both pointer and touch events,
so if we add event listener it will fire 2 times.
Change-Id: Ida9862e9ee24e47ccbe3963b5c4ff18a173fffe0
Reviewed-on: https://gerrit.libreoffice.org/82799
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
When the frame is null, it is referring to a container frame window that
it is always not accessible.
Change-Id: Ie96a05327928b2b72ce5577409e368061b2afa1e
Reviewed-on: https://gerrit.libreoffice.org/82798
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
The _coordinates function is called several times to compute
the same coordinate value.
Change-Id: I6e187b74709515ddde77d60649e564cb2b484fb4
Reviewed-on: https://gerrit.libreoffice.org/82797
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
Also add a config option for logging.protocol - to help catch early
protocol issues during startup.
Change-Id: I6f0cc6dcf14b2797bc6b2bd36c44750d74eb0608
Add some missing parameters when the "websocket" is connected after
loleaflet.html while "bundle.js" is downloading and ready to execute.
Change-Id: I36995a3c82ce17fa5a81923e8d6d785f0c1c7742
Reviewed-on: https://gerrit.libreoffice.org/82546
Reviewed-by: Henry Castro <hcastro@collabora.com>
Tested-by: Henry Castro <hcastro@collabora.com>
removed use of Poco::StringTokenizer from the common directory
used LOOLProtocol::tokenize and std::vecor<std::string>
regex is used in Authorization.cpp due to limitation of toeknize mathod
regular expression helps to keep the original intention of the code
Change-Id: Ic87597d8b30cb385000f983389a57dc5d2533d98
Reviewed-on: https://gerrit.libreoffice.org/82575
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
Change-Id: Iaf124c50980d9d53be412e7ffcb1bf666e64f85f
Reviewed-on: https://gerrit.libreoffice.org/82716
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Change-Id: I83de024825e9409ae494b5d4b24afa23c54ad604
Reviewed-on: https://gerrit.libreoffice.org/82713
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
I think better to just fire a new 'focussearch' event and handle all the
Ctrl+<something> at one place.
When at this, fix also a regression that Ctrl+f in the search bar
triggered the browser's search.
Change-Id: I759be9ed3934e14b1e05a37a65ca1ce54d99eb0b
Reviewed-on: https://gerrit.libreoffice.org/82700
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
Currently pressing Ctrl+F twice opens up browsers
search bar.
Signed-off-by: merttumer <mert.tumer@collabora.com>
Change-Id: I213d4e4c770a035714abe7bcd0e54c74929ade27
Reviewed-on: https://gerrit.libreoffice.org/82694
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
Change-Id: I0386a05adcf2aae2ca94e186cfcafcba8ff2ad8b
Reviewed-on: https://gerrit.libreoffice.org/82676
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Change-Id: Ide40b9a06485960e63dbee9e94c3745b2cd59e3e
Reviewed-on: https://gerrit.libreoffice.org/82666
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Change-Id: I3a8882fbb76fb5c1d364381650d1424f140ccbd2
Reviewed-on: https://gerrit.libreoffice.org/82606
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Fix merge error with:
The problem was that in CalcGridLine.onUpdate the integer costants
Number.MAX_SAFE_INTEGER and Number.MIN_SAFE_INTEGER were used.
Unfortunately such constants are not supported by IE11.
Change-Id: I973f232fa48ce5ba1d125325c2ccf5d4e6610935
Counter-intuitively, our retina scaling pixel multiplier is a
different integer multiplier than L.Util.dpiScaleFactor() in many
cases, being either 1 or 2.
Change-Id: I18078eca80b86216751738368d9885151a41dde2
The problem was that in CalcGridLine.onUpdate the integer costants
Number.MAX_SAFE_INTEGER and Number.MIN_SAFE_INTEGER were used.
Unfortunately such constants are not supported by IE11.
Change-Id: I0a0fba0c1eaa922b925168c16d3afd395c0d38f0
Based on information from UNO ViewRowColumnHeaders messages, this draws
the grid lines with Leaflet's SVG polylines, in a background pane.
To prevent visual glitches on touchscreens, this also reverts parts of
26aad36771ebd5a304b763912184acd2ea26dfdf - that introduced a regression
that prevented 'dragend' events from being fired when inertia-dragging.
Instead, inertia drag is disabled in the L.Map constructor.
[ Miklos: cherry-pick these early as they prevent automatic merging. ]
Change-Id: I86c68214822a0a2c2f8ced82ccf5a6c201e2cf04
We need to force the document size recounting in Impress, because the
document size actually "changes" there on zoom: The annotations are in
absolute pixels, so when you zoom out, it is as if the absolute-sized
annotations grew larger, so we have to extend the document size,
otherwise they are cut off.
But triggering the document size change too early leads to the
sideeffect that the layer with the cursor and the tiles grid get out of
sync.
Let's do the same as in writer - react on the 'updatemaxbounds' event,
and force resize from there.
Change-Id: If84ddf3597be0383ff6ae2271e6ba6ae3bbd5210
Reviewed-on: https://gerrit.libreoffice.org/82604
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
