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>
For an array, memset(array, ...) and memset(&array, ...) do the same
thing - but given that there has to be the sizeof(array), the former
is probably more readable / obvious here.
Change-Id: I7e329cb1dcabb564e26857b0c7d9f88431bb4ede
Reviewed-on: https://gerrit.libreoffice.org/82601
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Miklos Vajna <vmiklos@collabora.com>
mobile:impress:mobilewizard:hide scroll-x
Change-Id: I3faa5cc983e50dcfbd85b7e3e21edc6b9652300d
Reviewed-on: https://gerrit.libreoffice.org/82605
Reviewed-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
Tested-by: Pedro Pinto da Silva <pedro.silva@collabora.com>
It is not a good UX to change names of items in the menu - we have a
different mean of indicating that the master page view is active: the
tick in the menu.
Change-Id: I53486847b3929d990de6e398df96b04b943998e6
Reviewed-on: https://gerrit.libreoffice.org/82498
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
This simplifies lots of logic, making it trivial to get good row/column
header alignment at any zoom. Ultimately TWIPS were highly misleading
'print-twips', and in that state are ~impossible to map to pixels
accurately unless all data is present back to the origin
(which was not provided previously).
This requires the equivalent patch on master.
Change-Id: I110f6f646e2b52bec4584c193c640df6b41100f8
removed use of Poco::StringTokenizer from the wsd directory using LOOLProtocol::tokenize and std::vecor<std::string>
Change-Id: Ic50b4d4d71d4ffd005aacf6aef0ed2bfde66d40d
Reviewed-on: https://gerrit.libreoffice.org/82569
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
removed use of Poco::StringTokenizer from the tools directory using LOOLProtocol::tokenize and std::vecor<std::string>
Change-Id: I0673e658fd35cbdc7425a99f1dcea0b54923f52c
Reviewed-on: https://gerrit.libreoffice.org/82568
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
removed use of Poco::StringTokenizer from the test directory using LOOLProtocol::tokenize and std::vecor<std::string>
Change-Id: I20fc2e0ef0d0d8fc959fee7972aa095f2581c181
Reviewed-on: https://gerrit.libreoffice.org/82565
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
Tested-by: Jan Holesovsky <kendy@collabora.com>
This was originally moved in commit
b97789eb76 (Convert paste testcase to a
new-style one, 2019-10-18), then
194db8ed45 (test: improve stability of a
number of tests, 2019-10-12) restored a duplicate of this, probably due
to a mismerge.
Change-Id: Ie76266ddb85c58bb2370640025b91af980656908
ChildProcess instances log in the dtor, and this can
happen after the log subsystem has been destroyed,
because the order of destruction is undefined.
NewChildren is destroyed in exit, at which point
we have no control over the log subsystem.
To avoid this inversion of order, we clear
the NewChildren container before exiting.
Change-Id: I7b1bac3123fd7917616c11ab9674534c9222fff1
Reviewed-on: https://gerrit.libreoffice.org/82533
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
