Commit graph

21447 commits

Author SHA1 Message Date
Caolán McNamara
8bb41ee775 cid#318947 Unchecked return value from library
Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: Ia8b96035d5038d82a55f68a751fdbda5806ec1d6
2023-08-04 10:17:58 +02:00
Caolán McNamara
9e0b0a0c85 cid#318909 Argument cannot be negative
and

cid#318961 Argument cannot be negative

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I4bae768a3c72361309b84b624c019a315b574d97
2023-08-04 10:17:58 +02:00
Caolán McNamara
1d179cdf76 cid#321981 AUTO_CAUSES_COPY
Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I92d66a3d497d574d819f18960bff5b60e180b715
2023-08-04 10:17:58 +02:00
Caolán McNamara
7cb1dfd4eb cid#321977 AUTO_CAUSES_COPY
Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I50f94b290e40b11d555c51448630835d71a62019
2023-08-04 10:17:58 +02:00
Henry Castro
c56ebcc06d browser: remove hyperlink pop-up
Unfortunately, we cannot show the hyperlink pop-up,
in client side based on cell formula, because the formula
had not been evaluated.

=HYPERLINK(A1,A1);

The server should fire event "hyperlinkclicked".

Change-Id: Ie2b743812493e1790b9e54f0050974c585e61e00
Signed-off-by: Henry Castro <hcastro@collabora.com>
2023-08-04 08:52:59 +01:00
Pranam Lashkari
cbeee7727e accessibilty: added accessibilty fields to the mobile edit button
Signed-off-by: Pranam Lashkari <lpranam@collabora.com>
Change-Id: Ifdb59ff3463a0354549316994f8e80b341e28133
2023-08-04 09:17:58 +02:00
Szymon Kłos
e8b2ea5ca1 Deduplicate onInitialize method in Column/Row Groups
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I404b9de7159b410f922dd1a66991aadc5dae1c91
2023-08-04 08:12:03 +02:00
Szymon Kłos
8ce751a3ca Add action for detecting code duplicates
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I8a601aed33fbdc5654cd2e59f42307161e04e18b
2023-08-04 08:12:03 +02:00
Rash419
3831e6f4e2 docker: fix permission denied in archlinux
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: I6eac76976c6bd076f24c795b3eeec876bfef3e6b
2023-08-04 07:58:56 +02:00
Darshan-upadhyay1110
b38a91d1c5 Add check for Insert_buttons to display in tablet view
Added tablet check for Insert buttons to display on dom or not based on passed condition .
 - By default custom buttons will show up in tablet.
Signed-off-by: Darshan-upadhyay1110 <darshan.upadhyay@collabora.com>
Change-Id: I103461b02405bc03d63671b885a31735668d4111
2023-08-03 18:45:50 +02:00
Shehr Bano
1428888afc Writer: Fix to issue #6977, extends vertical space of ui-treeview in Manage Changes
Signed-off-by: Shehr Bano <shehr.bano@collabora.com>
Change-Id: I8551242fc2b987f00c67088caa0ed07ee8526a72
2023-08-03 18:37:58 +02:00
Rash419
c9a9e5635d show copy/paste timeout warning
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: I438ca03c8122c6c85acac7dda141f685a94fdc53
2023-08-03 09:43:18 -04:00
Jaume Pujantell
63473360b1 save-as preserves document password
Signed-off-by: Jaume Pujantell <jaume.pujantell@collabora.com>
Change-Id: Ibaa00fbe4e83c8e4edfe2a70ede861f9adbd20bf
2023-08-03 11:09:35 +01:00
Ashod Nakashian
75ae9894b9 wsd: Util::make_unique -> std::make_unique
Change-Id: I1063913f91571ea6ad95386a3da6a4072574d155
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
347b3c90e5 wsd: correct assertion
Change-Id: I07d6ec53c08a5f4d7d259224452db4d533e14689
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
5b52f7d8eb wsd: smart pointer cleanup
std::make_shared and std::make_unique
are superior to explict smart pointer
construction. Where we have private
constructors, we can't use them.

Change-Id: I492122f58e958113c6e6d31445c6614ad98c89aa
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
67dd4e4a9d wsd: test: new cool.html post request test
Using our own http library, we request
cool.html with realistic parameters and
validate many key values sent in the
request.

Change-Id: Ie3c0ac042f64bf5d6ed21354a33efac801b32846
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
35605510aa wsd: include string_view
Change-Id: I85632cd6a9fece7d63ca2402c2cf924465df5e43
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
b8854ea2e9 wsd: decouple FileServerRequestHandler from COOLWSD
Change-Id: Ie121849a16bb84195d32e11a67e4aad5753aebfc
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
d47001267b wsd: reduce manual http header creation
We have http::Response specifically for
this kind of thing.

