Commit graph

15709 commits

Author SHA1 Message Date
Ashod Nakashian
61954dc67a wsd: track saving activity
This prevents auto-saving while a
save is in progress.

Change-Id: I2606df704482ad54a954414d48ed3ca35d4764a9
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-07-17 22:58:23 -04:00
Ashod Nakashian
cc23a03dc0 wsd: correctly update the modified flag
There is an important distinction between
internally saving a document and uploading
the latest saved version to storage.

Core reports the modified state, which was
at some point used to flag the modified state
in the admin console. This state is misleading
if uploading failed.

This patch makes this distinction even more
explicit than it was. We now always update
the modified flag given by Core, and flag
the document as modified in the admin console
too.  But we only update the admin console to
unmodfied when we successfully upload and the
document is not modified at that point (which
it might very well be modified again by the
time we finish uploading).

Without this patch, we stop updating the
modified flag when uploading fails. This
is problematic when said failure occures
right before closing the document, because
the document will have no modification
but we will be stuck thinking it is modified.

This combines two commits:
8c602e179e
and
9db39ce741

Change-Id: I31c72aa2491b8e98597075cd35b6a809fab560a1
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-07-17 22:58:23 -04:00
Ashod Nakashian
9c50fdc155 wsd: better logging of the document activity
Change-Id: I89e673ffa819600bd227fab291a46d9108ed5cca
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-07-17 22:58:23 -04:00
Ashod Nakashian
84e9732e79 wsd: test close-after-modify to confirm no data loss
Change-Id: I538b0529b318d8a0609798dc0eecadb49861e188
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-07-17 22:58:23 -04:00
Ashod Nakashian
8ff82c007b wsd: test: correct logging of phase change
Change-Id: If71e5c5a6d6b4fd03d3f4fc98ba59d8ad55d8b0b
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2021-07-17 22:58:23 -04:00
Szymon Kłos
87cc41eb0b jsdialog: sidebar: add more options buttons for panels
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I49024be7a3a46665f709c1b6a063a25a0d6ddf38
2021-07-17 17:31:53 +02:00
Gökay Şatır
34b6f0d760 Revert "Avoid higlighting additional rows/cols"
This reverts commit a701f7c9094227422457d05cc5d90e9d1a9a7fc0.

I approved this one but it doesn't seem to work well, unfortunately.

Signed-off-by: Gökay Şatır <gokay.satir@collabora.com>
Change-Id: Iafc10a15dcf07aae661e7dd2562c6555f6b83596
2021-07-17 16:35:27 +02:00
Szymon Kłos
d7c985e9b5 jsdialog: handle exception on bad id
issue wrt busypopup sending string id for dialogEvent
was already fixed on JS side.

remove unnecessary comment from commit 48f7449
Ignore conversion error from std::stoull for "busypopup"

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I7640b4f693d9d018ae17acb3a9eb292a2423bb68
2021-07-17 16:35:27 +02:00
Michael Meeks
1c2bbced33 systemplate: ensure we have write access to directories before cleaning.
Some of the folders we setup from the system are:

dr-xr-xr-x 1 michael users 2824 Jun  9 21:05 systemplate/usr/share/fonts/

before adjustment, and resist removal.

Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Change-Id: I8e5a96264b98d8091b205f7469bc46c401f47ab1
2021-07-16 19:25:44 +01:00
Gökay ŞATIR
fc1948da83 Impress: Hide comment list panel when there are no comments on the selected part.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I33b30857d0ada79fe06f72e3e54042a8c5c4a64f
2021-07-16 21:14:18 +03:00
Gökay ŞATIR
9bd5154416 Calc Comments: Improvements.
Hide comment on scroll.

Don't show another while one is being edited.

Remove section properly, it was hanging on the background.

Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: Id0f85d4c84d3741a5120e24055061c2fe183b98a
2021-07-16 21:07:28 +03:00
Gökay ŞATIR
df75abe909 Calc: Hide comment on scroll.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I4852c15891f2467566964524bd7eb05450878ad6
2021-07-16 21:07:28 +03:00
Gökay ŞATIR
09c695a3f1 Calc: Issues with comments.
Open new comments directly in the cell's position, without animation.

Focus on the comment box.

Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I0dda23baaa9c8cdd940fc9d429e0e72e5464b854
2021-07-16 21:07:28 +03:00
Szymon Kłos
894f77e71c Avoid higlighting additional rows/cols
When used 120% zoom and selected any row after 50
there were multiple rows highlighted.

