summaryrefslogtreecommitdiff
path: root/libreofficekit/qa
AgeCommit message (Collapse)Author
2017-09-22Fix gtktiledviewer crash: this should be a null terminated listTamás Zolnai
Change-Id: I06e0923980b98b37b06ab45d8db68424b01d4f71 Reviewed-on: https://gerrit.libreoffice.org/42645 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-09-11clang-tidy modernize-use-emplace in hwpfilter..lotuswordproNoel Grandin
Change-Id: Ic11634ca28396fd156390c511087bae03bd5fb70 Reviewed-on: https://gerrit.libreoffice.org/42156 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-08-17implementing callback for ruler invalidationAditya Dewan
adding commands to fetch and changee ruler state '.uno:RulerState' and '.uno:RulerStateChange' Change-Id: I66107039a7ae5893691feb45c8ab2e4aa476ea76 Reviewed-on: https://gerrit.libreoffice.org/40727 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: pranavk <pranavk@collabora.co.uk>
2017-07-18Fix typosAndrea Gelmini
Change-Id: If6023dfa1d90f79185197622a738373a189ea6af Reviewed-on: https://gerrit.libreoffice.org/40118 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2017-07-18gtv: G_OPTION_FLAG_NONE is available since 2.42 onlyPranav Kant
Change-Id: Ic3aacb79a3e828e59fede437213f2b7298e49bc1
2017-07-17Fix typosAndrea Gelmini
Change-Id: I0fec2abc1bed9c0cfcd78d1b0f6daebc335831be Reviewed-on: https://gerrit.libreoffice.org/39982 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com>
2017-07-14gtktiledviewer: Typo; these should be classesPranav Kant
Change-Id: Iba449fc66423959340c7967c64bc422a28fc75dd
2017-07-14Modernize gtktiledviewer; use GApplicationPranav Kant
Put all the UI content in UI XML file. Unfortunately, lots of boilerplate code because G_DECLARE_* macros are available only since glib 2.44 Change-Id: Idc74ba8565d482c28abd00b6f6f75646ab3d40b9 Reviewed-on: https://gerrit.libreoffice.org/39913 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: pranavk <pranavk@collabora.co.uk>
2017-06-29loplugin:oncevar variousNoel Grandin
Change-Id: I8b82d46d4688b1a59d6fe1b05da7d5c8dfc13ca6 Reviewed-on: https://gerrit.libreoffice.org/38766 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-06-15use more SAL_N_ELEMENTS part 3Noel Grandin
Change-Id: I82e366fefd2e31928b99840fe76649cc3521e623 Reviewed-on: https://gerrit.libreoffice.org/38789 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-05-12coverity#1351757 Uninitialized pointer fieldCaolán McNamara
Change-Id: I99211e2dc016fb97b6744799b35e05acd7470464
2017-05-11lok: sc: notify cell cursor position to address control in clientMarco Cecchetti
A new callback has been introduced for notifying the client: LOK_CALLBACK_CELL_ADDRESS Change-Id: I40b38a3cb8fb658c3f00332d56cfcbaf98e13771 Reviewed-on: https://gerrit.libreoffice.org/37357 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-04-12CppunitTest_libreofficekit_tiledrendering: disable occasionally failing testMiklos Vajna
When it failed, it always failed like this: libreofficekit/qa/unit/tiledrendering.cxx:311:TiledRenderingTest::runAllTests equality assertion failed - Expected: 3 - Actual : It's not clear yet what triggers the failure, sounds like some kind of timing. Change-Id: Ib5f44523f9a29c892aeda9eb881f452b57adb8a8 Reviewed-on: https://gerrit.libreoffice.org/36445 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2017-04-11CppunitTest_libreofficekit_tiledrendering: add more processEventsToIdle callsMiklos Vajna
With this the test passes for me 10 times in a row, while previously it failed from time to time, hopefully this fixes the false negatives. Change-Id: I233276ddfe4e9d8c86557f7f1c29d997f2fb51f6 Reviewed-on: https://gerrit.libreoffice.org/36420 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2017-04-06lokit: compile-test a plain C versionThorsten Behrens
Change-Id: Ibe2108984df8f4fef9ed681941d517ca9ad9479c Reviewed-on: https://gerrit.libreoffice.org/36177 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2017-03-28Use symbolic names for keys from css::awt::KeyTor Lillqvist
Change-Id: Iee2f4540cddeb05fcf0ae2ecadf7de8fbb4e3a0d
2017-03-24lok: Unit test for setting of the language during load of the document.Jan Holesovsky
Change-Id: Idf4d3ba6b55be1f885f9d8fc89157e7e498d4e42
2017-03-21lok: Fix crash in paintPartTile() when the current view was destroyed.Jan Holesovsky
Change-Id: I59b71ee6815cbcfa4c8b5f68ae6dc9299856d49e Reviewed-on: https://gerrit.libreoffice.org/35494 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jan Holesovsky <kendy@collabora.com>
2017-03-06libreofficekit: GTK 3.22 deprecates gtk_show_uriMichael Stahl
Avoid -Werror=deprecated-declaration and use gtk_show_uri_on_window instead, which needs a GtkWindow. Change-Id: Ia77978348809c8d88cb2823a1549cbdeffcc6c97 Reviewed-on: https://gerrit.libreoffice.org/34926 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-02-23Better make the zero-initialization more explicitStephan Bergmann
...if initializing these members is necessary at all; a bit hard to tell from the code. Change-Id: I384fbeedb3a5b0690a80cabdd13ca13d07e8d627
2017-02-20gtktiledviewer: Different reply uno command for impressPranav Kant
Change-Id: I9dcd32a7e64198ca4772f263ed89a458e298c53b
2017-02-20gtktiledviewer: Id param is string nowPranav Kant
It was changed in ce96c5957c38384f8911d6307605c502eca0c644 Change-Id: I4cc1f90d85e31044306ab36eaeac9d4424d965c0
2017-02-20gtktiledviewer: Adjust for different delete uno cmd for impressPranav Kant
Change-Id: I7fa82096298a8a7c9f586a1d3c06aa5852e6842e
2017-02-20gtktiledviewer: enable comment sidebar for all doctypesPranav Kant
Impress was the only doctype remaining which didn't have comments api yet, but now it does Change-Id: I4e287301e879dff1c29c26880305505466ceaba4
2017-02-16gtktiledviewer: Different UNO command for calcPranav Kant
Change-Id: I229451bac7cf4c223efd3fa18c177d650d3f9954 Reviewed-on: https://gerrit.libreoffice.org/34327 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: pranavk <pranavk@collabora.co.uk>
2017-02-16gtktiledviewer: This is a better place to change doc-specific controlsPranav Kant
Change-Id: Id8fbebe2f6e339120741ed3744b18f50941c995e
2017-02-16gtktiledviewer: Kill CommentsSidebar's docConfigurePranav Kant
Move this code to initWindow too Change-Id: I057d9b8f60b9da2c20b28901ab958f9fc2095df3
2017-02-16gtktiledviewer: Kill signalSize; move its code to initWindowPranav Kant
Change-Id: I8aa0a00768e074b016ea20680c67884f19fe26df
2017-02-16gtktiledviewer: Factor out per view code to separate functionPranav Kant
This code (populating part selector, part mode selector etc.) is something that needs to be executed for each view, not just the view that makes open_document call. Put all such code in initWindow function from now. Change-Id: Ibf9b12adb156258aa3cfdf959fffb541a88da529
2017-02-16gtktiledviewer: Store author name of each viewPranav Kant
Change-Id: I9a5be9050f3bedd14b6b5602b51bf40da0a11175
2017-02-16gtktiledviewer: comment id can be a string too nowPranav Kant
Change-Id: Ia94041884b7fde5f6c104a41c77c6a3aa96b7be1
2017-02-13Fix typosAndrea Gelmini
Change-Id: Iaf6bbe81f519d49b82f624282d786c8cc18d641c Reviewed-on: https://gerrit.libreoffice.org/34199 Reviewed-by: Aleksandar Stefanović <theonewithideas@gmail.com> Tested-by: Aleksandar Stefanović <theonewithideas@gmail.com>
2017-01-31gtktiledviewer: gtk version checksPranav Kant
There is no major harm in excluding these lines of code on platforms with older gtk. Change-Id: Ica3f1e2b4f60f0bab55254632663ace7b4c61d4d
2017-01-30gtktiledviewer: Handle comment callback with Modify flagPranav Kant
Change-Id: I8333876cc4b540c385e316ce8dcbc5f7bd1f7902
2017-01-30gtktiledviewer: Unused variablePranav Kant
Change-Id: I1bf8833041cf0e0a42aca1b6d851f6f4ca28d68f
2017-01-30gtktiledviewer: Mark reply comments with lightgreen backgroundPranav Kant
Change-Id: Ic6623ebc05f1a09d5fc1b6bae5fab149d98724ab
2017-01-30gtktiledviewer: Can edit comment from comment sidebar nowPranav Kant
TODO: Ideal would be to replace these buttons with a popover menu containing all the actions that can be taken on a comment Change-Id: Iec19e6c4a934394f84bcf4bb20cc14f207ef95be
2017-01-28coverity#1351757 Uninitialized pointer fieldCaolán McNamara
Change-Id: Ieeae560fc6b6c82ef92ad5331cc6cddacb893a19
2017-01-27gtktiledviewer: Can reply, delete comments from sidebarPranav Kant
... and put the sidebar in a scrolled window Change-Id: I57e982c849ed0c4dda4e8f5fdbcb0b85e46da36c
2017-01-27gtktiledviewer: factor out user prompt entry dialogPranav Kant
Change-Id: I282390f7319de2e24564e0122e8c0081235a0ef5
2017-01-27lokdocview: 'comment' signal for comment callbacksPranav Kant
Change-Id: I82040893added83ff13395db3917b230ef6b01d5
2017-01-27gtktiledviewer: Comment dialog when tiled annotations are offPranav Kant
Change-Id: I6762baac6c387d968dc6ee94d3494907c817214c
2017-01-27lokdocview: Separate gobject property for turning off commentsPranav Kant
Pass --enable-tiled-annotations to gtktiledviewer on command line to turn comments on again, by default they are turned off now. Change-Id: I40e93ab683ec9538e088ae9aa83cda2eaaf9939e Reviewed-on: https://gerrit.libreoffice.org/33622 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-01-27gtktiledviewer: Comments sidebarPranav Kant
Change-Id: Icf5bb78859668bfb5c43efc3e5fa9afce8458f41 Reviewed-on: https://gerrit.libreoffice.org/33621 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-01-27CppunitTest_libreofficekit_tiledrendering: fix cppunitassertequals warningsMiklos Vajna
Change-Id: I0fce00d4c794918e5b14b2c4f3f9f36231f28718 Reviewed-on: https://gerrit.libreoffice.org/33604 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2017-01-23gtktiledviewer: Make edit mode the default onePranav Kant
This is a testing tool, most of the time developers want to do something with the document to test some functionality, lets keep edit mode on by default and save them a second. Change-Id: I61bbd2b85054b7ebdd14e69ca170674e260368e5
2017-01-23lok: Listen for state change events for cell alignment commandsPranav Kant
We used to use HorizontalAlignment uno command for cell alignment both in online and gtktiledviewer. To make things simpler, lets use separate cell alignment uno commands for spreadsheets like we have in writer and impres. Change-Id: I2a959d1892f6b02db7a9ce51d55309216e4b3c6c Reviewed-on: https://gerrit.libreoffice.org/33444 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-01-06gtktiledviewer: Lets avoid using manual buffer sizesPranav Kant
Change-Id: I76e70ccb5b1f40193a1eda1cbca19b3444168dae
2017-01-06gtktiledviewer: Override alignment buttons uno commands for calcPranav Kant
Use .uno:HorizontalAlignment instead of .uno:XXXPara for calc Change-Id: I5af4fe62dc3075af4ba779499d062e6d91f21ecf Reviewed-on: https://gerrit.libreoffice.org/32773 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>
2017-01-06gtktiledviewer: Support for posting arguments to uno commandsPranav Kant
Change-Id: I5db217b525d0f976ef6a7840e370f8f6a21d60cb Reviewed-on: https://gerrit.libreoffice.org/32772 Reviewed-by: pranavk <pranavk@collabora.co.uk> Tested-by: pranavk <pranavk@collabora.co.uk>