summaryrefslogtreecommitdiff
path: root/vcl
AgeCommit message (Collapse)AuthorFilesLines
2016-03-10Avoid reserved identifier, remove unnecessary typedefStephan Bergmann2-2/+2
Change-Id: I2dfce92ef98ca18ac0fe2c415240216228b4ee0a
2016-03-10Avoid reserved identifierStephan Bergmann1-4/+4
Change-Id: I554fde45000114dd19f117d93ef5c7a780231594
2016-03-10Avoid reserved identifierStephan Bergmann2-3/+3
Change-Id: Ie62bbf64a9cdb74725fd48a8f8dcc1ab76d97219
2016-03-10Remove unnecessary typedef (that introduced a reserved identifier)Stephan Bergmann5-16/+16
Change-Id: I6c0aaac10b2244271f3cdf45f4eceb6d685b213c
2016-03-10Avoid reserved identifiersStephan Bergmann1-4/+4
Change-Id: I751e75e0b63e95cc92be7b61a77ed21eeb52bc1f
2016-03-10Related: tdf#98419 gtk3: reimplement passing window move control to wm...Caolán McNamara7-37/+39
for wayland Dragging toolbars around to move them (starting with an undocked toolbar, not the moving outline fake thing) doesn't work under wayland (as far as I can see) without using gtk_window_begin_move_drag i.e. gtk_window_move doesn't work. But this is supposed to be used from the initial mouse click (while it works under wayland from a move, it doesn't work under X from a move) so rework the last attempt to occur right at the initial click to drag. Change-Id: I612f188b3e8482307bc816f5aa775530e6092eda
2016-03-10Follow the coding style of surrounding codeTor Lillqvist1-6/+12
Change-Id: I80a2d935d92c8330f3815b6e79ccc58bc39541b0
2016-03-10gtk3: set decoration bits etc before realizeCaolán McNamara1-9/+3
cause wayland is sticking a title bar into the toolbars otherwise, sigh Change-Id: Id012e9508cc0dfafbda344974a96d8a038c6c9f4
2016-03-10Add comment wondering what the code thinks it is doingTor Lillqvist1-0/+1
Change-Id: Icb135c12b2bc159e0542994bb496b45bf1fdb9e2
2016-03-10Factor out the calls to FillRect in the DrawTextImpl() implementationsTor Lillqvist1-26/+5
They were all in fact doing exactly the same, so do it at the call site instead. Change-Id: Id61cf9f5881411ddb7df5c3fd98c83252637e4d7
2016-03-10Extract Directories from BootstrapFixtureBaseStephan Bergmann3-12/+12
(as some tests derive from the latter only for the Directories part, not for the setUp/tearDown overrides: those tests will be cleaned up next) Change-Id: Ib6b78eea868b8bc21d4cc6e8fd9e1d025deca05f Reviewed-on: https://gerrit.libreoffice.org/23078 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-10loplugin:rangedforcopyStephan Bergmann1-1/+1
Change-Id: Iba02e6a142b52d00c824eeddb1970546f7aba3e9
2016-03-10loplugin:staticmethodsStephan Bergmann1-2/+2
Change-Id: Ieaebacd43fa27b91b1ab3e6704bb591488954492
2016-03-09Related: tdf#98419 use gtk_window_begin_move_drag bodge for wayland onlyCaolán McNamara1-1/+10
Change-Id: Ica19aef9b94e0c11e014f48b7801ecb0c110c44b
2016-03-09-Werror,-Wmismatched-new-deleteStephan Bergmann1-1/+1
Change-Id: I0357af933795aad10db8ab23ae9ef0373e73287f
2016-03-09cups_dest_s::instance can be nullStephan Bergmann1-1/+1
Change-Id: I7015bd2e13d5bf08cb9181dc9470372f0b3002c0
2016-03-09Make the new D2D Graphite rendering work also in the OpenGL caseTor Lillqvist1-5/+8
It used to render just black boxes. The change in DrawTextImpl() semantics from 61085083e4a5060ba7e2135818264d63c6da13c2 was not properly implemented in the new Graphite code. The return value is not some kind of "success" indicator, but tells the caller whether to continue the loop at that level. We do need to call FillRect() to fill the requested rectangle of the HDC with white. On the other hand, the call to Clear() is not needed and in fact makes no text show up. (I now see that that same code snippet that calls FillRect() is used in all the DrawTextImpl() implementations, so it should be factored out to the call site in WinLayout::DrawText().) Change-Id: If0533ea1edf065b06ae888c6e57c026f447bcf78
2016-03-09tdf#96685 - ensure FindFocus a11y context is valid EditableTextJustin Luth2-2/+12
signalIMDelete and Retrieve Surrounding Text search for the accessible context that has the focus. When a context with the FOCUSED state was found it was automatically returned, assuming that the reference was valid. In Draw tables, especially when using the arrow keys to move between cells, that often was not true. So, instead of returning a broken reference, keep searching through the children until a valid AccessibleEditableText item that is marked as FOCUSED is found. Change-Id: I71e3e9bfda56d1dfbdbd93945882560a39e40714 Reviewed-on: https://gerrit.libreoffice.org/22263 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-03-09vcldemo: improved text rendering test.Michael Meeks2-27/+263
Merge in some interesting font / unicode combinations from bugs. Change-Id: I2c89cf505a7850fcc482826328e1cdb8e37508aa Reviewed-on: https://gerrit.libreoffice.org/23056 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2016-03-09Resolves: rhbz#1315385 use preferred size if widget supports itCaolán McNamara1-1/+1
when deciding if a popup needs to be placed up or down to stay visible on screen Change-Id: I718e0ee4a79152e919ac95841e15d4b53764ac78
2016-03-09fix for older gtkCaolán McNamara1-1/+1
Change-Id: I1a3286b2f967f50fbaef7e3726e48ae82fd145aa
2016-03-09gtk3: try and keep sync with yesterdays upstream scrollbar changesCaolán McNamara2-5/+32
Change-Id: If3942704602b82bc99ce49a230930ef59cc862da
2016-03-09Add missing includesDavid Ostrovsky1-0/+1
These include were needed to compile on Windows, MSVC 14.0 with clang-cl. Change-Id: I4ca5cec8314920e90fcca6fa69ec4df87d680f29 Reviewed-on: https://gerrit.libreoffice.org/23044 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-09DirectWrite rendering for Graphite to work around ExTextOut bugTim Eves1-20/+352
ExTextOut has a bug on Windows 7 and above where it incorrectly positions certain diacritics, using DirectWrite and Direct2D fixes this. Implemented on-demand loading of the DLL so the old ExTextOut based renderer will be used when drwite and d2d1 cannot be found allowing this work on Windows XP (where this bug doesn't seem to occur) Change-Id: I767d62c8188511e745373b61ba51e7e2745f7b8b Reviewed-on: https://gerrit.libreoffice.org/23020 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Reviewed-by: Martin Hosken <martin_hosken@sil.org>
2016-03-08tdf#98052 - even polygons were not drawn on the alpha deviceMarco Cecchetti1-0/+2
Change-Id: I5725ec23869b295c8021bef0330ee6f69f206351 Reviewed-on: https://gerrit.libreoffice.org/23024 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2016-03-08tdf#97966 Drop static keywords from two functions inside an unnamed namespaceWastack1-2/+2
Change-Id: I4fca1e813eccfeb5185e7a50aa301e7ad1ee61b5 Reviewed-on: https://gerrit.libreoffice.org/23015 Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-03-08convert custom bibliography toolbar to .ui formatCaolán McNamara1-0/+25
Change-Id: I70eb0e2ff96879d1168b241852a0079f540b0319
2016-03-08loplugin:constantparams in vclNoel Grandin20-72/+62
Change-Id: I2114436f4bef3ac71a3035a206186cefaf88bca1 Reviewed-on: https://gerrit.libreoffice.org/23023 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-08building against 110 different operation system variants is a painCaolán McNamara1-0/+2
Change-Id: Icbb9e2590f1b8b18c021b978baf7f79c838c4c9c
2016-03-08Bin unneeded inclusions of dbggui.hxxTor Lillqvist3-3/+0
Change-Id: I5cac6fb0278e3952e2538f06188ed510644fcaa0
2016-03-08Bin a pointless level of macro abstractionTor Lillqvist2-10/+6
Change-Id: I343844aca956a1ce05c733f60a28d51115574ef8
2016-03-07gtk3: popover positions come pre-computedCaolán McNamara1-15/+30
i.e. the position provided is the location where the popover is to be drawn, not the bounds of the thing to be pointed to, so adapt them so gtk will point to the desired place. e.g. the slide names/numbers in the presentation slide view sidebar Change-Id: I8c87d5dba32e27f9e627b3282f34d87a8ee460ca
2016-03-07Resolves: tdf#98425 gtk3: grab the SolarMutex when setting the clipboardCaolán McNamara2-1/+3
Change-Id: Ic439affbd69532534f5caf82a743786d5975939e
2016-03-07loplugin:unuseddefaultparamsNoel Grandin8-36/+21
Change-Id: Ia414f7845425ef73859ed04853378e96cc738795 Reviewed-on: https://gerrit.libreoffice.org/22971 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-07tdf#96099 Reduce number of trivial typedefsChirag Manwani2-8/+6
Change-Id: I39e9fcfdf2203239ac56d1c8195ca7ac07054817 Reviewed-on: https://gerrit.libreoffice.org/22898 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-07improve defaultparams lopluginNoel Grandin4-27/+18
to catch calling params with defaults like "= OUSString()" Change-Id: Iad060e318ed492c22f8be44e326174fe6d28fff9 Reviewed-on: https://gerrit.libreoffice.org/22932 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-07loplugin:unuseddefaultparam in variousNoel Grandin2-10/+7
Change-Id: I03d7381aad055cbe9bd905e4082586073f4112e0 Reviewed-on: https://gerrit.libreoffice.org/22900 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-07opengl: fix loading shader with preamble that uses #versionTomaž Vajngerl1-2/+2
The preamble was inserted into a false position so the shader could was constructed incorrectly and would fail to compile. Change-Id: I4c51adde9014a326bbe38a5d2d17dd0047e33195
2016-03-07opengl: fix inv.scale values are always integer - cid#1352479Tomaž Vajngerl1-2/+2
Force cast to double to avoid integer division - which gives a wrong inverse scale value. Change-Id: I0135e44ef07f3915619f9dfead9aadf50fc03685
2016-03-04tdf#79679 vcl: dashed lines show as solid lines when importing EMF filesChris Sherlock4-46/+27
Issue is a regression in commit 09c722873b2d378d2d155f5f1dd7d8f3fb2012e9. (EMF/WMF: fix rendering of pen styles (dash, dot, dashdot, dashdotdot). I've looked at how the latest version of Word on the Mac works, and it turns out that the spacings for the PenStyle enumerations in the LogPen objects for all the create pen EMF records are as follows: * PS_DOT - ■ □ ■ □ ■ □ ■ □ ■ □ ■ * PS_DASHDOT - ■ ■ ■ □ ■ □ ■ ■ ■ □ ■ * PS_DASHDOTDOT - ■ ■ ■ □ ■ □ ■ □ ■ ■ ■ (where ■ is the actual filled in area, and □ is the space between the filled in areas) In other words, each dash fills in the space of three dots, and there is the one dot worth of empty space between the dashes and dots. Each "dot" has a width and height equal to the width specified in the pen. So basically, we seem to be arbitrarily setting the dot, dash and distance lengths arbitrarily, which were reasonable guesses but tended to produce very odd lines at different zoom levels. Change-Id: Ie8b5fa396e4fb0f480cb3594c8129a59f472c1b8 Reviewed-on: https://gerrit.libreoffice.org/22886 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-03-04tdf#98052 - polypolygons were not drawn on the alpha deviceMarco Cecchetti1-0/+2
In OutputDevice::DrawPolyPolygon when b2dpolygon are used for drawing the source polygon is not drawn on the alpha device. Change-Id: I54f4e5a13469d9844866cea61b074420219b836d Reviewed-on: https://gerrit.libreoffice.org/22892 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2016-03-04Don't build ui-previewer in the App Store caseAndras Timar1-2/+3
Change-Id: I25ca4b8672702095cc04723bc9c6bdf01a06382f
2016-03-04-Werror,-Wunused-macrosStephan Bergmann1-1/+0
Change-Id: Ib333ad2880e0a1cbfed2f6a919ca2b92f42dde45
2016-03-04-Werror,-Wunused-macrosStephan Bergmann1-3/+0
Change-Id: I0ac1ba124ba452deeeeaa473ccbbf865e490c447
2016-03-04-Werror,-WshadowStephan Bergmann1-1/+1
Change-Id: I1e26a805ac96f1c57d2f7c795b598c4eaa8d2a3e
2016-03-04-Werror,-WshadowStephan Bergmann1-2/+2
Change-Id: I4893f538911449953fadf4cf10f6adb819bc023f
2016-03-04-Werror,-WshadowStephan Bergmann1-4/+4
Where wglChoosePixelFormatARB is defined as a macro expanding to __wglewChoosePixelFormatARB (as WGLEW_GET_FUN is just expanding to its argument), and __wglewChoosePixelFormatARB is declared in global scope in workdir/UnpackedTarball/glew/include/GL/wglew.h. itself) in workdir/UnpackedTarball/glew/include\GL/wglew.h Change-Id: I0c4d09e9112c2233d25a262ea1f2b35bdf49645c
2016-03-04loplugin:unuseddefaultparams in vcl and xmloffNoel Grandin14-73/+45
and teach the plugin about code that takes the address of a function Change-Id: Ia9d5afef44520aca236659e8176f1e27135ef4fc Reviewed-on: https://gerrit.libreoffice.org/22861 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-03vcl: type inference is beyond MSVC's limited abilitiesMichael Stahl1-2/+2
Change-Id: I3e6c84c1f49e9523e881ce12dbb2f7e0d1df956e
2016-03-03coverity#1355126 Logically dead codeCaolán McNamara1-8/+5
maybe we should be using more precision, but we haven't been in the past Change-Id: Icf1e1e95099c783435e04f60d8ade755a8d2d4d5