Since it's available already from the context menu.
Change-Id: Ia79a5f21b2d757621a3cb8d7114b90617c6c2acf
Reviewed-on: https://gerrit.libreoffice.org/64181
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Tested-by: Andras Timar <andras.timar@collabora.com>
Not enforced at the moment
Change-Id: I5e2b0def768e7b887425b6b0adc2c790565e42ea
Reviewed-on: https://gerrit.libreoffice.org/64148
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
not the whole URL with all parameters
Change-Id: I42e3a6a4c05410284afea51bb13ac3f692b243ef
Reviewed-on: https://gerrit.libreoffice.org/64147
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Once the Format>Character... dialog works better in a mobile app we
might revert this. But for now the font selection feature in that
dialog is sadly practically unusable.
Change-Id: I60ae6469a588b50b5e2acb5a6a04a04271b1ca47
Even if the LO core code, as called by the Online code, already has
saved the document back to the file from which it was loaded, in order
for a file provider extension like NextCloud to notice that, it is
essential to call saveToURL:forSaveOperation:completionHandler:. The
contentsForType:error: method can just return a NSFileWrapper for the
same, already saved, file, though.
Change-Id: Ic063c8603ca38930083866d973e500336cad517e
MOBILEAPP does of course not expand to empty if it is not defined as
an m4 macro, it stays as is.
Thanks to samuel_m for noticing.
Change-Id: Ia76062b347627289adab0c1d6635f03850a0f2b3
One more special case where we do want to do the whileScrolling
callback from the onUpdate callback: When selecting another sheet of a
multi-sheet Calc document. Handle it by adding an optional parameter
to setPart() in Parts.js from _serPart() in Control.Tabs.js, that
eventually gets passed down as the "scroll" parameter to
_onUpdateCursor() in TileLayer.js, which turns on the special
handling.
Change-Id: I44772912a0598b78bef182f1b6147ff91d6c88e9
Beforehand, any host could embed the iframe as the Referer was always allowed.
Now, only the loolwsd and the WOPI host are allowed to do that.
Additionally, a config option has been added to add more allowed hosts.
X-Frame-Options supports has been removed as it supports only one host
and CSP is meanwhile supported in ~all major browsers.
Change-Id: I222720e1220116102708c50edaf08e2a4a0aebda
Reviewed-on: https://gerrit.libreoffice.org/63864
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Follow-up to the touch pinch zooming fixes in
e27ae38beb. Need to pass the special
calledFromInvalidateCursorMsg property to the scrollto handler here,
too.
The calledFromInvalidateCursorMsg property is now slightly
misleadingly named. Should rename it to something that covers both use
cases.
Change-Id: I232b6a40708c7635fcebef37ff5640e1ba9eaac5
Don't check window.ThisIsAMobileApp but instead use a much more
convoluted test to catch only the cases where we do want the document
to be scrolled, and ignore the case that causes the irritating
jumping.
In _onUpdateCursor() in TileLayer.js, we know that the scroll
parameter is not undefined when and only when _onUpdateCursor() is
called from _onInvalidateCursorMsg(). We pass that information down to
the onUpdate callback handler passed to the _tweenTo() function in the
_scrollTo() function in jquery.mCustomScrollbar.js where it shows up
as property of the options object.
In that location we also make use of the incidental fact that
options.timeout==1 only when that code is called from mouse-wheel scrolling.
Quite possibly all this could be done much cleaner.
Change-Id: Iefa257bceb54137f25534ccb6786c1d2e315931c
For now the same as before, 256, for both normal Online and mobile
app. If you want to experiment with another value, you only need to
change it in loleaflet.html.m4. I hope.
Note the FIXME in loleaflet/src/core/Socket.js.
Change-Id: Ic881758d7fa70bbfebcf932b7ed6a1da352e375d
My previous commit broke scrolling for normal Online with touchpad
(and presumably wheel mouse). So do it only in the mobile app case.
Change-Id: Ia085549b66ed4584c75d5efb78948f19a33c7907
