This reduces some logs from debug to trace,
especially in Socket where the amount of
activity is high, but the logs in question
don't add value in debug level.
Also supressed an error when superfluous.
This makes debug-level logs much more readable
and relevant to the main events.
Change-Id: Ie2698b56f320d15a09060cf2877477d9da0e1582
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
Transitioning the state must happen before
fiting the events that require the new state.
This was a source of random failures.
Also, improved other aspects and added
more assertions and logs.
Change-Id: I55d5a7d5f13eeedaff7f309402baaae37b0134d6
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
First Group app specific commands
Second Group Zoom commands
Third Group visibility toggle
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I822d1ea8ef3460bea9869235b039749dad028c17
after switching to master view previews would remain unchanged
this patch will invalidate tiles for all the previews
Signed-off-by: Pranam Lashkari <lpranam@collabora.com>
Change-Id: Ie7320d625bd95fe80fc76a459b576d6148b39276
there are 4 type of color sections:
1. text
2. background
3. border
4. primary
and for each section there are
1. without extension for default color usage
2. -dark suffix for selected
3. -darker suffix for hover
4. -lighter for non available
with this setting the developer can choose
easy the section and the -suffix.
In addition there is primary color
which bring color the the apps and should depend
on the app so
- writer = LibO brand color blue,
- calc = LibO brand color green,
- ...
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I668b5505abd5d70345221bc79d921071c4a49cd7
In the past there was a padding which was removed
now padding was added again.
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I52f11aa3e36e331e47bee345212d103f2ebf1b65
Avoid doubled PNG generation when graphic is selected.
We only need 'rendershapeselection' request to get preview.
'gettextselection' will only setup '_selectionContent' in
the Clipboard code but it will be never used as
`_getHtmlForClipboard` will ignore it for complex selection
with graphic.
Change-Id: Ic185aa3cb222ea97cf4c69fc65b5db350e568e42
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
To be safe, we can ignore the idea of this.map.isPermissionReadOnly()
control would always return true for PDF views and add
additional check for app.file.fileBasedView which is always
true and readonly true for PDF view
Signed-off-by: Mert Tumer <mert.tumer@collabora.com>
Change-Id: I8f0a4401087c735508534404ee987bb35c3bb0fd
- Remove extra left padding of input field parent
- Increase input field's height
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ic2b862cf2e6ad3bdc03504d5550b4deddd15d59b
and move everything related to view application to that tab
Without this change items related with document layout and
interface view were mixed together within the same "layout" tab. This
makes it harder to scan and find the desired action.
- Split Document layout commands and interface commands by creating a
new tab.
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I9d9d7fe88ceae8bc057946e20628132ee63577a0
similar to what has been done with fullscreen command:
8db1fc9bc8
The current is noW present within the notebookbar tabs
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I30822df1a84d62f7c000a3a39df95e84640ffebc
- Make sure a trigger is fired no matter the UI mode
- Handle it (do anything) just from withing the NB widget
(get proper css classes)
Note: similar to what we did with statusbar in:
e384c561798e7616f10e763788e92acec6caff9c
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I6c8a38f2e426f20cc5b7a06241f7a92fa4d938e4
Without this change items related with document layout and
interface view were mixed together within the same "layout" tab. This
makes it harder to scan and find the desired action.
- Split Document layout commands and interface commands by creating a
new tab.
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I625bb48a6c135d390f595511ddc8639ea5bdd2a8
Fullscreen is not present within the notebookbar tabs and since it's a
not so frequent user interaction is better to remove it from the other
places
note: keeping the presentation mode present in the impress since it's
different command and it's not present in the NB at the moment
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I7f59387c32d71fddae0fb4001b0b098e71a1545b
Fullscreen is not present within the notebookbar tabs and since it's a
not so frequent user interaction is better to remove it from the other
places
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I9f1d95ab30a8168471fe78a4590268f9338bcf81
Make sure the toggle also reacts not only to the user interaction¹ but
also at the startup time
1 Which is already done with:
- e384c561798e7616f10e763788e92acec6caff9c
- from PR: https://github.com/CollaboraOnline/online/pull/4088/
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I29e401eede4fc11c821da91b66430ba99d1ec1aa
- Make sure a trigger is fired no matter the UI mode
- Handle it (do anything) just from withing the NB widget
(get proper css classes)
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ifdf18dc34a3b7d09e8224695ee0e72908b2166d8
They are toggles and not just on switchers
ref: 71831e2733
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: If3b1af0097723ba0caf66804dbb4e4e219f79abc
If you hover at the classic toolbar at the menubar,
a border was shown and the string jump at 1px size.
Signed-off-by: Andreas_K <andreas_k@abwesend.de>
Change-Id: I5d03a380c2bfa34bd169bb12295d535875e54a5f
In the layout tab there are the object related stuff like
align and front/back. So I add there SelectAll command.
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I4cdd9fbb0351cede9cd78dd1edd50e0612d895f0
if you hover over the last-mod string the color switch from
var(--color-text-lighter) to var(--color-text-accent).
In addition the text-decoration: underline was removed
cause it use text-lighter the focus shouldn't be there
with an underline.
Signed-off-by: Andreas_K <andreas_k@abwesend.de>
Change-Id: I750981730b6daa8605f8d70ebd3df4029f990e36
2px thick lines same as other icons in the statusbar
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I08d493038047be16af4c8f4a12cdb22684decf11
For example, to skip exporting the first page of a document:
curl -k -F "data=@3page.odg" -F "format=pdf" -F "options={\"PageRange\":{\"type\":\"string\",\"value\":\"2-\"}}" https://localhost:9980/cool/convert-to > out.pdf
https://gerrit.libreoffice.org/c/core/+/128849 has more examples.
Signed-off-by: Miklos Vajna <vmiklos@collabora.com>
Change-Id: I6c4ce25bc580dac041f2865f74c856780d6fe137
it only checks the cellPos and this can cause
modifying the existing comment when we try to add
a new one if the same cell has a comment in another
tab already.
Signed-off-by: Mert Tumer <mert.tumer@collabora.com>
Change-Id: I69fd607b487fe18c9a48420b7df202b7d3db311d
AS opposed to color code the tabs' titles depending on the application
(doc type)
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I448428d37d2b9f607f176d05c9ef59ce79b1be28
there are 3 color var's added
var(--color-main-text)
this is the default text color EVERYWHERE
var(--color-text-accent)
will be used when a label was selected, or during hover
also for the filename accent text color was used
var(--color-text-lighter)
was use for saved date or when a label was disabled
PR was tested for classic and notebookbar
for the sidebar and dropdown elements.
Signed-off-by: andreas kainz <kainz.a@gmail.com>
Change-Id: I21795875e58b956081f2020422e77ccecd9bfba3
It should be Shapes as it was before.
----
Context:
Regression started with following commit:
5c2e457eae
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I85f064e2158175dd7910b09d03c29255117c345e
To avoid overlapping of elements (FAB + snackbar) with the same color
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I9254316a16b754fd6ea41a19f55dcacdf4b463e2
- Pass additional class if in presence of an infobar
- Styling
- Also avoid hard coded pixels
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ic464e4b0928bf45cb395f4655fb2aeaa9d390118
in order to differentiate from the others since:
- It's the welcome on-boarding
- It's not possible to move the dialog
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I838a8ce0977864374372c46fa8e81945f57afaa2
- Avoid relaying on absolute measurements
- So we don't need to keep track and maintain them
- Also make the wrapper occupy the whole screen
- So the does not accidentally and unexpectedly starts to
interact with the document itself underneath
- Can be also good if we need/want to bind an action for when the
user trigers a click outside of the dialog
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I7c84554b933457800c9ddd579acfd76761baad23
use Uri::getAuthority():
"Returns the authority part (userInfo, host and port) of the URI. "
.
Change-Id: I8c63a4c00cc1ad199f6f0cfd15f4e42fac4e7aaa
Signed-off-by: Henry Castro <hcastro@collabora.com>
