Commit graph

14245 commits

Author SHA1 Message Date
Pranam Lashkari
e3fa8bafcd annotation: fixed error on annotation insertion in impress
Signed-off-by: Pranam Lashkari <lpranam@collabora.com>
Change-Id: Id12b3e422fcb7970241535345f3e90271ede42eb
2021-03-11 08:45:28 +05:30
Pedro Pinto Silva
a1134c8cd9 Mobile: Comment (annotation) layout improvements with grid
- Position the number of replies near to the chevron to indicate what
that is supposed to do.
- Reply counter gets the same color as its icon

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Idd692f40f5f3420d7577147e8ca7bcf306e59899
2021-03-10 16:44:36 +01:00
Dennis Francis
5e33750de5 clear canvas before painting async non-calc tiles
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: I642c0efdbe2c5d0492406b0e180ba5c15578d36f
2021-03-10 17:14:50 +05:30
Dennis Francis
0c8e11b7d3 redefine desktop calc view position on zoom
The view area after zoom is computed by the following rules:

1. If there are not edit cursors or it is out of view, anchor on the
   top left cell.

2. Else, compute the new view area w.r.t top left cell, but if the edit
   cursor is not in that resulting area, translate it such that edit cursor
   is just inside the closest edge of the final view rectangle.

Notes:

As part of the implementation the default behaviour of view scrolling on
cursor msg was disabled while the new view area is being set.

If there was an active edit cursor to start with, ensure it remains the
same way. This is acheived by replaying the last print twips cursor
messages after the new view area is set.

Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: Iaada6ec1f4a06f7ad05dec759e702bd37240dff8
2021-03-10 17:14:50 +05:30
Dennis Francis
78cec95d77 calc: call refreshViewData() on section initiated map scroll
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: If1383f4d845f4b3a77e0a21b8af5f5d8378bfc8e
2021-03-10 17:14:50 +05:30
Dennis Francis
2577342e1b avoid body/tilepane bg implicitly setting the border of toolbar
by setting a top padding of right amount.

Before the patch the top border of toolbar "appear" white on the part that
intersects with row-headers and it was gray where it intersects with the
document content(tiles). These were due to the background of body and
tile-pane div respectively.

Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: Ia3b95d41ea150d815469498e3e2044d7e1d36a4d
(cherry picked from commit 66416be43980205dea1a2addefbf5954e013c3cc)
2021-03-10 17:14:50 +05:30
Dennis Francis
51c178d060 overlay: translate mousepos to doc coordinates
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: Iae9222a6704a2f8fdb2c1dc2347bc5106d458bff
2021-03-10 17:14:50 +05:30
Dennis Francis
ce70646b61 factor out and generalize latlng->layerpoint computation
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: Ie983003dfd8cb7e7093fcfe5374bb31ed2f5995b
2021-03-10 17:14:50 +05:30
Dennis Francis
a186f4ea90 include dpiScale for viewcursor and references
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: I99b12cd6a1500136dca9c339c4a5cf8624aebe41
2021-03-10 17:14:50 +05:30
Dennis Francis
743ccdd370 set pen position for all path initiated drawings
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: Iab1c9defa417b03232a53024eaa6f52232244d87
2021-03-10 17:14:50 +05:30
Gökay Şatır
46aa9ed6cc ScrollSection: Improve scrolling with touchpad.
Signed-off-by: Gökay Şatır <gokay.satir@collabora.com>
Change-Id: I831df582ec21aa040f6af2df7b5e6c4f8883c988
2021-03-10 12:27:19 +03:00
Pedro Pinto Silva
36ecadffe7 Add missing image for connectortoolbox
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ib3a7a99ccab6b5ad70234cf56e6573fea1eb56e7
2021-03-09 15:16:20 +01:00
Szymon Kłos
051460559d Don't steal focus from inputs in calc
Change-Id: I03647e328949c30f132a6f88af2e302743ef606f
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-09 14:08:25 +01:00
Szymon Kłos
0a5c8011e9 Don't steal focus from annotation in calc
When CellCursor command result arrives we focused
the document what caused annotation closing.

Change-Id: Idf2eb8c5b3b0049dd0aeb7c82cca80083b304698
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-09 14:08:25 +01:00
merttumer
51705336d7 Added Draw to "Launch one of these" prompt on startup
Signed-off-by: merttumer <mert.tumer@collabora.com>
Change-Id: Idbcd5d1b9063e2ef3b7478906a6a346e236db51d
2021-03-09 12:40:20 +03:00
mert
0a9b93df43 Selection marker should be enough for search test
Now we hide selection handlers while typing due to
not being able to use them, these tests fail to find
rotation marker. When text is found on mobile selection marker
wraps the found text and they become visible. when not found they
become hidden again. Checking the selection marker for this test cases
should be enough.