Change-Id: I57130754f95c06f5ab958ca373f21763e74cb6a2
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
5e8707bce9 make: better clang detection
Change-Id: I979014429b0a09883d828acf5ba76105ac255632
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
4f8d0fc4cb wsd: test: better UnitWSDClient interface
Change-Id: I4ed292bfccc57f5b41e2ce44a3c30bcad405ae09
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
c1725faac1 wsd: dump the last quarantined version with the status
Change-Id: I46e1d84ba46bea3bf6cfd9c043af247e8b37afc1
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Ashod Nakashian
fbc4373ceb gcov: support cleaning coverage data and document
Change-Id: I9d309a9e551f2a110b71c773af10e65b60c07c9c
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
2023-08-03 08:40:20 +02:00
Henry Castro
b9a1254126 browser: admin: use access token
The GetCookie was removed due to sensitive
access data because the browser can block to
security concerns.

Change-Id: Ifaeb495202a53125bda9e3705b6e33b7e54e56df
Signed-off-by: Henry Castro <hcastro@collabora.com>
2023-08-02 20:23:42 +01:00
Henry Castro
47485adc3e admin: add access token
The access token will validate the session
when a websocket is connected to the server.

Change-Id: I008b04a499ab0d85fd6f95fc06a511f800c0bb2e
Signed-off-by: Henry Castro <hcastro@collabora.com>
2023-08-02 20:23:42 +01:00
Maks Wolkowinski
1f5919140d Collapse notebook bar from view tab
Signed-off-by: Maks Wolkowinski <m.wolkowinski@beebuzziness.com>
Change-Id: Ic15fb1723bff6861b1fc115ef161cf6caa1b70af
2023-08-02 18:25:37 +02:00
Marco Cecchetti
d99aece12f Improve a11y support for tabbed UI
Provided support for screen reader to report key accellerators
description.

When <alt> is pressed screen reader says something like: "tab control
Home tab selected".
When an accellerator is typed such as "N" screen reader says: "N
Insert tab selected".
When an accellerator for a tab item is typed the screen reader says
the typed letters and the item name as set in the 'alt' attrinute for
the button element.
For instance by typing accellerator for bold style ("1"), the screen
reader says: "1 Bold"

Signed-off-by: Marco Cecchetti <marco.cecchetti@collabora.com>
Change-Id: I64b198f79cb9f7f13dcd1a4fe0890e4317e512dd
2023-08-02 16:43:21 +02:00
Gökay Şatır
7afe5dfe8b Remove tabIndex from container of color pickers.
Turn image buttons into actual buttons.

Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I51dc5c900bdcf2c06f9768526a9363f4539deb76
2023-08-02 15:18:30 +03:00
Gökay Şatır
e414ac3c86 Rename conflicting accelerator keys.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I926dc7145991b5cb3c251cdba43d2bbd67363482
2023-08-02 15:18:30 +03:00
Gökay Şatır
54b16ea222 Make color buttons interactable with keyboard.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: Ibf35d6304a717a5870998fb761ca6630db5df4b2
2023-08-02 15:18:30 +03:00
Gökay Şatır
fd7e041c90 Add focus loop to color control when keyboard is used.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: Ia860c055ac10b31d388bf159eb3dccc4ec101c7b
2023-08-02 15:18:30 +03:00
Gökay Şatır
a73ae5a1a5 Focus on the color input popup when keyboard is used.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: Ibf189aaa6029b3de1e1b4233d583f42e07be21d6
2023-08-02 15:18:30 +03:00
Maks Wolkowinski
3a8ff4ef3d feat: Add hide menu entry in compact mode
Signed-off-by: Maks Wolkowinski <m.wolkowinski@beebuzziness.com>
Change-Id: Id40312f6731cddaab110dc5e55ee7115ce08d6fc
2023-08-02 14:06:13 +02:00
Gökay Şatır
325ef1be01 Add functionality for setting access keys of buttons which are added via insertbutton post message.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I6c5746c36b89f4ad4d6c383a2ee5144aecefc1ed
2023-08-01 10:35:14 +02:00
Gökay Şatır
429f895bbf Allow notebookbar insertbutton post message to specify accessKey.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I0693cae17a71810a1c957b5c5600700073b1242d
2023-08-01 10:35:14 +02:00
Darshan-upadhyay1110
76fc8f176a Add reset accessibilityKey
On focus away or if NotebookbarAccessibility is disabled then remove accesskey
Signed-off-by: Darshan-upadhyay1110 <darshan.upadhyay@collabora.com>
Change-Id: I04310f7ee4ab599a0c53c7aceaba459dd7641681
2023-08-01 13:57:14 +05:30
Pedro Pinto Silva
498a02c522 Show userlist on tablets
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I7c3b45846e465b608f87fa0a0067828e3d276ecd
2023-08-01 09:26:35 +02:00
Pedro Pinto Silva
218583ce22 Tablet: Hide document file name input field
The input field is already being hidden for mobile. For Tablet and
with the increased amount of UI elements on the top bar best to hide
it also. Renaming the document is a quite rare used action when
compared to any of the other top bar actions (switching tabs, open
sidebar etc). Better to hide the #document-name-input when on tablets
instead of trying to fit everything (and end up obstructing tabs for example)

Tablet user can always:
- Use the integration (already used to do many other file manipulations) to rename the file
- Use the in-app Save as can still be used to save a copy of the current
document with a different name

----