Change-Id: I4bc5f4272bf65fb0f9d00d96527890281377c0a8
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-07-16 13:20:45 +02:00
Szymon Kłos
556ef0d85d notebookbar: reduce importance of 'not initialized' message
- when we receive any notification - mark notebookbar as initialized
- if we haven't received any information eg. in impress - warn
  reduce importance of this from error to warn as it is not important
  probably there is no widget which sends updates (correct for draw and
impress)

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I88ae3a090014dd25e59e04993c0c244d8830435e
2021-07-16 13:20:45 +02:00
Szymon Kłos
a92c71e3ba jsdialog: style all buttons
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I919f4948962e378fcd4690ca9cfa73fa4a273146
2021-07-16 12:05:19 +02:00
Szymon Kłos
631930da92 jsdialog: handle images
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Ib8b0e17696791a6f9c32ab4e2879d226b7889dfe
2021-07-16 12:05:19 +02:00
Jan Holesovsky
41c0b1add3 Sanitize the css theming.
Change-Id: Id0eb21fc7e7039bf406c074cd210c6c355bd7dbf
Signed-off-by: Andras Timar <andras.timar@collabora.com>

Sanitize the css theming.

Signed-off-by: Andras Timar <andras.timar@collabora.com>
Change-Id: Ifccb58c2bc9a070409c9a4dd9e9e4034e7338469
2021-07-16 11:57:43 +02:00
Mert Tumer
cb5a91cce2 Parameters for perf-test
now it's possible to pass parameters
typing_speed, typing_duration, single_view, node_inspect

also replaced the old perf-test with new-perf-test

Signed-off-by: Mert Tumer <mert.tumer@collabora.com>
Change-Id: I03f1f14db4ddd9afb9d4a9a60a87c135497db70c
2021-07-16 10:41:05 +03:00
Mike Kaganski
c13fa52613 Use custom local branch name as explicit remote user branch name for PR
In a scenario where onee has several parallel PRs, one needs to use
several local branches, and when updating PRs, _needs to provide PR
branch name explicitly_ each time (my-PR-name-* in the example):

  git checkout featureFoo
  [ edit session ]
  git commit
  ./g review my-PR-name-1
  git checkout featureBar
  [ edit session ]
  git commit
  ./g review my-PR-name-2

This changes the processing, so that when the user's local branch name
is different from the tracked branch name, it is treated as if user
used that name explicitly in a call:

  git checkout -b featureFoo origin/master
  git checkout -b featureBar origin/distro/collabora/co-6-4
  [ ... ]
  git checkout featureFoo
  [ edit session ]
  git commit
  ./g review # this uses "featureFoo" as implicit second argument
  git checkout featureBar
  [ edit session ]
  git commit
  ./g review # this uses "featureBar" as implicit second argument

so no need to remember which PR name was used where.
This only works for local branch names that are not equal to a remote
branch name, or to the last part (after last /) of any distro branch.

Signed-off-by: Mike Kaganski <mike.kaganski@collabora.com>
Change-Id: I8b95f92f6e205e3c93ef30f4813f3549c2315872
2021-07-16 08:53:58 +02:00
Rash419
506509cdcd cypress: make some tests more stable
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: I61b963771c05d8f45140404c58325638b4e03b58
2021-07-16 12:23:17 +05:30
Rash419
5ea11097b9 cypress: enabled some tests
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: I34473c4b471b30d05be61e27ecb33d5ad430f2c3
2021-07-16 12:23:17 +05:30
Mert Tumer
1e015b3d07 moved initializing fontcombobox with select2 to Toolbar.js
This way both notebookbar and classic toolbar will have the select2
styling

Signed-off-by: Mert Tumer <mert.tumer@collabora.com>
Change-Id: I9b8294cb1c0308fcfeeed8f9e886e3075c2eb3a1
2021-07-15 21:15:00 +03:00
Gökay ŞATIR
c3f6d9fbd4 ScrollSection: Make vertical scroll bar always visible on desktop.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I39cfe37dddc603cc35a4cb02a2bfd45db224df52
2021-07-15 18:53:21 +03:00
Gökay ŞATIR
82d5a9c185 Cypress: Workaround for a bug inside the library.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: If8689bbc9505d05d8343b624cb0213a583f362a2
2021-07-15 18:18:11 +03:00
Henry Castro
21aa279659 loleaflet: check the 'statusindicatorstart' label
The LO Core server will send the label of the status
indicator start.

Change-Id: Ie33aa56fcf2bb87362ad91c79498292c51bd32a8
Signed-off-by: Henry Castro <hcastro@collabora.com>
2021-07-15 15:21:24 +02:00
Henry Castro
f43c4b32e5 kit: add payload to 'statusindicatorstart'
Change-Id: I3145c6c622204e4c4898c8490176baff39b07636
Signed-off-by: Henry Castro <hcastro@collabora.com>
2021-07-15 15:21:24 +02:00
Rash419
046dbb72f9 cypress: disabled failing tests
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: Ia302f05f2524c446f32b7baccdd86d25766b1707
2021-07-15 17:54:02 +05:30
Michael Meeks
823c30b39b Ensure that we get the file-name right before sending the save result.
We cannot rely on wsd being slower than the kit. The rename
has to happen before wsd / DocumentBroker gets to trying to access
the filename. fixes #2874.