Change-Id: I6037450bc17438e4819ae3b724849f74f26910d4
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-09 09:59:20 +03:00
mert
a63a08223f Hide selection handlers when cursor is active
Dragging is impossible when cursor is active so
displaying them only leads to confusion

Change-Id: Ib0586ce6ac38354b84076972e01be136f7ec66f9
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-09 09:59:20 +03:00
Ashod Nakashian
95bb52d945 wsd: test: better logging
Change-Id: I3d17afb2e76f1d4b2b27182a6419482801342e8c
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
89a587a695 wsd: always set the Fin bit for WS frames
Change-Id: I60f676fb5022ee1b2740f5cc6974595f3e86d039
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
a533627b5a wsd: test: improved logging
Change-Id: Iac80450cd88997572727ffe30e0b26737df96f46
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
8178a7be9f wsd: test: log the length in decimal instead of hex
Change-Id: I4abaa903a529e2e6a99af5e0056883c084c1e9ea
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
05ec2a68d6 wsd: copy strings only when needed
Change-Id: Ia63d5f9526bbcd295a9a5d18990a4f1ab744a5b2
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
af454178ae wsd: avoid unnecessary string copies
Change-Id: I61d50fa2bd1c7e9fc673eb58c9ba85127d6e4883
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
c740603d14 wsd: simplify dequeue to do more outside the lock
Change-Id: Id27739401e2083e481f120f306418733a3e3563f
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
43eaf4d8df wsd: test: better exception handling in UnitCopyPaste
invokeWSDTest is called in an exception-safe scope
and logs errors before failing tests. No need to
have the same logic here.

Also improve the exception handling and reporting
in the fetchClipboardAssert helper.

Change-Id: I306aedc3f50f93bf8a5448f1d098e5ce884ceb37
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
74d019e716 wsd: cosmetic
Change-Id: I98d16305d461f5b081756d4a5c518d3616483538
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
1837fb7fa1 wsd: avoid using shared_ptr copies
Change-Id: I5ee6e1cf56d984e0cb5407c556d53fc736cdb794
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
6f59004e5c wsd: remove unused time_point argument from drainQueue
Change-Id: Ib6c48ab63ab09205e90968c6d455848e2d24f3e4
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
a7aeea33cf wsd: cosmetic
Change-Id: I4885e91a3fdf0f1cd5caf79e7a41a289dde42150
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:49:53 -05:00
Ashod Nakashian
3f5e9d5922 wsd: terminate Kit when disconnected while loading
When loading hasn't completed yet, DocBroker can't
simply send a message to Kit that the session is
disconnected, because there would be no UI or
handlers for events if that were the only session.

Of course letting the document load, only to close
it, is also less than helpful to resource consumption.

Here, we recognize this case and simply terminate the
loading by killing the Kit process altogether to
avoid any potential hangs or resource wastage.

Change-Id: Ia72de715cc6238831c244444bb47b417b9b1e1a4
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-03-08 22:17:42 -05:00
Michael Meeks
07885e2f2b copy/paste dialogs: stop propagation to avoid confusing the document.
Previously a 'click' on 'start download' - also resulted in a pair
of mouse down/up events getting sent to the map, the TileLayer, and
thus the core - resetting the object selection during the download,
and causing nasty issues in the second 'Confirm copy to clipboard'
which had a different & random selection depending what was under
that popup.

Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Change-Id: I7f6510494500dd36beb232e4720a66f2d9e09f27
2021-03-08 22:16:22 +00:00
Tamás Zolnai
ea5983e957 cypress: remove this redundant logging.
cy.visit() logs this URI anyway.

Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: Idbab1d0cd8cc0f9f0b62b194fb7f1a526c8b9f97
2021-03-08 19:08:25 +01:00
Tamás Zolnai
e034866430 cypress: make: spec can be optional here.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I98448dbb8710ea869618b341b251a0f8657386da
2021-03-08 19:08:25 +01:00
Szymon Kłos
864b761684 jsdialog: don't scroll whole page
scrollIntoView scrolled whole page inside iframe
when selection in the fontwork dialog was changed