Possible future improvements would be to add the possibility to rename
the file when using a tablet:
- Add menu entry and tabbed view button
  - Control.Menubar.js something like {name: _('Rename Document'), id: 'renamedocumentbtn'}
  - Control.NotebookbarWriter.js add bigtoolitem Rename Document
- Maybe in Control.UIManager.js add a renameDocumentBtn function that
  - Creates a JSDialog with document-name-input element + Cancel btn +
  OK btn
  - Ideally that input element would re-use the functionality from L.control.documentNameInput() -> Control.DocumentNameInput.js

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I968e806a4a9e0018b38b87fc249a57e5af099c78
2023-08-01 09:26:35 +02:00
Szymon Kłos
cf545e8152 Don't steal focus from outside iframe on load Calc
When Collabora Online is inside iframe and we load
Calc spreadsheet, after few seconds we receive CellCursor
message. In case when user was typing outside iframe in
some textbox - we were stealing focus, what caused
that user started to type inside spreadsheet.

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I12a5501be6f59f73830ff9bf25c98a00492b6caa
2023-08-01 09:22:01 +02:00
Michael Meeks
6c382f5415 rle: optimize inner loop bit-bashing.
Simplified code:

+ re-use the bit mask as an inner loop counter.
+ turn alpha only rows into zero length with agreed lastPix.
+ keep lastPix around on the stack.
+ handle odd widths in a duplicate slow-path

Change-Id: Ibc7630f7187ea5f4904c6fed14dda28cdfbf7523

rle: further optimize the pixel reading & de-compression.

Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Change-Id: Ifcaaa45d1de6532b3fd43015c47a37daf56c2ea5
2023-07-31 21:31:13 +01:00
Caolán McNamara
8a715cb04c calc insert comment checks undefined tab field
and so this loop never finds anything, the intent is clearly to find
an existing comment in this cell so use the right field for that.

possibly a problem since:

commit 922ae4924a
Date:   Wed Jan 26 15:08:31 2022 +0300

    calc: Fix new comments do not check tabid on insert

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I11208493ef7eae6a01576d90129b25ed9f792d62
2023-07-31 09:05:23 +01:00
Michael Meeks
03d83ab6b8 paste: cleanup excessive debugging.
We don't want to dump the whole paste buffer as hex, or indeed any
large mis-encoded string - instead crop to the problematic section
for easier debugging, and no performance hit.

Change-Id: I30518beea436895a42b3429c3cb6e16e2093cca9
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
2023-07-28 21:38:02 +01:00
Pedro Pinto Silva
ec67c92264 Tablet: Non-functional sidebar button shown in read-only mode
Do not wrongly switch part of the UI to edit mode when on read-only
mode. Best to be only remove readonly css class when it's not needed as
opposed to removed it whenever refreshNotebookbar() is called.

Before this commit:
- We were showing tabs in wrong mode (readonly mode) -> every-time
refreshNotebookbar() was called. Thus, initial tablet mode was
appearing with icons and actions from edit mode.

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I6ad9c8c14e60d4c3513f63b75717db0a124b73ea
2023-07-27 16:44:09 +02:00
Szymon Kłos
a56f034dc6 Share code for findTailsGroup
Introduce getTailsGroupRect method which will be
different in every implementation but share findTailsGroup logic
using above function.

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I321abaa2579bfdff08c8cd4df2e781cd9a28c3d3
2023-07-27 16:01:40 +02:00
codewithvk
8b5df0c4b3 Fix : Menubutton should have disabled state styling different
Signed-off-by: codewithvk <vivekpatel7202@gmail.com>
Change-Id: I1cd97503e0d3ca4333b11552ca3571093cf10bfe
2023-07-27 12:25:26 +02:00
Tex2002ans
06f8ee610f Update feature_request.md to use Headings
I added Headings + updated some wording/examples in:

- Feature Request

so when users submit a "New Issue" to Github, it will look nicer.

Signed-off-by: Tex2002ans <Tex2002ans@users.noreply.github.com>
Change-Id: Ic7c5b0a841d39b2cd2b20c5d4a31846eba0c3f4e
2023-07-27 12:21:25 +02:00
Tex2002ans
6b79e52831 Update bug_report_smoke-test.md to use Headings
I added Headings + updated some wording/examples in:

- Bug Report (Smoke Test)

so when users submit a "New Issue" to Github, it will look nicer.

Signed-off-by: Tex2002ans <Tex2002ans@users.noreply.github.com>
Change-Id: Id72ed9ac10a6ae935fb5fa4b1001c11683f46672
2023-07-27 12:21:25 +02:00
Tex2002ans
dcbc0208bb Update bug_report.md
I added Headings + updated some wording/examples in:

- Bug Report

so when users submit a "New Issue" to Github, it will look nicer.

Signed-off-by: Tex2002ans <Tex2002ans@users.noreply.github.com>
Change-Id: Ie32f2c9a4c6ed47aaf198286a0d39831fa397992
2023-07-27 12:21:25 +02:00
genofire
850089377f fix(helm): update appVersion
Change-Id: I497d36c76f74791b8535e6e89bc9acca536c58e3
Signed-off-by: genofire <geno+dev@fireorbit.de>
2023-07-27 12:47:29 +05:30