Change-Id: Ie1e67cd059fb6a663048967c47759238c067172d
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
2021-07-15 09:08:06 +01:00
Miklos Vajna
f569c2b496 log: don't try to get the channel of a non-existing logger
Signed-off-by: Miklos Vajna <vmiklos@collabora.com>
Change-Id: Ic08043dac01b682184ff440b2f7d7f4f3df9522e
2021-07-15 09:01:16 +02:00
Gökay ŞATIR
72076ff8cb _refreshRowColumnHeaders: Function is calc-only.
Thanks to cypress tests. It was being called in Impress document.

Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I4681486cdd0930429f824c5636a2becb26700ff6
2021-07-14 22:24:57 +03:00
Gökay ŞATIR
115d20082d CanvasTileLayer: Correct the check for the tile source.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I4990f8c9d44c92f711fb4d973bf44ef77812c757
2021-07-14 21:51:32 +03:00
Szymon Kłos
07459e2b7b mobile-wizard: remove old uno status getters
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I73d3b5fdd034fe61ff32a4ef3cddfb9b85b6166e
2021-07-14 18:29:23 +02:00
Szymon Kłos
8152a3f9b5 mobile-wizard: don't use uno status updates
we receive that information by welding updates

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I9ff7f5d26740b1f13d1f1cccd88793ca35a4ee76
2021-07-14 18:29:23 +02:00
Gökay ŞATIR
587e09acd4 Adapt addSection function with pauseDrawing.
User can pause drawing before adding multiple sections.
When resumeDrawing is called, reNewAllSections function will be called automatically.

Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: Ie83265b5a57c11bd828a980c1797d6bfa3d6e287
2021-07-14 19:24:51 +03:00
Szymon Kłos
e3202f6783 jsdialog: apply selected state for toolitems
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I85015f5f10e38a26ed6811180be1eb309679b399
2021-07-14 14:48:08 +02:00
Muhammet Kara
c16bb5058f Add welcome messages for pull requests
Signed-off-by: Muhammet Kara <muhammet.kara@collabora.com>
Change-Id: I54050204ecddc6fe0871a1424db164977442a176
2021-07-14 15:13:10 +03:00
Szymon Kłos
1b9a73c0b8 mobile-wizard: don't show hidden decks
this fixed mobile Page Setup for writer

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I262da01529dff4740089f03181c90f9b59b9935c
2021-07-13 20:39:57 +02:00
Szymon Kłos
76733b7e79 cypress: fix mobile color picker tests
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I5c8f705e3ca69780ed0c441e7a82bf257da33bda
2021-07-13 20:39:57 +02:00
Szymon Kłos
c2058a8b95 mobile-wizard: handle toolbox update properly
if any of explorable toolitems was opened - open it also
after update

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: If98f68befb1b0e7283564c2bda3f18524e498977
2021-07-13 20:39:57 +02:00
Szymon Kłos
0d3c00dff8 mobile-wizard: when popup is opened don't allow to go back
we need to force proper closing of popup to avoid empty popup
when it will be opened next time

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I8e6b3164dc0d828bcb10c37c657d403664f0395a
2021-07-13 20:39:57 +02:00
Szymon Kłos
2a9678595f mobile-wizard: use mobile colorpicker for shadow color
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I04b0262ec27fc325297e1abf55a9193a8bbcefbf
2021-07-13 20:39:57 +02:00
Szymon Kłos
ea56502a7e mobile-wizard: on control update use the same visibility as old control
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I5b2a8824895c90480b6038d899d1ca52347fbcaf
2021-07-13 20:39:57 +02:00
Szymon Kłos
5f2167c2bf mobile-wizard: show mobile colorpicker for slide background
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Ia02ff7562df2e1051d05c82ab77d29acd44f824a
2021-07-13 20:39:57 +02:00
Szymon Kłos
74be366bc1 mobile-wizard: fix explorable entries inside toolboxes
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Id2e3de7942cf8f7b4016f2c250c80971cb9d9d01
2021-07-13 20:39:57 +02:00
Szymon Kłos
0d96e1ce97 mobile-wizard: build also toolbox container
we need it because toolboxes are sometimes hidden dynamically
and we need the DOM element with correct id for hiding.
example: colorbar_others / colorbar_writer in Characters Panel.

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Idccb9cd46feda34c3093b61354a4352677bb582b
2021-07-13 20:39:57 +02:00
Szymon Kłos
da39e28fa7 mobile-wizard: on explorable entry update save visibility state
When update occurs and explorable entry was opened eg. color picker
in shape effects panel we need to rebuild it as opened.

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I7a564bb32c8e7ee075a2ac32235088bdda19130e
2021-07-13 20:39:57 +02:00
Szymon Kłos
e5fc1d0bca mobile-wizard: fix shape glow panel
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I5903b0f8fe0b39d7243db59a0627f62644942156
2021-07-13 20:39:57 +02:00
Szymon Kłos
0689ff580a mobile-wizard: fix Panels icons and visibility
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I25e7f7704f3c0fd1cbd23fcf8c3a0750e631ef9d
2021-07-13 20:39:57 +02:00
Szymon Kłos
4cb9d8cd33 mobile-wizard: use mobile colorpickers for gradient
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Ic5ef466713cec184328d945424affb0d1a2a3b30
2021-07-13 20:39:57 +02:00