Change-Id: I40f59c1821d1327aaf6ce910e07587f0dcb5af31
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-08 17:40:24 +01:00
Henry Castro
fe3135e49c jsdialog: fix line breaks for 'multiline' control
Change-Id: Id3213519a9224ee0287b5dfdd0251230e4c5f00e
Signed-off-by: Henry Castro <hcastro@collabora.com>
2021-03-08 16:32:12 +01:00
Szymon Kłos
486152f400 notebookbar: draw file tab
show only .odg and .pdf formats in file tab

Change-Id: If95d871d0ec546569a3ccb0334b9acbd971a9f86
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-08 16:24:23 +01:00
Pedro Pinto Silva
a00e20ad50 Vex input field: ensure box-sizing
to avoid padding affecting otter size anywhere

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I45122bbb27d37de3bfb15f201c10a4888f2ff5c2
2021-03-08 14:41:56 +01:00
Pedro Pinto Silva
c9ff69c2f5 Mobile: Fontwork: Shape: Fix alignment and make it more dynamic..
- center main div
- set minimum size for children in %
- make sure everything is centered

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Iab199dca2a4907ca7292cc8b56cf6535b3013c0d
2021-03-08 14:41:56 +01:00
Pedro Pinto Silva
52d2b02297 Mobile: Fontwork: treat 'Same Letter Heights' as a main btn
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I20f44fb6b653073226cfe13be0b3c70bdfd77ba2
2021-03-08 14:41:56 +01:00
Tor Lillqvist
e20db7c18b Guard against JS error when there is no "children" in the notebookbar JSON
I saw that happening now and then when experimenting with the Insert >
Comment functionnality in the iOS app. The message in those cases
seemed to be 'jsdialog: { "id": "0", "jsontype": "notebookbar",
"action": "close" }'.

Change-Id: Ia8e07f849f88bb46cedd95f7c38534232c388efe
Signed-off-by: Tor Lillqvist <tml@collabora.com>
2021-03-08 14:36:09 +02:00
Tor Lillqvist
e142d26a2f Don't use CollaboraOnlineWebViewKeyboardManager after all
Unclear whether it is actually useful or not any longer. Quite
possibly cross-platform changes in the JavaScript since
CollaboraOnlineWebViewKeyboardManager was introduced have fixed the
problems that it was solving.

Fixes https://github.com/CollaboraOnline/online/issues/1432

Change-Id: Ifa000d0a7d0a3dc87527536719063b368266f038
Signed-off-by: Tor Lillqvist <tml@collabora.com>
2021-03-08 14:00:38 +02:00
Szymon Kłos
0d2578a66d notebookbar: border style dropdown
Change-Id: I3cc064410fad613984c8e2e531b0e77f2810adaf
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-08 10:19:06 +01:00
Miklos Vajna
d35a5f963e android: de_document -> the_document
Signed-off-by: Miklos Vajna <vmiklos@collabora.com>
Change-Id: Ic85297eb112def4a9197116177c857db910459cf
2021-03-08 10:11:59 +01:00
mert
0e00c3af98 android: Fix typo on onBackpressed
Change-Id: Ic9b6765ec4d881091cc3b72b7e693b93fa598426
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-08 12:07:38 +03:00
mert
c6e4b260da Hide presentation button for Draw
Change-Id: Iaef6f2a405e0e55b082e303447fa9377a0ec96d0
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-08 11:19:21 +03:00
mert
dda2550eb4 Draw: Display Presentationbar for Insert/Duplicate/Delete Pages
We have these on the Top toolbar but in notebookbar case
We have only insert and duplicate options and they are
located in Insert tab. So deletion is not possible
with notebookbar. This patch re-enables presentationbar
with the correct hints of these items according to the doctype

Change-Id: I885b6353bce82232cfdfb5577f8cfbee16b219c7
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-08 11:19:21 +03:00
mert
1546799bfe Fix Slide Sorter does not update on Draw
Change-Id: Id8612ebbcb725b3c03ff6c2262c48c0f42544754
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-08 11:19:09 +03:00
mert
87a82105c6 android: Fix crash when could not load the document
Change-Id: Ie08fcf77560ff285b7f9a94253439e551a08aefb
Signed-off-by: mert <mert.tumer@collabora.com>
2021-03-08 11:14:48 +03:00
Szymon Kłos
74cc7d0278 Don't show blinking cursor when no focus on document
Change-Id: I6dd2ba0a88eff71870dc608ba6d961c9d0f7a67f
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-03-08 07:34:51 +01:00
Andras Timar
f939aef0ba typo fix
Signed-off-by: Andras Timar <andras.timar@collabora.com>
Change-Id: Ib43f1112170ca4d66e8127644e22e3f7c17f10e4
2021-03-07 20:25:32 +01:00