Age | Commit message (Collapse) | Author |
|
Change-Id: I3a84b1f7c8521b861e99384872127bb4d224c18b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118409
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120542
Tested-by: Jenkins
|
|
The change allowed to simplify many places where previously this API was
used, to avoid inefficient calculations (e.g., moving rectangle keeping
its size, and then immediately changing the size).
Change-Id: Ica2dc594d91cae83e2c2740c1f4fb23f44998916
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120461
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Iacf121483aefa71d6f24e14782fcaf22c0d5d56e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118859
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
This reverts commit ca7dab5d96e73b7b4b045e2460e0b2ee150757db.
It created tdf#143749 + the dup tdf#143865
Change-Id: Ibd68c45213491b0df7dc36f0418de307412499c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120531
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: Ib2ddc8475e2476ce377f3541caa0d48e7c57241a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118783
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
Change-Id: I483d494a11b951fd415a314fddd4823db11dcb44
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120295
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
|
|
Change-Id: If1a6f224f6ff4118617128630077a81da4c469a1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118656
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Jenkins
|
|
Change-Id: I16b21d2bec8de9e5f4e8402c838c99d73637fe99
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120290
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ib52a4cc56fafc23ced1ce4f27d2c69977471d1d2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118491
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: Ibf67c85f4af80e5420b017a1a1f89e3eac8a3118
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120219
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I188a9ce69a89958f776f696f1da2f96b82a7f6da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120217
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: If7d7462571cbee1c54965678fab790959c6e3f33
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120218
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I239befdaa05beea5200c81f23d0af861c9fc3baa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120177
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Add a test for drawing a drop like shape using polygon and polyline
drawing code-paths.
Change-Id: Ifee7b978009d66af01ccbfc92b36d1423084ed48
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118278
Tested-by: Tomaž Vajngerl <quikee@gmail.com>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
|
|
Change-Id: I049bd94eaaa771584d18859903ef75e4319494b3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119893
Tested-by: Jenkins
Reviewed-by: Roman Kuznetsov <antilibreoffice@gmail.com>
|
|
Change-Id: I036dd26b2e70c12896afcef2ce25586e8db3098c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119959
Tested-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
|
|
Change-Id: If73604d1e329f089a357021bdce2163f6f059160
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120014
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I6b3debabe2c927f5ce3962fe97d944c07153028b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120013
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
This effectively reverts commit 76be678e7478e4863a8360e26425a39afa5af4fb.
"rename ImplWindowAutoMnemonic and expose"
Change-Id: Ibeeeda68d96286e74113a7567ef12dae3a80cf48
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120012
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
because this is often on a hot path, and we can avoid the splitting and
joining of strings like this.
Change-Id: I2a24a123a64b762fd0741c45eaca3ad4bdd5580d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119884
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I95e4dc636fc16095044d01e1c1e1314f107b23b7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119850
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I1c172273e14d3bd2cd973abb38bf19be144ef210
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119849
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
The original method SvTreeListBox::StartDrag always triggers
a MouseButtonUp event and tries to initiate a Drag of a Line of
a TreeListBox (on MouseMove, btw).
This is not wanted if the last MouseButtonDown started a
ButtonActive mode and activated CaptureMouse, prepared to
trigger Action on that Button on MouseButtonUp. It leads to
unwanted/strange behaviour of Buttons/CheckBoxes when used in
TreeListBoxes.
The behaviour is also dependent on the UI implementation used
under Linux (gen/gtk3_kde5/gtk3/qt5/kf5) which are all
(unfortunately) behaving differently, but a first suggestion/
step to enhance the situation.
Found now for gen/qt5/kf5 that when on the LineEntry, but
not on the CheckBox, on MouseButtonUp the other line gets
switched. Corrected that.
Note: for gtk3_kde5/gtk3 there remains the problem that the
CheckBoxes get switched on MouseButtonDown, butt these are
generic widgets and this needs to be solved differently.
Change-Id: If4cfe894b716185293beff64fc7e482d6f6313d3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119644
Tested-by: Jenkins
Reviewed-by: Armin Le Grand <Armin.Le.Grand@me.com>
|
|
the struct is already visible in the headers, so it's not a pimpl
pattern, and it's only two pointers, so no point in allocating
separately.
The tdf#91081 bug comment I removed is no longer relevant,
this general class of problem was solved with the introduction
of VclPtr.
Change-Id: Ie5d896baf0167273f8936930dd65891919c5bf7e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119679
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Main problem is, that Qt::Popup grabs the focus and therefore
generates a focus-out event for the combobox, which then closes
the just shown popup window. The grab happens inside QWidget
private code and there is no way around it. But instead of
"faking" Qt::Tooltip, this uses Qt::Widget with additional flags.
Regression from commit 7e6fee830116823b9cd8e46d6962df4ea2bc1ea6
("Qt5 fix Qt::Popup window handling").
Change-Id: Ia1f8e33d98f7ec36cf1ebc350886121dfaadd658
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119691
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
Change-Id: I85881531f041cbd5402ac8de2a96b2c7150cbba8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117839
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119609
Tested-by: Jenkins
|
|
unify menubutton popups and toolbox dropdowns
Change-Id: I61c0c33a17d96f03d6513507bda6d5c8edbc55dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117786
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119608
Tested-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
Change-Id: I4eb49a81d12ac37f50c6595eeec6d472fdbe6d82
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117251
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119581
Tested-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
Change-Id: I58d021d3e4a3f5d4b8b805380526c1d32b782674
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117345
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119563
Tested-by: Jenkins
|
|
Change-Id: I53687f67042490c19d1056c74619d43566bf1289
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117132
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119562
Tested-by: Jenkins
|
|
Change-Id: I79b753e070183daa809b23dbb7d3052000ebecd4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116995
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119552
Tested-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
Change-Id: I5f5f0ceb58925874c6878760a99179a4936198fa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116991
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119543
Tested-by: Jenkins
|
|
Change-Id: I863022b0b1efc741626b0ba4a8a6183c169eaa85
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116891
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119523
Tested-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
Change-Id: I43b10e2314b81dc490714ad9fb809c1324fe17c8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116890
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119512
Tested-by: Jenkins
|
|
Change-Id: Ia5a936c44e5d83fd4626098bcc027b0382523231
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116098
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119511
Tested-by: Jenkins
|
|
Change-Id: I9516538cac8d2e42e0d029feaf84c3bc3a80151a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119499
Tested-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
Change-Id: I30f40c3906c38d29bab7aeef0a9562bd96934a72
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119329
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
no need to allocate ImplEntryList separately
Change-Id: I276e92895a00ebccff60f5067567a0ed066d4e66
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119328
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ia3c0d6624ccf2d3c6189064ab7c8b07dc0734c5b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119330
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ie37503b5964606a2b3e5b305c949f570616dc1cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117070
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119243
Tested-by: Jenkins
|
|
Change-Id: I590595adbd89fa3d4833de2743b71168448d4c2c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119283
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
I originally omitted this in the Qt implementation, as I couldn't
find any non-working use case.
The implementation got a bit hacky, because Qt doesn't have a non-
native way to identify different L/R modifier keys, so we must
process the X11/xkb keycode ourself, which obviously won't work on
Wayland... but most times this is not relevant, so the default
modifier notification may be good enough.
P.S. it's basically the same code then X11 and Gtk...
Change-Id: I37fbfde4a33a966b6017f3e0c280e2c7ea91e4db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119235
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
This bug was "unveiled" by commit
963f252cd1ea9c268a6ced68a3454b10cbee1a89 ("Qt5/KF5 get rid
of unneeded own grahics handling"). I'm not sure why I ever came
up with that code. Eventually we would need a paint event, but at
least changing the theme correctly updated LO UI, so that seems
to be handled correctly.
Change-Id: I528b551180be184427eeeeeb2977e2b7da073ce6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119237
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
|
|
This fixes non working comboboxes in the sidebar in online.
Where we simulate picking from the list.
Change-Id: Ie78e461b2bce7fa0ccd68731e76d0c721242b98e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116097
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119155
Tested-by: Jenkins
|
|
Change-Id: I98c92b336fbf76f3dc94265a408c9b3b46fcea08
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115453
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119014
Tested-by: Jenkins
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
|
|
it is never instantiated by itself. And this also means it now only has
one OWeakObject base class
Change-Id: I3074130539ab023fec5537e0e7794e1c2db7e6ee
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118835
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
I realized that Impress does not have the same screen
size as PowerPoint. tdf#142286, tdf#142288
When I load a PowerPoint slide into Impress,
it was a custom size, but when I add this screen size,
it shows up with the screen size name.
And when I load a slide with the added screen size
into PowerPoint, it also shows up with the PowerPoint
preset screen size name and not the custom size.
The screen sizes I have added are as follows
* Widescreen (Current PowerPoint default)
* On-scren Show (4:3)
* On-scren Show (16:9) (Google slide defualt)
* On-scren Show (16:10)
Change-Id: I33e2bbc36b2c92ecbb85c610c864f735bf540d03
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116374
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Remove gtk_widget_set_can_target on the overlay so that when we put
native gtk widgets into the overlay they get events and can be
interacted with.
The DrawingArea is 'underneath' the overlay. So just move the mouse
event handlers to the overlay itself so we can still appear to interact
with the DrawingArea.
Change-Id: I43056c5ee6228af85a194c1baa9f83c58b3e389d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118680
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
Change-Id: I729584b71cf8d0a507b97fce96add66fb22508a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118513
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
there isn't a dedicated GtkToolbar anymore, just a generic GtkBox with
GtkButton children
Change-Id: I6d67c9ffde13dc4f787804466ece498f973ba0b7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118467
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|