[Libreoffice-ux-advise] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

Heiko Tietze  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #3 from Heiko Tietze  ---
How about an (expert) option here? 

The general approach would be something like Browsers offer where you can zoom
into the content. We could do the same for font size and maybe kind reintroduce
bug 101646 (UI scaling), which receives a lot of interest, but per control vs.
per application. Though, this sounds like an overkill to me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-bugs] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

Heiko Tietze  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #3 from Heiko Tietze  ---
How about an (expert) option here? 

The general approach would be something like Browsers offer where you can zoom
into the content. We could do the same for font size and maybe kind reintroduce
bug 101646 (UI scaling), which receives a lot of interest, but per control vs.
per application. Though, this sounds like an overkill to me.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: sd/qa

2019-08-22 Thread Andrea Gelmini (via logerrit)
 0 files changed

New commits:
commit dd7d90055545cb20aa5c12c0be44cccaefdeac47
Author: Andrea Gelmini 
AuthorDate: Thu Aug 22 22:23:26 2019 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 23 07:24:45 2019 +0200

Removed executable permission on file

Change-Id: Icc51eaef76e48e3acdbc9602720a3e12902a
Reviewed-on: https://gerrit.libreoffice.org/77981
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/sd/qa/unit/data/odp/testLegacyShapeAutofit.odp 
b/sd/qa/unit/data/odp/testLegacyShapeAutofit.odp
old mode 100755
new mode 100644
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: vbahelper/source

2019-08-22 Thread Andrea Gelmini (via logerrit)
 vbahelper/source/vbahelper/vbaapplicationbase.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b47f733178335779283152f9a24381644cf9c5a6
Author: Andrea Gelmini 
AuthorDate: Thu Aug 22 22:23:15 2019 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 23 07:24:19 2019 +0200

Fix typo

Change-Id: I94efc4ce7e197fb8dc68840939ed75393983be1c
Reviewed-on: https://gerrit.libreoffice.org/77980
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/vbahelper/source/vbahelper/vbaapplicationbase.cxx 
b/vbahelper/source/vbahelper/vbaapplicationbase.cxx
index 87c82e472d7c..94c3419f6e2a 100644
--- a/vbahelper/source/vbahelper/vbaapplicationbase.cxx
+++ b/vbahelper/source/vbahelper/vbaapplicationbase.cxx
@@ -472,7 +472,7 @@ void VbaApplicationBase::Quit()
 {
 // This is the case of a call from an (OLE) Automation client.
 
-// When an Automation client itself asks the proces to quit, it should 
obey it.
+// When an Automation client itself asks the process to quit, it 
should obey it.
 AsyncQuitHandler::instance().SetForceQuit();
 
 // TODO: Probably we should just close any document windows open by 
the "application"
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: sw/inc sw/qa sw/source

2019-08-22 Thread Justin Luth (via logerrit)
 sw/inc/doc.hxx  |3 
 sw/inc/fesh.hxx |2 
 sw/qa/extras/uiwriter/data2/tdf126784_distributeSelectedColumns.odt |binary
 sw/qa/extras/uiwriter/uiwriter2.cxx |   29 
+
 sw/source/core/docnode/ndtbl1.cxx   |   32 
+-
 sw/source/core/frmedt/fetab.cxx |4 -
 sw/source/uibase/shells/tabsh.cxx   |3 
 7 files changed, 50 insertions(+), 23 deletions(-)

New commits:
commit d0e268ca18ec0d907980ca608c97fe8f727d20df
Author: Justin Luth 
AuthorDate: Wed Aug 21 19:41:17 2019 +0300
Commit: Justin Luth 
CommitDate: Fri Aug 23 07:18:24 2019 +0200

tdf#126784 sw: only distribute across selected columns

partial revert of LO6.2 commit ab18c17d70e1dcf5cf9db38256d35e6af479373e
because MERGED cells had a very unexpected effect on the
columns that were altered. Unselected columns are INCLUDED
in the calculation IF they extend over the selected columns
when SwTableSearchType::Col uses the entire table column.

The last table in the unit test demonstrates why the regressive behaviour
was included. A user can make an ugly layout if they don't choose a
good set of columns.

However, the merged cell really changes the way the calculation
works, and since giving power to the user is ultimately the best,
just revert the portion that tries to save the user from themself.

I struggled then and now to get a good enough understanding
to improve the documentation - which really is needed
since reading it didn't help me a lot.
However, since my understanding was and is obviously flawed,
I'll just revert back to the original wording.

Change-Id: Icf8f01f095652e2a023e0398852f43aa6b90332f
Reviewed-on: https://gerrit.libreoffice.org/77976
Tested-by: Jenkins
Reviewed-by: Justin Luth 

diff --git a/sw/inc/doc.hxx b/sw/inc/doc.hxx
index b5a5d3bb1139..af2e97aa9985 100644
--- a/sw/inc/doc.hxx
+++ b/sw/inc/doc.hxx
@@ -1498,8 +1498,7 @@ public:
 /// Adjusts selected cell widths in such a way, that their content does 
not need to be wrapped (if possible).
 /// bBalance evenly re-distributes the available space regardless of 
content or wrapping.
 /// bNoShrink keeps table size the same by distributing excess space 
proportionately.
-/// bColumnWidth tests the entire column for content width, not just 
selected cells.
-void AdjustCellWidth( const SwCursor& rCursor, const bool bBalance, const 
bool bNoShrink, const bool bColumnWidth );
+void AdjustCellWidth( const SwCursor& rCursor, const bool bBalance, const 
bool bNoShrink );
 
 SwChainRet Chainable( const SwFrameFormat , const SwFrameFormat 
 );
 SwChainRet Chain( SwFrameFormat , const SwFrameFormat  );
diff --git a/sw/inc/fesh.hxx b/sw/inc/fesh.hxx
index 25bf062c1e82..6f6a5e860802 100644
--- a/sw/inc/fesh.hxx
+++ b/sw/inc/fesh.hxx
@@ -710,7 +710,7 @@ public:
 bool IsInRepeatedHeadline() const { return CheckHeadline( true ); }
 bool IsInHeadline() const { return CheckHeadline( false ); }
 
-void AdjustCellWidth( bool bBalance, const bool bNoShrink, const bool 
bColumnWidth );
+void AdjustCellWidth( const bool bBalance, const bool bNoShrink );
 
 /// Not allowed if only empty cells are selected.
 bool IsAdjustCellWidthAllowed( bool bBalance = false ) const;
diff --git 
a/sw/qa/extras/uiwriter/data2/tdf126784_distributeSelectedColumns.odt 
b/sw/qa/extras/uiwriter/data2/tdf126784_distributeSelectedColumns.odt
new file mode 100644
index ..90c23c56a32d
Binary files /dev/null and 
b/sw/qa/extras/uiwriter/data2/tdf126784_distributeSelectedColumns.odt differ
diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx 
b/sw/qa/extras/uiwriter/uiwriter2.cxx
index 0d35c0cec77f..eb85d96d25fe 100644
--- a/sw/qa/extras/uiwriter/uiwriter2.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter2.cxx
@@ -16,6 +16,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -627,6 +628,34 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, 
testTdf64242_optimizeTable)
 CPPUNIT_ASSERT_EQUAL_MESSAGE("Row set to auto-height", double(0), 
minimalRowHeight);
 }
 
+CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf126784_distributeSelectedColumns)
+{
+SwDoc* pDoc = createDoc("tdf126784_distributeSelectedColumns.odt");
+SwWrtShell* pWrtShell = pDoc->GetDocShell()->GetWrtShell();
+
+uno::Reference xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Reference xModel(mxComponent, uno::UNO_QUERY);
+uno::Reference 
xTables(xTablesSupplier->getTextTables(),
+uno::UNO_QUERY);
+uno::Reference xTextTable(xTables->getByIndex(0), 
uno::UNO_QUERY);
+uno::Reference xTableRows = xTextTable->getRows();
+
+auto aSeq = 

[Libreoffice-bugs] [Bug 127051] FILEOPEN: XLSX: Large White Square objects several levels deep appear in the upper left corner of some sheets in the workbook

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127051

tom1willi...@yandex.com changed:

   What|Removed |Added

 Status|NEEDINFO|UNCONFIRMED
 Ever confirmed|1   |0

--- Comment #7 from tom1willi...@yandex.com ---
This document has been around for many years and gets updated twice per year by
multiple people. Most of the company now uses MSO 365 which updates itself
automatically. It is safe to assume that at some time in its lifespan this
document was edited by MSO 2010, 2015, 2016 and 365 on both Mac and Windows. 

But the main point is my LO 6.2.6.2 put in these empty frames and once they
were in the document they stay in the document no matter what software opens
it. I am guessing that when I edit certain sheets and hit FILESAVE it puts in
an empty frame. If I saved 5 times after different edits on a given sheet there
may be 5 empty frames on top of one another on that sheet the next time the
document is opened. At least that is my best guess as to what might be
happening. Thank you for your help!!

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[ANN] orcus 0.15.1 on master

2019-08-22 Thread Kohei Yoshida
Hi there,

Just a quick announcement that I have updated the version of orcus
library on the master branch from 0.15.0 to 0.15.1.

This version is primarily to address build issues on 32-bit debian
systems (as Rene reported) as well as to finally fix the memory mis-
handling issues in the XML mapping code used in Calc's XML Source
feature.  I've used Stephan's patch as a starting point to hopefully
overhaul the way object life cycles were handled.  Obviously this was a
long standing issue that I never had a chance to look at since 2012,
which may have been a root cause of occasional crashes...

Anyway, it's good to finally give it proper treatment.

Best,

Kohei

-- 
Kohei Yoshida, LibreOffice Calc volunteer hacker

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice

[Libreoffice-bugs] [Bug 122218] After Update to 6.1.4 on macOS fonts are blurred on retina display

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122218

--- Comment #34 from bunkem  ---
The font problem is fixed in the build:

https://dev-builds.libreoffice.org/daily/libreoffice-6-3/MacOSX-x86_64@49-TDF/2019-08-21_17.28.20/
- that was done with XCode 9

Version: 6.3.2.0.0+
Build ID: ad2fde8797bd7753987dc1bbbae35fb1cc5ce1df
CPU threads: 8; OS: Mac OS X 10.14.6; UI render: default; VCL: osx; 
TinderBox: MacOSX-x86_64@49-TDF, Branch:libreoffice-6-3, Time:
2019-08-21_17:28:20
Locale: en-CA (en_CA.UTF-8); UI-Language: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: download.lst external/liborcus

2019-08-22 Thread Kohei Yoshida (via logerrit)
 download.lst  |4 -
 external/liborcus/UnpackedTarball_liborcus.mk |1 
 external/liborcus/create-element.patch.0  |   55 --
 3 files changed, 2 insertions(+), 58 deletions(-)

New commits:
commit 52800fd6d9867252b795b6afacce19f66b5a5107
Author: Kohei Yoshida 
AuthorDate: Thu Aug 22 16:48:50 2019 -0400
Commit: Kohei Yoshida 
CommitDate: Fri Aug 23 01:41:23 2019 +0200

Update orcus to 0.15.1.

Change-Id: Ifd945d03719bf2ed1fb145b405f1ea9297ebeb68
Reviewed-on: https://gerrit.libreoffice.org/77983
Tested-by: Jenkins
Reviewed-by: Kohei Yoshida 

diff --git a/download.lst b/download.lst
index c0217d93b9d0..1b2d943defe8 100644
--- a/download.lst
+++ b/download.lst
@@ -194,8 +194,8 @@ export OPENLDAP_SHA256SUM := 
cdd6cffdebcd95161a73305ec13fc7a78e9707b46ca9f84fb89
 export OPENLDAP_TARBALL := openldap-2.4.45.tgz
 export OPENSSL_SHA256SUM := 
ae51d08bba8a83958e894946f15303ff894d75c2b8bbd44a852b64e3fe11d0d6
 export OPENSSL_TARBALL := openssl-1.0.2r.tar.gz
-export ORCUS_SHA256SUM := 
06ec7567896d76ca579efada794cc281a5d6b427195def01cbcfd79795e8a944
-export ORCUS_TARBALL := liborcus-0.15.0.tar.gz
+export ORCUS_SHA256SUM := 
ab835e131c861401d52a30a74d0c83f175650f3f45460ae95bbf95493cab2579
+export ORCUS_TARBALL := liborcus-0.15.1.tar.gz
 export OWNCLOUD_ANDROID_LIB_SHA256SUM := 
b18b3e3ef7fae6a79b62f2bb43cc47a5346b6330f6a383dc4be34439aca5e9fb
 export OWNCLOUD_ANDROID_LIB_TARBALL := 
owncloud-android-library-0.9.4-no-binary-deps.tar.gz
 export PAGEMAKER_SHA256SUM := 
66adacd705a7d19895e08eac46d1e851332adf2e736c566bef1164e7a442519d
diff --git a/external/liborcus/UnpackedTarball_liborcus.mk 
b/external/liborcus/UnpackedTarball_liborcus.mk
index 834b8128d39f..e1aef32edcf7 100644
--- a/external/liborcus/UnpackedTarball_liborcus.mk
+++ b/external/liborcus/UnpackedTarball_liborcus.mk
@@ -22,7 +22,6 @@ $(eval $(call gb_UnpackedTarball_add_patches,liborcus,\
external/liborcus/rpath.patch.0 \
external/liborcus/gcc9.patch.0 \
external/liborcus/libtool.patch.0 \
-   external/liborcus/create-element.patch.0 \
 ))
 
 ifeq ($(OS),WNT)
diff --git a/external/liborcus/create-element.patch.0 
b/external/liborcus/create-element.patch.0
deleted file mode 100644
index b1012e5c90d8..
--- a/external/liborcus/create-element.patch.0
+++ /dev/null
@@ -1,55 +0,0 @@
 src/liborcus/xml_map_tree.cpp
-+++ src/liborcus/xml_map_tree.cpp
-@@ -723,35 +723,28 @@
- element* elem = r.first;
- bool created = r.second;
- 
--if (created)
--{
--// No element of that name exists.
--elem->elem_type = element_linked;
--elem->ref_type = ref_type;
--}
--else
-+if (!created)
- {
- // This element already exists.  Check if this is already linked.
- if (elem->ref_type != reference_unknown || elem->elem_type != 
element_unlinked)
- throw xpath_error("This element is already linked.  You can't 
link the same element twice.");
--
--// Turn this existing non-linked element into a linked one.
--delete elem->child_elements;
--elem->elem_type = element_linked;
--elem->ref_type = ref_type;
--switch (ref_type)
--{
--case reference_cell:
--elem->cell_ref = new cell_reference;
--break;
--case reference_range_field:
--elem->field_ref = new field_in_range;
--break;
--default:
--throw general_error("Unknown reference type in 
xml_map_tree::get_element_stack.");
--}
--
- }
-+
-+// Turn this existing non-linked element into a linked one.
-+delete elem->child_elements;
-+elem->elem_type = element_linked;
-+elem->ref_type = ref_type;
-+switch (ref_type)
-+{
-+case reference_cell:
-+elem->cell_ref = new cell_reference;
-+break;
-+case reference_range_field:
-+elem->field_ref = new field_in_range;
-+break;
-+default:
-+throw general_error("Unknown reference type in 
xml_map_tree::get_element_stack.");
-+}
- 
- ret.elem_stack.push_back(elem);
- ret.node = elem;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 127090] FILESAVE: PPTX: Text in pieWedge shape rotates with shape instead of staying upright

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127090

Gerald Pfeifer  changed:

   What|Removed |Added

Summary|FILESAVE: PPTX: Text in |FILESAVE: PPTX: Text in
   |quarter circles rotated |pieWedge shape rotates with
   |arbitrarily |shape instead of staying
   ||upright

--- Comment #5 from Gerald Pfeifer  ---
(In reply to Regina Henschel from comment #4)
> I wonder, how you have generated the file. The preset shape "pieWedge" does
> not belong to shapes, which are available in the UI, neither in LibreOffice
> nor in PowerPoint. And rotating text inside a custom shape has no UI in
> LibreOffice.

This came from a PPTX I got.  The PPTX to ODP path went surprisingly
okay-ish (no undesired rotations).  Just when then saving as PPTX since
a colleague could not handle the ODP did this materialize.

(Nice analysis, by the way. Thank you!)

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127100] Formula font size cannot be changed when using new Tabbed UI in Impress

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127100

Benedikt Bieringer <2xb.cod...@wwu.de> changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

--- Comment #2 from Benedikt Bieringer <2xb.cod...@wwu.de> ---
Roman: Yes, it is an OLE element. Since the Tabbed UI replaces the menu bar
with tabs, selecting the menu "Format" is no longer possible. Probably when in
OLE mode there should be an OLE tab containing e.g. the "Format" menu, but
there is not.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127102] FILEOPEN Combined Chart of type stacked area + stacked column loses column stacking

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127102

--- Comment #2 from Durgapriyanka  ---
Thank you for reporting this bug. I can reproduce this in

Version: 6.3.0.0.alpha0+
Build ID: b6b28931435e44aca92b8c0e1659f701e3ed1a87
CPU threads: 2; OS: Windows 6.1; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-01-30_06:57:04
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: shell/source

2019-08-22 Thread Michael Weghorn (via logerrit)
 shell/source/backends/kf5be/kf5backend.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ff398785bede2b403f8f8e6391ce07538f6cc213
Author: Michael Weghorn 
AuthorDate: Thu Aug 22 19:09:46 2019 +0200
Commit: Michael Weghorn 
CommitDate: Thu Aug 22 22:58:41 2019 +0200

kf5backend.cxx: Use 'free()' instead of 'delete'

'strdup' doc [1] mentions:
"The returned pointer must be passed to 'free' to avoid a memory leak."

This silences a "Mismatched free() / delete / delete []" error that
valgrind outputs.

[1] https://en.cppreference.com/w/c/experimental/dynamic/strdup

Change-Id: I28877be762256a7b995c09415a8ad9977b7998c4
Reviewed-on: https://gerrit.libreoffice.org/77974
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/shell/source/backends/kf5be/kf5backend.cxx 
b/shell/source/backends/kf5be/kf5backend.cxx
index a2cae6c6c4fd..f55a5150bd3f 100644
--- a/shell/source/backends/kf5be/kf5backend.cxx
+++ b/shell/source/backends/kf5be/kf5backend.cxx
@@ -190,7 +190,7 @@ void initQApp(std::map>& rSettings
 std::unique_ptr app(new QApplication(nFakeArgc, pFakeArgv));
 QObject::connect(app.get(), ::destroyed, app.get(), [nFakeArgc, 
pFakeArgv]() {
 for (int i = 0; i < nFakeArgc; ++i)
-delete pFakeArgv[i];
+free(pFakeArgv[i]);
 delete[] pFakeArgv;
 });
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126902] ^m (default) does not un-bold

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126902

Durgapriyanka  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #2 from Durgapriyanka  ---
Thank you for reporting the bug. I can confirm the bug present in 


Version: 6.3.0.0.alpha0+
Build ID: b6b28931435e44aca92b8c0e1659f701e3ed1a87
CPU threads: 2; OS: Windows 6.1; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-01-30_06:57:04
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

and

LibreOffice 3.3.0 
OOO330m19 (Build:6)
tag libreoffice-3.3.0.4

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127073] Redaction done right

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127073

--- Comment #2 from Muhammet Kara  ---
(In reply to Ulrich Windl from comment #0)
> (the current implementation is basically the digital version of "print it,
> blacken it, scan it again to make a new document").

That's exactly how it was designed.

(In reply to Roman Kuznetsov from comment #1)
> I think it's WONTFIX
> 
> Muhammet, what do you think?

What the reporter describes might be a nice (but risky -check the internet for
security incidents of similar implementations-) implementation of a redaction
feature, but not this one. Unless there is a strong will, and necessary
resources to implement another version of the redaction feature, this is a
WONTFIX for me.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 104442] [META] DOCX (OOXML) shape (DrawingML and VML) related issues

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=104442

Regina Henschel  changed:

   What|Removed |Added

 Depends on||127090


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=127090
[Bug 127090] FILESAVE: PPTX: Text in quarter circles rotated arbitrarily
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127090] FILESAVE: PPTX: Text in quarter circles rotated arbitrarily

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127090

Regina Henschel  changed:

   What|Removed |Added

 Blocks||104442
 CC||rb.hensc...@t-online.de
 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #4 from Regina Henschel  ---
The text does not rotate arbitrarily but follows the shape rotation. The
problem is, that LibreOffice does not support the needed attribute "upright" of
the  element in OOXML. And OOXML does not support the additional text
rotation of text inside a custom shape of ODF.

So a proper fix would be to make LibreOffice able to transform the "upright"
attribute into a suitable text rotation on import and transform the text
rotation back to attribute "upright" on export.

A similar problem exists as bug 104290 for Calc.

I wonder, how you have generated the file. The preset shape "pieWedge" does not
belong to shapes, which are available in the UI, neither in LibreOffice nor in
PowerPoint. And rotating text inside a custom shape has no UI in LibreOffice.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=104442
[Bug 104442] [META] DOCX (OOXML) shape (DrawingML and VML) related issues
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127107] New: Will not print documents

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127107

Bug ID: 127107
   Summary: Will not print documents
   Product: LibreOffice
   Version: 6.2.0.1 rc
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: fred...@gmail.com

Description:
Since your last update, every time I click print, Libre Writer, routes my
document to "One Note", so I uninstalled Libre Office,  I had no other
choice.  I cannot uninstall "One Note".

Actual Results:
No results..

Expected Results:
Nothing


Reproducible: Always


User Profile Reset: No



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127104] CALC: One COPY and Multiple PASTES

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127104

m.a.riosv  changed:

   What|Removed |Added

 CC||miguelangelrv@libreoffice.o
   ||rg
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

--- Comment #1 from m.a.riosv  ---
I can't reproduce with
Version: 6.2.6.2 (x64)
Build ID: 684e730861356e74889dfe6dbddd3562aae2e6ad
CPU threads: 4; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: es-ES (es_ES); UI-Language: en-US Calc: CL

The reported version 3.5.4 it's a really old version, please could you try with
6.2 or 6.3

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: sfx2/source

2019-08-22 Thread Stephan Bergmann (via logerrit)
 sfx2/source/doc/objmisc.cxx |   11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

New commits:
commit 7a6f8baa738c7ec0348015f5c423802e10a524fd
Author: Stephan Bergmann 
AuthorDate: Thu Aug 22 17:34:42 2019 +0200
Commit: Stephan Bergmann 
CommitDate: Thu Aug 22 21:54:52 2019 +0200

Replace use of INetURLObject::GetNewAbsURL with rtl::Uri::convertRelToAbs

Change-Id: Ida2a0553ede3d6d9d7ff4a50bca6256a70286bc2
Reviewed-on: https://gerrit.libreoffice.org/77970
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/sfx2/source/doc/objmisc.cxx b/sfx2/source/doc/objmisc.cxx
index df52c23c0b7c..33341b58a3a3 100644
--- a/sfx2/source/doc/objmisc.cxx
+++ b/sfx2/source/doc/objmisc.cxx
@@ -80,6 +80,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1477,10 +1478,12 @@ void SfxHeaderAttributes_Impl::SetAttribute( const 
SvKeyValue& rKV )
 pDoc->getDocProperties());
 if( aURL.startsWithIgnoreAsciiCase( "url=" ) )
 {
-INetURLObject aObj;
-INetURLObject( pDoc->GetMedium()->GetName() ).GetNewAbsURL( 
aURL.copy( 4 ),  );
-xDocProps->setAutoloadURL(
-aObj.GetMainURL( INetURLObject::DecodeMechanism::NONE ) );
+try {
+xDocProps->setAutoloadURL(
+rtl::Uri::convertRelToAbs(pDoc->GetMedium()->GetName(), 
aURL.copy( 4 )) );
+} catch (rtl::MalformedUriException &) {
+TOOLS_WARN_EXCEPTION("sfx", "");
+}
 }
 try
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 127106] New: No icons in the menus of Libreoffice 6.3.0.4

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127106

Bug ID: 127106
   Summary: No icons in the menus of Libreoffice 6.3.0.4
   Product: LibreOffice
   Version: 6.3.0.4 release
  Hardware: All
OS: Linux (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: jeanpaul.th...@cegetel.net

Description:
No icons in the menus even if, in the options of Libreoffice 6.3.0.4, 
-> Display -> Icons in the menus -> Show
is chosen

Steps to Reproduce:
1.Choose : Options of Libreoffice 6.3.0.4
2.Choose : -> Display 
3.Choose : -> Icons in the menus 
4.Choose : -> Show

Actual Results:
There no icons in the menus

Expected Results:
Icons in the menus


Reproducible: Always


User Profile Reset: Yes



Additional Info:
LibreOffice

Version: 6.3.0.4
Build ID: 057fc023c990d676a43019934386b85b21a9ee99
Threads CPU : 2; OS : Linux 4.15; UI Render : par défaut; VCL: gtk3; 
Locale : fr-FR (fr_FR.UTF-8); Langue IHM : fr-FR
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127105] New: Threaded comments support (OOXML) in XLSX

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127105

Bug ID: 127105
   Summary: Threaded comments support (OOXML) in XLSX
   Product: LibreOffice
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: tbertels+bugzi...@gmail.com

Since "newer versions of Excel", there is a new "threaded comments" feature.

It's also simply known as "comments" while the old comments are now known as
"notes".[1]

When viewing a comment saved by a recent version of Excel in LibreOffice, we
see a message "[Threaded comment] This threaded comment can be viewed, but not
edited(...)"[2]

By the way, this kind of feature could be added to a future version of
OpenDocument, if not already planned in OpenDocument 1.3.

[1]
https://support.office.com/en-us/article/the-difference-between-threaded-comments-and-notes-75a51eec-4092-42ab-abf8-7669077b7be3
[2]
https://support.office.com/en-us/article/this-threaded-comment-can-be-viewed-but-not-edited-17b3eb02-efa7-4934-9e8e-b8916482bf41?ui=en-US=en-US=US

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127104] New: CALC: One COPY and Multiple PASTES

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127104

Bug ID: 127104
   Summary: CALC: One COPY and Multiple PASTES
   Product: LibreOffice
   Version: 3.5.4 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: neilhmur...@yahoo.com

Description:
I was trying to do multiple PASTEs of the same row. The second one
failed. I had done an insert row between the first and second PASTE. I
went back and issued a COPY then a PASTE. That worked.

Steps to Reproduce:
1.COPY
2.PASTE
3.PASTE

Actual Results:
#3 Didn't work=row was still blank

Expected Results:
#3 would have the same contents as #2


Reproducible: Didn't try


User Profile Reset: No



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127073] Redaction done right

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127073

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru,
   ||muhammet.k...@collabora.com

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
But then any enemy (or journalists) will delete your shapes using good PDF
Editor and will see your secret info. Export to bitmap PDF will guarantee us
that our secret info will remain secret.

I think it's WONTFIX

Muhammet, what do you think?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: extensions/source

2019-08-22 Thread Caolán McNamara (via logerrit)
 extensions/source/dbpilots/controlwizard.cxx |2 +-
 extensions/source/dbpilots/controlwizard.hxx |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit d9d096a567653d8daddedf88ec40d13113354c3b
Author: Caolán McNamara 
AuthorDate: Thu Aug 22 16:25:18 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 21:09:27 2019 +0200

OControlWizard->vcl::Window

Change-Id: Ibb007621aca910cd2fd983b2f255e93160bddb5c
Reviewed-on: https://gerrit.libreoffice.org/77968
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/extensions/source/dbpilots/controlwizard.cxx 
b/extensions/source/dbpilots/controlwizard.cxx
index 6589b2bb2f25..e02dbbfb2c24 100644
--- a/extensions/source/dbpilots/controlwizard.cxx
+++ b/extensions/source/dbpilots/controlwizard.cxx
@@ -74,7 +74,7 @@ namespace dbp
 OAccessRegulator() { }
 };
 
-OControlWizardPage::OControlWizardPage( OControlWizard* _pParent, const 
OString& rID, const OUString& rUIXMLDescription )
+OControlWizardPage::OControlWizardPage( vcl::Window* _pParent, const 
OString& rID, const OUString& rUIXMLDescription )
 :OControlWizardPage_Base( _pParent, rID, rUIXMLDescription )
 ,m_pFormDatasourceLabel(nullptr)
 ,m_pFormDatasource(nullptr)
diff --git a/extensions/source/dbpilots/controlwizard.hxx 
b/extensions/source/dbpilots/controlwizard.hxx
index d84babbfc250..68b0551d1780 100644
--- a/extensions/source/dbpilots/controlwizard.hxx
+++ b/extensions/source/dbpilots/controlwizard.hxx
@@ -65,7 +65,7 @@ namespace dbp
 css::uno::Reference< css::sdbc::XConnection >
 getFormConnection() const;
 public:
-OControlWizardPage( OControlWizard* _pParent, const OString& rID, 
const OUString& rUIXMLDescription );
+OControlWizardPage( vcl::Window* _pParent, const OString& rID, const 
OUString& rUIXMLDescription );
 virtual ~OControlWizardPage() override;
 virtual void dispose() override;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126990] Colours inverted in video played in slideshow

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126990

--- Comment #3 from Jan-Marek Glogowski  ---
It works correct in gen and qt5, but is broken in kf5 and gtk3, which IMHO
looks like the cairo canvas bug, as it also works correctly in Impress edit
mode.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127091] TVM Formula, Begin & end mode decryption is wrong.

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127091

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru,
   ||er...@redhat.com

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
Eike, please look at this

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127098] Functions to Track and View Changes should really become Useful

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127098

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Blocks||83946


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=83946
[Bug 83946] [META] Tracking changes issues
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 83946] [META] Tracking changes issues

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=83946

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Depends on||127098


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=127098
[Bug 127098] Functions to Track and View Changes should really become Useful
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 107237] [META] Notebookbar Tabbed

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=107237

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Depends on||127100


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=127100
[Bug 127100] Formula font size cannot be changed when using new Tabbed UI in
Impress
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127100] Formula font size cannot be changed when using new Tabbed UI in Impress

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127100

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Blocks||107237


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=107237
[Bug 107237] [META] Notebookbar Tabbed
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127100] Formula font size cannot be changed when using new Tabbed UI in Impress

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127100

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
Benedikt, how did you add your formula? 
If you used Insert->OLE Object->Formula then you can't change font size from
Impress UI. You should enter to Edit formula mode (double click) and then
select menu Format->Font size and set up font size there.

Or you can use extension
https://extensions.libreoffice.org/extensions/formatting-of-all-math-formulas

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: sw/source

2019-08-22 Thread Caolán McNamara (via logerrit)
 sw/source/ui/dbui/mmaddressblockpage.cxx  |8 
 sw/source/ui/dbui/mmgreetingspage.cxx |2 
 sw/source/ui/dbui/mmresultdialogs.cxx |2 
 sw/source/uibase/dbui/mailmergehelper.cxx |  252 --
 sw/source/uibase/inc/mailmergehelper.hxx  |   56 --
 5 files changed, 11 insertions(+), 309 deletions(-)

New commits:
commit 43a394e111a579b1d01ab2703101eabe6abfffbf
Author: Caolán McNamara 
AuthorDate: Thu Aug 22 17:26:21 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 20:40:25 2019 +0200

drop newly unused SwAddressPreview

Change-Id: I84cf6570d048df86b4690cb70cc3e7a256303951
Reviewed-on: https://gerrit.libreoffice.org/77972
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/source/ui/dbui/mmaddressblockpage.cxx 
b/sw/source/ui/dbui/mmaddressblockpage.cxx
index 0edb9bddee1f..f4b7359be648 100644
--- a/sw/source/ui/dbui/mmaddressblockpage.cxx
+++ b/sw/source/ui/dbui/mmaddressblockpage.cxx
@@ -246,7 +246,7 @@ IMPL_LINK_NOARG(SwMailMergeAddressBlockPage, 
AddressBlockSelectHdl_Impl, LinkPar
 const sal_uInt16 nSel = m_xSettings->GetSelectedAddress();
 const uno::Sequence< OUString> aBlocks =
 m_pWizard->GetConfigItem().GetAddressBlocks();
-m_xPreview->SetAddress(SwAddressPreview::FillData(aBlocks[nSel],
+m_xPreview->SetAddress(AddressPreview::FillData(aBlocks[nSel],
  
m_pWizard->GetConfigItem()));
 m_pWizard->GetConfigItem().SetCurrentAddressBlockIndex( nSel );
 GetWizard()->UpdateRoadmap();
@@ -291,7 +291,7 @@ void 
SwMailMergeAddressBlockPage::InsertDataHdl(weld::Button* pButton)
 const sal_uInt16 nSel = m_xSettings->GetSelectedAddress();
 const uno::Sequence< OUString> aBlocks =
 m_pWizard->GetConfigItem().GetAddressBlocks();
-m_xPreview->SetAddress(SwAddressPreview::FillData(aBlocks[nSel], 
rConfig));
+m_xPreview->SetAddress(AddressPreview::FillData(aBlocks[nSel], 
rConfig));
 }
 }
 m_xPrevSetIB->set_sensitive(bEnable);
@@ -569,7 +569,7 @@ IMPL_LINK(SwCustomizeAddressBlockDialog, 
ListBoxSelectHdl_Impl, weld::TreeView&,
 
 IMPL_LINK_NOARG(SwCustomizeAddressBlockDialog, EditModifyHdl_Impl, 
AddressMultiLineEdit&, void)
 {
-m_xPreview->SetAddress(SwAddressPreview::FillData(GetAddress(), 
m_rConfigItem));
+m_xPreview->SetAddress(AddressPreview::FillData(GetAddress(), 
m_rConfigItem));
 UpdateImageButtons_Impl();
 }
 
@@ -1026,7 +1026,7 @@ IMPL_LINK_NOARG(SwAssignFieldsDialog, OkHdl_Impl, 
weld::Button&, void)
 IMPL_LINK_NOARG(SwAssignFieldsDialog, AssignmentModifyHdl_Impl, 
LinkParamNone*, void)
 {
 uno::Sequence< OUString > aAssignments = CreateAssignments();
-const OUString sPreview = SwAddressPreview::FillData(
+const OUString sPreview = AddressPreview::FillData(
 m_rPreviewString, m_rConfigItem, );
 m_xPreview->SetAddress(sPreview);
 }
diff --git a/sw/source/ui/dbui/mmgreetingspage.cxx 
b/sw/source/ui/dbui/mmgreetingspage.cxx
index b80cc86480d7..92ae163f 100644
--- a/sw/source/ui/dbui/mmgreetingspage.cxx
+++ b/sw/source/ui/dbui/mmgreetingspage.cxx
@@ -173,7 +173,7 @@ void SwMailMergeGreetingsPage::UpdatePreview()
 OUString sPreview = bFemale ? m_xFemaleLB->get_active_text() :
 bNoValue ? m_xNeutralCB->get_active_text() : 
m_xMaleLB->get_active_text();
 
-sPreview = SwAddressPreview::FillData(sPreview, m_rConfigItem);
+sPreview = AddressPreview::FillData(sPreview, m_rConfigItem);
 m_xPreview->SetAddress(sPreview);
 }
 
diff --git a/sw/source/ui/dbui/mmresultdialogs.cxx 
b/sw/source/ui/dbui/mmresultdialogs.cxx
index f107758feda0..912cb9a3f8cd 100644
--- a/sw/source/ui/dbui/mmresultdialogs.cxx
+++ b/sw/source/ui/dbui/mmresultdialogs.cxx
@@ -1134,7 +1134,7 @@ IMPL_LINK_NOARG(SwMMResultEmailDialog, 
SendDocumentsHdl_Impl, weld::Button&, voi
 SwMailMergeConfigItem::FEMALE :
 SwMailMergeConfigItem::MALE;
 
-sGreeting = SwAddressPreview::FillData(
+sGreeting = AddressPreview::FillData(
 xConfigItem->GetGreetings(eGenderType)
 [xConfigItem->GetCurrentGreeting(eGenderType)],
 *xConfigItem);
diff --git a/sw/source/uibase/dbui/mailmergehelper.cxx 
b/sw/source/uibase/dbui/mailmergehelper.cxx
index bbf60a49dd63..4d1aa49f3445 100644
--- a/sw/source/uibase/dbui/mailmergehelper.cxx
+++ b/sw/source/uibase/dbui/mailmergehelper.cxx
@@ -180,257 +180,7 @@ struct  SwAddressPreview_Impl
 }
 };
 
-SwAddressPreview::SwAddressPreview(vcl::Window* pParent, WinBits nStyle)
-: Window( pParent, nStyle )
-, aVScrollBar(VclPtr::Create(this, WB_VSCROLL))
-, pImpl(new SwAddressPreview_Impl())
-{
-aVScrollBar->SetScrollHdl(LINK(this, SwAddressPreview, ScrollHdl));
-

[Libreoffice-commits] core.git: 2 commits - extras/source sfx2/source solenv/bin sw/source sw/uiconfig vcl/source

2019-08-22 Thread Caolán McNamara (via logerrit)
 extras/source/glade/libreoffice-catalog.xml.in |3 
 sfx2/source/dialog/backingcomp.cxx |   10 +
 solenv/bin/native-code.py  |1 
 sw/source/ui/dbui/mailmergewizard.cxx  |2 
 sw/source/ui/dbui/mmaddressblockpage.cxx   |  201 -
 sw/source/ui/dbui/mmaddressblockpage.hxx   |   59 ---
 sw/source/uibase/dbui/mailmergehelper.cxx  |9 +
 sw/source/uibase/inc/mailmergehelper.hxx   |5 
 sw/uiconfig/swriter/ui/mmaddressblockpage.ui   |   38 
 vcl/source/control/button.cxx  |   10 +
 vcl/source/window/bufferdevice.cxx |2 
 vcl/source/window/bufferdevice.hxx |1 
 vcl/source/window/menubarwindow.cxx|   16 +
 13 files changed, 218 insertions(+), 139 deletions(-)

New commits:
commit b5f2f32defd22c8ec0c47c0100836a8d5bb132ea
Author: Caolán McNamara 
AuthorDate: Thu Aug 22 17:15:44 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 20:40:07 2019 +0200

weld SwMailMergeAddressBlockPage

Change-Id: Ib3383f1ae590766ecb1371cab2ccdb7c835c95a8
Reviewed-on: https://gerrit.libreoffice.org/77971
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/extras/source/glade/libreoffice-catalog.xml.in 
b/extras/source/glade/libreoffice-catalog.xml.in
index eef7bdd744b6..a86845bd17d1 100644
--- a/extras/source/glade/libreoffice-catalog.xml.in
+++ b/extras/source/glade/libreoffice-catalog.xml.in
@@ -159,9 +159,6 @@
 
-
 
diff --git a/solenv/bin/native-code.py b/solenv/bin/native-code.py
index a58a30654a06..45a3fee13118 100755
--- a/solenv/bin/native-code.py
+++ b/solenv/bin/native-code.py
@@ -512,7 +512,6 @@ custom_widgets = [
 'SvxRelativeField',
 'SvxTextEncodingBox',
 'SvxTextEncodingBox',
-'SwAddressPreview',
 'SwNavHelpToolBox',
 'TableValueSet',
 'TemplateDefaultView',
diff --git a/sw/source/ui/dbui/mailmergewizard.cxx 
b/sw/source/ui/dbui/mailmergewizard.cxx
index 49ea7c661fc9..6ca8822cbfc3 100644
--- a/sw/source/ui/dbui/mailmergewizard.cxx
+++ b/sw/source/ui/dbui/mailmergewizard.cxx
@@ -106,7 +106,7 @@ VclPtr SwMailMergeWizard::createPage(WizardState 
_nState)
 
SetRoadmapHelpId("modules/swriter/ui/mmoutputtypepage/MMOutputTypePage");
 break;
 case MM_ADDRESSBLOCKPAGE   :
-pRet = VclPtr::Create(this);
+pRet = VclPtr::Create(this, 
TabPageParent(this));
 
SetRoadmapHelpId("modules/swriter/ui/mmaddressblockpage/MMAddressBlockPage");
 break;
 case MM_GREETINGSPAGE  :
diff --git a/sw/source/ui/dbui/mmaddressblockpage.cxx 
b/sw/source/ui/dbui/mmaddressblockpage.cxx
index dd12860e54ca..0edb9bddee1f 100644
--- a/sw/source/ui/dbui/mmaddressblockpage.cxx
+++ b/sw/source/ui/dbui/mmaddressblockpage.cxx
@@ -57,46 +57,46 @@ using namespace ::com::sun::star::sdb;
 using namespace ::com::sun::star::sdbc;
 using namespace ::com::sun::star::sdbcx;
 
-SwMailMergeAddressBlockPage::SwMailMergeAddressBlockPage( SwMailMergeWizard* 
_pParent) :
-vcl::OWizardPage(_pParent, "MMAddressBlockPage",
-"modules/swriter/ui/mmaddressblockpage.ui")
-, m_pWizard(_pParent)
-{
-get(m_pAddressListPB, "addresslist");
-get(m_pCurrentAddressFI, "currentaddress");
-get(m_pStep2, "step2");
-get(m_pStep3, "step3");
-get(m_pStep4, "step4");
-get(m_pSettingsFI, "settingsft");
-get(m_pAddressCB, "address");
-get(m_pSettingsWIN, "settingspreview");
+SwMailMergeAddressBlockPage::SwMailMergeAddressBlockPage(SwMailMergeWizard* 
pWizard, TabPageParent pParent)
+: vcl::OWizardPage(pParent, "modules/swriter/ui/mmaddressblockpage.ui", 
"MMAddressBlockPage")
+, m_pWizard(pWizard)
+, m_xAddressListPB(m_xBuilder->weld_button("addresslist"))
+, m_xCurrentAddressFI(m_xBuilder->weld_label("currentaddress"))
+, m_xStep2(m_xBuilder->weld_container("step2"))
+, m_xStep3(m_xBuilder->weld_container("step3"))
+, m_xStep4(m_xBuilder->weld_container("step4"))
+, m_xSettingsFI(m_xBuilder->weld_label("settingsft"))
+, m_xAddressCB(m_xBuilder->weld_check_button("address"))
+, m_xSettingsPB(m_xBuilder->weld_button("settings"))
+, m_xHideEmptyParagraphsCB(m_xBuilder->weld_check_button("hideempty"))
+, m_xAssignPB(m_xBuilder->weld_button("assign"))
+, m_xDocumentIndexFI(m_xBuilder->weld_label("documentindex"))
+, m_xPrevSetIB(m_xBuilder->weld_button("prev"))
+, m_xNextSetIB(m_xBuilder->weld_button("next"))
+, m_xDifferentlist(m_xBuilder->weld_label("differentlist"))
+, m_xSettings(new 
AddressPreview(m_xBuilder->weld_scrolled_window("settingspreviewwin")))
+, m_xPreview(new 
AddressPreview(m_xBuilder->weld_scrolled_window("addresspreviewwin")))
+, m_xSettingsWIN(new weld::CustomWeld(*m_xBuilder, "settingspreview", 
*m_xSettings))
+, m_xPreviewWIN(new weld::CustomWeld(*m_xBuilder, "addresspreview", 
*m_xPreview))
+{
 

[Libreoffice-bugs] [Bug 120965] Phantom control point line in Bézier curve

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120965

--- Comment #12 from Commit Notification 
 ---
Regina Henschel committed a patch related to this issue.
It has been pushed to "libreoffice-6-2":

https://git.libreoffice.org/core/+/46c7341014870b4d2c9babd91c14dfed4422f93a%5E%21

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

It will be available in 6.2.7.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126271] Selecting of control points of curves not working correctly for combined polygons

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126271

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:6.4.0 target:6.3.2   |target:6.4.0 target:6.3.2
   ||target:6.2.7

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126271] Selecting of control points of curves not working correctly for combined polygons

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126271

--- Comment #13 from Commit Notification 
 ---
Regina Henschel committed a patch related to this issue.
It has been pushed to "libreoffice-6-2":

https://git.libreoffice.org/core/+/46c7341014870b4d2c9babd91c14dfed4422f93a%5E%21

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

It will be available in 6.2.7.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 120965] Phantom control point line in Bézier curve

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120965

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:6.2.0 target:6.4.0   |target:6.2.0 target:6.4.0
   |target:6.3.2|target:6.3.2 target:6.2.7

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'libreoffice-6-2' - svx/source

2019-08-22 Thread Regina Henschel (via logerrit)
 svx/source/svdraw/svdopath.cxx |   71 +
 1 file changed, 44 insertions(+), 27 deletions(-)

New commits:
commit 46c7341014870b4d2c9babd91c14dfed4422f93a
Author: Regina Henschel 
AuthorDate: Wed Aug 21 14:11:23 2019 +0200
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 20:39:24 2019 +0200

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

This reverts commit 26001f4f84b0d921c8612d56613735ba5314.

Change-Id: Ib4df476b405e65be9203ab376c9c68b03198c040
Reviewed-on: https://gerrit.libreoffice.org/77882
Tested-by: Regina Henschel 
Reviewed-by: Regina Henschel 
(cherry picked from commit 59189f3e02acbdf4e5bbd35fc278252b30022f43)
Reviewed-on: https://gerrit.libreoffice.org/77940
Tested-by: Jenkins

diff --git a/svx/source/svdraw/svdopath.cxx b/svx/source/svdraw/svdopath.cxx
index 63481a141248..eda7c34db46a 100644
--- a/svx/source/svdraw/svdopath.cxx
+++ b/svx/source/svdraw/svdopath.cxx
@@ -2008,45 +2008,62 @@ void SdrPathObj::AddToHdlList(SdrHdlList& rHdlList) 
const
 
 void SdrPathObj::AddToPlusHdlList(SdrHdlList& rHdlList, SdrHdl& rHdl) const
 {
-// exclude some error situations
-const XPolyPolygon aPathPolyPolygon(GetPathPoly());
+// keep old stuff to be able to keep old SdrHdl stuff, too
+const XPolyPolygon aOldPathPolygon(GetPathPoly());
+sal_uInt16 nPnt = static_cast(rHdl.GetPointNum());
 sal_uInt16 nPolyNum = static_cast(rHdl.GetPolyNum());
-if (nPolyNum>=aPathPolyPolygon.Count())
-return;
 
-const XPolygon& rXPoly = aPathPolyPolygon[nPolyNum];
-sal_uInt16 nPntCount = rXPoly.GetPointCount();
-if (nPntCount<=0)
+if (nPolyNum>=aOldPathPolygon.Count())
 return;
 
-sal_uInt16 nPnt = static_cast(rHdl.GetPointNum());
-if (nPnt>=nPntCount)
-return;
+const XPolygon& rXPoly = aOldPathPolygon[nPolyNum];
+sal_uInt16 nPntMax = rXPoly.GetPointCount();
 
-if (rXPoly.IsControl(nPnt))
+if (nPntMax<=0)
+return;
+nPntMax--;
+if (nPnt>nPntMax)
 return;
 
-// segment before
-if (nPnt==0 && IsClosed())
-nPnt=nPntCount-1;
-if (nPnt>0 && rXPoly.IsControl(nPnt-1))
+// calculate the number of plus points
+sal_uInt16 nCnt = 0;
+if (rXPoly.GetFlags(nPnt)!=PolyFlags::Control)
 {
-std::unique_ptr pHdl(new SdrHdlBezWgt());
-pHdl->SetPos(rXPoly[nPnt-1]);
-pHdl->SetPointNum(nPnt-1);
-pHdl->SetSourceHdlNum(rHdl.GetSourceHdlNum());
-pHdl->SetPlusHdl(true);
-rHdlList.AddHdl(std::move(pHdl));
+if (nPnt==0 && IsClosed())
+nPnt=nPntMax;
+if (nPnt>0 && rXPoly.GetFlags(nPnt-1)==PolyFlags::Control)
+nCnt++;
+if (nPnt==nPntMax && IsClosed())
+nPnt=0;
+if (nPnt(rHdl.GetPointNum());
 std::unique_ptr pHdl(new SdrHdlBezWgt());
-pHdl->SetPos(rXPoly[nPnt+1]);
-pHdl->SetPointNum(nPnt+1);
+pHdl->SetPolyNum(rHdl.GetPolyNum());
+
+if (nPnt==0 && IsClosed())
+nPnt=nPntMax;
+if (nPnt>0 && rXPoly.GetFlags(nPnt-1)==PolyFlags::Control && 
nPlusNum==0)
+{
+pHdl->SetPos(rXPoly[nPnt-1]);
+pHdl->SetPointNum(nPnt-1);
+}
+else
+{
+if (nPnt==nPntMax && IsClosed())
+nPnt=0;
+if (nPntSetPos(rXPoly[nPnt+1]);
+pHdl->SetPointNum(nPnt+1);
+}
+}
+
 pHdl->SetSourceHdlNum(rHdl.GetSourceHdlNum());
 pHdl->SetPlusHdl(true);
 rHdlList.AddHdl(std::move(pHdl));
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 102495] [META] KDE VCL backend bugs and enhancements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=102495
Bug 102495 depends on bug 126560, which changed state.

Bug 126560 Summary: KDE-integration (Plasma 5): shifting/inserting rows with 
Alt+Shift not working correctly
https://bugs.documentfoundation.org/show_bug.cgi?id=126560

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-ux-advise] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

--- Comment #2 from Mike Kaganski  ---
A transcript from #libreoffice IRC chat:

> Marqeaux has joined
> Marqeaux: Hi everybody. I want to ask a question about the formula bar in 
> Calc. The font size in the formula bar has been increased for better 
> readability. Is there a way to make the font size in the formula bar smaller 
> again, just like it was in previous versions of LibreOffice?
> Marqeaux: Oh, I'm using LibreOffice 6.3.0. I forgot to mention that piece of 
> information...
> quikee[m]: Marqeaux: interesting ... it's the same size for me
> Marqeaux: I got LibreOffice 6.3.0. on more machines installed. The formula 
> bar shows a bigger font (too big for my taste), and the formula bar is also 
> increased in height. It used to have the same height as the cell bar (where 
> you see in which cell you are)...
> Marqeaux: I prefer the situation like it was in LO 6.2. (the formula bar, I 
> mean...)
> Marqeaux: ;)
> mikekaganski: Marqeaux: I confirm font size increase in 6.3
> quikee[m]: maybe it manifests in windows. I was looking in Linux with gtk3 
> backend
> quikee[m]: we modified the scrollbars of the formula bar.. could be that has 
> influenced the size
> mikekaganski: quikee[m]: /me bisects
> Marqeaux: Is there a way in the expert configuration to alter the size of the 
> fonts in the formula bar, and also the height of the formula bar?
> quikee[m]: Marqeaux: no, I don't think so... increase is also not intentional 
> so it looks like you "found" a bug
> mikekaganski: Marqeaux: ... which needs a bug report
> mikekaganski: quikee[m]: 04b60370a73b79e3aa0a04bc7cff45dff1db6286
> Marqeaux: I made a screenshot of what I see (on multiple machines, not only 
> on Xubuntu): https://mupload.nl/img/edk3kknwn.png
> mikekaganski: Marqeaux: so it was intentional in the end: tdf#101443
> Marqeaux: I guess it's intentional, because more and more software follow the 
> GTK3 guidelines. I think it's awful, because GTK3 makes everything big and 
> bold, and with no borders. I looks like a "Fischer-Price" kinda of 
> approach
> mikekaganski: Marqeaux: it wasn't because of "GTK3" (and most LO users are on 
> Windows btw); it was because of the previous font was unreadable on some 
> macOS (not using GTK3 also)
> mikekaganski: Marqeaux: "In addition to inputline redesign, character size 
> has been increased to 120% of original size. This prevents very small / 
> unreadable inputline character sizes on some OS X resolutions"
> Marqeaux: Ah, alright. But anyway I takes up too much space now. It's 
> fine that the fonts in the formula bar are bigger, but I hope that there's a 
> way to make it "normal" again...
> mikekaganski: Marqeaux: the commit mentioned above has the min font size 
> fixed (16, strange size for UI actually). You may file an enhancement request 
> to make it user-configurable or smaller.
> Marqeaux: Yes, I also found it a very strange decission to make it that big. 
> It just doesn't fit into the GUI. I think it's better to make it 
> user-configurable, rather than to leave it "hard-coded". Although my eyes 
> aren't that good (I wear glasses and I'm middle aged), but I never had 
> trouble reading the formula bar as it was before.
> Marqeaux: Where can I file an enhancement request?
> mikekaganski: Marqeaux: I already wrote a comment there in the original bug. 
> I gave you the link to the bug, and that's the site where we write and track 
> reports.
> mikekaganski: Marqeaux: please file your request separately regardless of my 
> comment: it would help to provide arguments when needed. Please put the 
> mentioned bug to See Also in the new report. Thanks.
> Marqeaux: Thanks for the advice. I will report it, along with my arguments. 
> Thank you too! ;)
> Marqeaux has disconnected: Quit: Ik ben weg! Tot ziens allemaal!

I still think that it's easy to make a smll tweak - just to make this
little thingie a bit bigger... and then do another tiny tweak to change that a
little... and so on. But if there's a problem with readability of some text
size that is used throughout our application, and there's no such problems in
other applications on that OS, then that means that either we use wrong text
size *universally* in our app (which means we don't get correct size from OS),
or (if we use proper size) that we render text wrong (blurry?). And tweaking
text size in one specific control to workaround is just wrong. A fix is needed,
but in a different place. Please don't enrich the codebase with hacks that
would make it a PITA later to find why something works differently and
unexpectedly.

And in general - that patch included *different* fixes - something that is
*never* a good thing, e.g. in case when you need to revert one change, but not
another.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list

[Libreoffice-bugs] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

--- Comment #2 from Mike Kaganski  ---
A transcript from #libreoffice IRC chat:

> Marqeaux has joined
> Marqeaux: Hi everybody. I want to ask a question about the formula bar in 
> Calc. The font size in the formula bar has been increased for better 
> readability. Is there a way to make the font size in the formula bar smaller 
> again, just like it was in previous versions of LibreOffice?
> Marqeaux: Oh, I'm using LibreOffice 6.3.0. I forgot to mention that piece of 
> information...
> quikee[m]: Marqeaux: interesting ... it's the same size for me
> Marqeaux: I got LibreOffice 6.3.0. on more machines installed. The formula 
> bar shows a bigger font (too big for my taste), and the formula bar is also 
> increased in height. It used to have the same height as the cell bar (where 
> you see in which cell you are)...
> Marqeaux: I prefer the situation like it was in LO 6.2. (the formula bar, I 
> mean...)
> Marqeaux: ;)
> mikekaganski: Marqeaux: I confirm font size increase in 6.3
> quikee[m]: maybe it manifests in windows. I was looking in Linux with gtk3 
> backend
> quikee[m]: we modified the scrollbars of the formula bar.. could be that has 
> influenced the size
> mikekaganski: quikee[m]: /me bisects
> Marqeaux: Is there a way in the expert configuration to alter the size of the 
> fonts in the formula bar, and also the height of the formula bar?
> quikee[m]: Marqeaux: no, I don't think so... increase is also not intentional 
> so it looks like you "found" a bug
> mikekaganski: Marqeaux: ... which needs a bug report
> mikekaganski: quikee[m]: 04b60370a73b79e3aa0a04bc7cff45dff1db6286
> Marqeaux: I made a screenshot of what I see (on multiple machines, not only 
> on Xubuntu): https://mupload.nl/img/edk3kknwn.png
> mikekaganski: Marqeaux: so it was intentional in the end: tdf#101443
> Marqeaux: I guess it's intentional, because more and more software follow the 
> GTK3 guidelines. I think it's awful, because GTK3 makes everything big and 
> bold, and with no borders. I looks like a "Fischer-Price" kinda of 
> approach
> mikekaganski: Marqeaux: it wasn't because of "GTK3" (and most LO users are on 
> Windows btw); it was because of the previous font was unreadable on some 
> macOS (not using GTK3 also)
> mikekaganski: Marqeaux: "In addition to inputline redesign, character size 
> has been increased to 120% of original size. This prevents very small / 
> unreadable inputline character sizes on some OS X resolutions"
> Marqeaux: Ah, alright. But anyway I takes up too much space now. It's 
> fine that the fonts in the formula bar are bigger, but I hope that there's a 
> way to make it "normal" again...
> mikekaganski: Marqeaux: the commit mentioned above has the min font size 
> fixed (16, strange size for UI actually). You may file an enhancement request 
> to make it user-configurable or smaller.
> Marqeaux: Yes, I also found it a very strange decission to make it that big. 
> It just doesn't fit into the GUI. I think it's better to make it 
> user-configurable, rather than to leave it "hard-coded". Although my eyes 
> aren't that good (I wear glasses and I'm middle aged), but I never had 
> trouble reading the formula bar as it was before.
> Marqeaux: Where can I file an enhancement request?
> mikekaganski: Marqeaux: I already wrote a comment there in the original bug. 
> I gave you the link to the bug, and that's the site where we write and track 
> reports.
> mikekaganski: Marqeaux: please file your request separately regardless of my 
> comment: it would help to provide arguments when needed. Please put the 
> mentioned bug to See Also in the new report. Thanks.
> Marqeaux: Thanks for the advice. I will report it, along with my arguments. 
> Thank you too! ;)
> Marqeaux has disconnected: Quit: Ik ben weg! Tot ziens allemaal!

I still think that it's easy to make a smll tweak - just to make this
little thingie a bit bigger... and then do another tiny tweak to change that a
little... and so on. But if there's a problem with readability of some text
size that is used throughout our application, and there's no such problems in
other applications on that OS, then that means that either we use wrong text
size *universally* in our app (which means we don't get correct size from OS),
or (if we use proper size) that we render text wrong (blurry?). And tweaking
text size in one specific control to workaround is just wrong. A fix is needed,
but in a different place. Please don't enrich the codebase with hacks that
would make it a PITA later to find why something works differently and
unexpectedly.

And in general - that patch included *different* fixes - something that is
*never* a good thing, e.g. in case when you need to revert one change, but not
another.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list

[Libreoffice-bugs] [Bug 125696] Introduce CSS preprocessor

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125696

Buovjaga  changed:

   What|Removed |Added

 CC||todven...@suomi24.fi

--- Comment #5 from Buovjaga  ---
I don't think there is much need for a preprocessor nowadays. Responding to
some points in the raygun.com blog:

1. https://caniuse.com/#feat=css-variables

2. @import causing multiple requests is a non-problem in the age of HTTP/2:
https://discourse.wicg.io/t/is-css-import-still-considered-as-an-anti-pattern/1967/5

3. Whole article in general, but as a comment to mixins:
https://cathydutton.co.uk/posts/why-i-stopped-using-sass/
"CSS custom properties can do more than just replace pre processor variables,
they have the added bonus of being updated dynamically (instead of compiled at
build). This allows values to be altered at run time with Javascript and to be
scoped without the need for mixins and functions to outwit the cascade."

5. https://caniuse.com/#feat=calc

Postprocessing [1] might be more useful, but it is always a danger to introduce
more moving parts and complicate the build process.

[1] https://postcss.org/

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: dbaccess/source

2019-08-22 Thread Caolán McNamara (via logerrit)
 dbaccess/source/ui/dlg/dbwiz.cxx|   70 +++-
 dbaccess/source/ui/inc/dbwiz.hxx|5 --
 dbaccess/source/ui/uno/DBTypeWizDlg.cxx |4 -
 3 files changed, 38 insertions(+), 41 deletions(-)

New commits:
commit 055c249847548a5c28ab5b29f3603ffbe74fe1f0
Author: Caolán McNamara 
AuthorDate: Thu Aug 22 16:01:01 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 18:31:16 2019 +0200

weld ODbTypeWizDialog

Change-Id: Ie85e8f7a9a7551078e3108e7be9747ae4501d5eb
Reviewed-on: https://gerrit.libreoffice.org/77966
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/dbaccess/source/ui/dlg/dbwiz.cxx b/dbaccess/source/ui/dlg/dbwiz.cxx
index d36da8d5c0e0..9e6fef02f0e6 100644
--- a/dbaccess/source/ui/dlg/dbwiz.cxx
+++ b/dbaccess/source/ui/dlg/dbwiz.cxx
@@ -62,14 +62,11 @@ using namespace com::sun::star::container;
 #define ADDITIONAL_PAGE_MYSQL_NATIVE   12
 
 // ODbTypeWizDialog
-ODbTypeWizDialog::ODbTypeWizDialog(vcl::Window* _pParent
-   ,SfxItemSet const * _pItems
-   ,const Reference< XComponentContext >& _rxORB
-   ,const css::uno::Any& _aDataSourceName
-   )
-:OWizardMachine(_pParent, WizardButtonFlags::NEXT | 
WizardButtonFlags::PREVIOUS | WizardButtonFlags::FINISH | 
WizardButtonFlags::CANCEL | WizardButtonFlags::HELP )
+ODbTypeWizDialog::ODbTypeWizDialog(weld::Window* _pParent, SfxItemSet const * 
_pItems,
+   const Reference< XComponentContext >& 
_rxORB, const css::uno::Any& _aDataSourceName)
+: WizardMachine(_pParent, WizardButtonFlags::NEXT | 
WizardButtonFlags::PREVIOUS | WizardButtonFlags::FINISH | 
WizardButtonFlags::CANCEL | WizardButtonFlags::HELP )
 {
-m_pImpl.reset(new ODbDataSourceAdministrationHelper(_rxORB,GetFrameWeld(), 
_pParent ? _pParent->GetFrameWeld() : nullptr, this));
+m_pImpl.reset(new ODbDataSourceAdministrationHelper(_rxORB, 
m_xAssistant.get(), _pParent, this));
 m_pImpl->setDataSourceOrName(_aDataSourceName);
 Reference< XPropertySet > xDatasource = m_pImpl->getCurrentDataSource();
 m_pOutSet.reset(new SfxItemSet( *_pItems->GetPool(), _pItems->GetRanges() 
));
@@ -77,15 +74,14 @@ ODbTypeWizDialog::ODbTypeWizDialog(vcl::Window* _pParent
 m_pImpl->translateProperties(xDatasource, *m_pOutSet);
 m_eType = 
dbaui::ODbDataSourceAdministrationHelper::getDatasourceType(*m_pOutSet);
 
-SetPageSizePixel(LogicToPixel(::Size(PAGE_X, PAGE_Y), 
MapMode(MapUnit::MapAppFont)));
 defaultButton(WizardButtonFlags::NEXT);
 enableButtons(WizardButtonFlags::FINISH, false);
 enableAutomaticNextButtonState();
 
-m_pPrevPage->SetHelpId(HID_DBWIZ_PREVIOUS);
-m_pNextPage->SetHelpId(HID_DBWIZ_NEXT);
-m_pCancel->SetHelpId(HID_DBWIZ_CANCEL);
-m_pFinish->SetHelpId(HID_DBWIZ_FINISH);
+m_xPrevPage->set_help_id(HID_DBWIZ_PREVIOUS);
+m_xNextPage->set_help_id(HID_DBWIZ_NEXT);
+m_xCancel->set_help_id(HID_DBWIZ_CANCEL);
+m_xFinish->set_help_id(HID_DBWIZ_FINISH);
 // no local resources needed anymore
 
 const DbuTypeCollectionItem& rCollectionItem = dynamic_cast(*_pItems->GetItem(DSID_TYPECOLLECTION));
@@ -93,17 +89,12 @@ ODbTypeWizDialog::ODbTypeWizDialog(vcl::Window* _pParent
 
 ActivatePage();
 setTitleBase(DBA_RES(STR_DATABASE_TYPE_CHANGE));
-}
 
-ODbTypeWizDialog::~ODbTypeWizDialog()
-{
-disposeOnce();
+m_xAssistant->set_current_page(0);
 }
 
-void ODbTypeWizDialog::dispose()
+ODbTypeWizDialog::~ODbTypeWizDialog()
 {
-m_pOutSet.reset();
-vcl::OWizardMachine::dispose();
 }
 
 IMPL_LINK(ODbTypeWizDialog, OnTypeSelected, OGeneralPage&, _rTabPage, void)
@@ -232,50 +223,56 @@ VclPtr ODbTypeWizDialog::createPage(WizardState 
_nState)
 {
 const char* pStringId = STR_PAGETITLE_ADVANCED;
 VclPtr pPage;
+
+OString sIdent(OString::number(_nState));
+weld::Container* pPageContainer = m_xAssistant->append_page(sIdent);
+// TODO eventually pass DialogController as distinct argument instead of 
bundling into TabPageParent
+TabPageParent aParent(pPageContainer, this);
+
 switch(_nState)
 {
 case START_PAGE: // start state
 {
-pPage = VclPtr::Create(this,*m_pOutSet);
+pPage = VclPtr::Create(aParent, *m_pOutSet);
 OGeneralPage* pGeneralPage = static_cast< OGeneralPage* >( 
pPage.get() );
 pGeneralPage->SetTypeSelectHandler( LINK( this, ODbTypeWizDialog, 
OnTypeSelected));
 pStringId = STR_PAGETITLE_GENERAL;
 }
 break;
 case CONNECTION_PAGE:
-pPage = OConnectionTabPage::Create(this,m_pOutSet.get());
+pPage = OConnectionTabPage::Create(aParent, m_pOutSet.get());
 pStringId = STR_PAGETITLE_CONNECTION;
 break;
 
 case ADDITIONAL_PAGE_DBASE:
-pPage = 

[Libreoffice-commits] core.git: chart2/source chart2/uiconfig dbaccess/source dbaccess/uiconfig dbaccess/UIConfig_dbaccess.mk include/vcl vcl/source vcl/uiconfig vcl/UIConfig_vcl.mk vcl/unx

2019-08-22 Thread Caolán McNamara (via logerrit)
 chart2/source/controller/dialogs/dlg_CreationWizard.cxx |   51 +---
 chart2/source/controller/dialogs/dlg_CreationWizard_UNO.cxx |   69 
 chart2/source/controller/dialogs/dlg_DataSource.cxx |4 
 chart2/source/controller/dialogs/tp_DataSource.cxx  |   24 
 chart2/source/controller/dialogs/tp_DataSource.hxx  |3 
 chart2/source/controller/dialogs/tp_RangeChooser.cxx|   24 
 chart2/source/controller/dialogs/tp_RangeChooser.hxx|4 
 chart2/source/controller/inc/dlg_CreationWizard.hxx |6 -
 chart2/source/controller/inc/dlg_CreationWizard_UNO.hxx |4 
 chart2/uiconfig/ui/tp_RangeChooser.ui   |1 
 chart2/uiconfig/ui/wizelementspage.ui   |4 
 dbaccess/UIConfig_dbaccess.mk   |1 
 dbaccess/source/ui/dlg/dbwizsetup.cxx   |9 +
 dbaccess/source/ui/inc/dbwizsetup.hxx   |1 
 dbaccess/source/ui/uno/DBTypeWizDlgSetup.cxx|5 
 include/vcl/roadmapwizard.hxx   |2 
 include/vcl/weld.hxx|2 
 vcl/UIConfig_vcl.mk |1 
 vcl/source/app/salvtables.cxx   |5 
 vcl/source/control/roadmapwizard.cxx|6 -
 vcl/source/control/wizardmachine.cxx|4 
 vcl/uiconfig/ui/wizard.ui   |5 
 vcl/unx/gtk3/gtk3gtkinst.cxx|   17 ++
 23 files changed, 112 insertions(+), 140 deletions(-)

New commits:
commit d661cb17610af93b1607025284f23187c6d2fc06
Author: Caolán McNamara 
AuthorDate: Thu Aug 22 11:15:57 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 18:26:41 2019 +0200

weld chart CreationWizard

Change-Id: I7ee88c1e1b0eca5c2510a4bbba8b60c0d8a4e3e8
Reviewed-on: https://gerrit.libreoffice.org/77954
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/chart2/source/controller/dialogs/dlg_CreationWizard.cxx 
b/chart2/source/controller/dialogs/dlg_CreationWizard.cxx
index db54937be38c..58a744980556 100644
--- a/chart2/source/controller/dialogs/dlg_CreationWizard.cxx
+++ b/chart2/source/controller/dialogs/dlg_CreationWizard.cxx
@@ -22,6 +22,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "tp_ChartType.hxx"
 #include "tp_RangeChooser.hxx"
@@ -30,9 +31,6 @@
 #include "ChartTypeTemplateProvider.hxx"
 #include "DialogModel.hxx"
 
-#define CHART_WIZARD_PAGEWIDTH  250
-#define CHART_WIZARD_PAGEHEIGHT 170
-
 using namespace css;
 
 namespace chart
@@ -45,14 +43,14 @@ namespace chart
 #define STATE_OBJECTS  3
 #define STATE_LAST STATE_OBJECTS
 
-CreationWizard::CreationWizard(vcl::Window* pParent, const 
uno::Reference& xChartModel,
+CreationWizard::CreationWizard(weld::Window* pParent, const 
uno::Reference& xChartModel,
const uno::Reference& 
xContext)
-: vcl::RoadmapWizard(pParent)
-, m_xChartModel(xChartModel,uno::UNO_QUERY)
-, m_xComponentContext(xContext)
-, m_pTemplateProvider(nullptr)
-, m_aTimerTriggeredControllerLock(xChartModel)
-, m_bCanTravel(true)
+: vcl::RoadmapWizardMachine(pParent)
+, m_xChartModel(xChartModel,uno::UNO_QUERY)
+, m_xComponentContext(xContext)
+, m_pTemplateProvider(nullptr)
+, m_aTimerTriggeredControllerLock(xChartModel)
+, m_bCanTravel(true)
 {
 m_pDialogModel.reset(new DialogModel(m_xChartModel, m_xComponentContext));
 defaultButton(WizardButtonFlags::FINISH);
@@ -69,12 +67,6 @@ CreationWizard::CreationWizard(vcl::Window* pParent, const 
uno::ReferencegetModel().isDataFromSpreadsheet())
 {
@@ -84,6 +76,16 @@ CreationWizard::CreationWizard(vcl::Window* pParent, const 
uno::Referenceset_current_page(0);
+}
+
+short CreationWizard::run()
+{
+int nRet = vcl::RoadmapWizardMachine::run();
+if (nRet == static_cast(WizardButtonFlags::FINISH))
+return ui::dialogs::ExecutableDialogResults::OK;
+return ui::dialogs::ExecutableDialogResults::CANCEL;
 }
 
 CreationWizard::~CreationWizard() = default;
@@ -91,12 +93,18 @@ CreationWizard::~CreationWizard() = default;
 VclPtr CreationWizard::createPage(WizardState nState)
 {
 VclPtr pRet;
+
+OString sIdent(OString::number(nState));
+weld::Container* pPageContainer = m_xAssistant->append_page(sIdent);
+// TODO eventually pass DialogController as distinct argument instead of 
bundling into TabPageParent
+TabPageParent aParent(pPageContainer, this);
+
 switch( nState )
 {
 case STATE_CHARTTYPE:
 {
 m_aTimerTriggeredControllerLock.startTimer();
-VclPtrInstance pChartTypeTabPage(this,m_xChartModel);
+VclPtrInstance pChartTypeTabPage(aParent, 

[Libreoffice-bugs] [Bug 127099] CppUnitTest_sc_dataprovider fails on ppc64le

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127099

--- Comment #4 from Jonathan Brielmaier  ---
How could I/we further debug this? So you assume it's a compiler bug (on
ppc64le). Did I understand that correctly?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: package/source

2019-08-22 Thread Arkadiy Illarionov (via logerrit)
 package/source/manifest/ManifestExport.cxx |  105 +++
 package/source/xstor/ocompinstream.cxx |  109 +++
 package/source/xstor/owriteablestream.cxx  |  345 ++---
 package/source/xstor/xfactory.cxx  |   27 +
 package/source/xstor/xstorage.cxx  |  277 
 package/source/zippackage/ZipPackage.cxx   |  114 +++-
 package/source/zippackage/ZipPackageStream.cxx |6 
 package/source/zippackage/zipfileaccess.cxx|4 
 8 files changed, 441 insertions(+), 546 deletions(-)

New commits:
commit 38001018fa06f721cf87edae923b54ce9a5ab5a7
Author: Arkadiy Illarionov 
AuthorDate: Sat Aug 10 18:34:33 2019 +0300
Commit: Arkadiy Illarionov 
CommitDate: Thu Aug 22 18:03:16 2019 +0200

Simplify Sequence iterations in package

Use range-based loops, STL and comphelper functions

Change-Id: Ibd836b9b2df2f30b42f2d7a621188d78f5b53196
Reviewed-on: https://gerrit.libreoffice.org/77246
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 
Reviewed-by: Arkadiy Illarionov 

diff --git a/package/source/manifest/ManifestExport.cxx 
b/package/source/manifest/ManifestExport.cxx
index f8c2d0399d4f..4b4d57ce9294 100644
--- a/package/source/manifest/ManifestExport.cxx
+++ b/package/source/manifest/ManifestExport.cxx
@@ -111,33 +111,30 @@ ManifestExport::ManifestExport( uno::Reference< 
xml::sax::XDocumentHandler > con
 const OUString  sPGP_Name( PGP_NAME );
 
 ::comphelper::AttributeList * pRootAttrList = new 
::comphelper::AttributeList;
-const uno::Sequence < beans::PropertyValue > *pSequence = 
rManList.getConstArray();
-const sal_uInt32 nManLength = rManList.getLength();
 
 // find the mediatype of the document if any
 OUString aDocMediaType;
 OUString aDocVersion;
-sal_Int32 nRootFolderPropIndex=-1;
-for (sal_uInt32 nInd = 0; nInd < nManLength ; nInd++ )
+const uno::Sequence* pRootFolderPropSeq = nullptr;
+for (const uno::Sequence < beans::PropertyValue >& rSequence : rManList)
 {
 OUString aMediaType;
 OUString aPath;
 OUString aVersion;
 
-const beans::PropertyValue *pValue = pSequence[nInd].getConstArray();
-for (sal_uInt32 j = 0, nNum = pSequence[nInd].getLength(); j < nNum; 
j++, pValue++)
+for (const beans::PropertyValue& rValue : rSequence)
 {
-if (pValue->Name == sMediaTypeProperty )
+if (rValue.Name == sMediaTypeProperty )
 {
-pValue->Value >>= aMediaType;
+rValue.Value >>= aMediaType;
 }
-else if (pValue->Name == sFullPathProperty )
+else if (rValue.Name == sFullPathProperty )
 {
-pValue->Value >>= aPath;
+rValue.Value >>= aPath;
 }
-else if (pValue->Name == sVersionProperty )
+else if (rValue.Name == sVersionProperty )
 {
-pValue->Value >>= aVersion;
+rValue.Value >>= aVersion;
 }
 
 if ( !aPath.isEmpty() && !aMediaType.isEmpty() && 
!aVersion.isEmpty() )
@@ -148,7 +145,7 @@ ManifestExport::ManifestExport( uno::Reference< 
xml::sax::XDocumentHandler > con
 {
 aDocMediaType = aMediaType;
 aDocVersion = aVersion;
-nRootFolderPropIndex = nInd;
+pRootFolderPropSeq = 
 break;
 }
 }
@@ -217,15 +214,13 @@ ManifestExport::ManifestExport( uno::Reference< 
xml::sax::XDocumentHandler > con
 xHandler->startElement( sManifestElement, xRootAttrList );
 
 const uno::Any *pKeyInfoProperty = nullptr;
-if ( nRootFolderPropIndex >= 0 )
+if ( pRootFolderPropSeq )
 {
 // do we have package-wide encryption info?
-const beans::PropertyValue *pValue =
-pSequence[nRootFolderPropIndex].getConstArray();
-for (sal_uInt32 j = 0, nNum = 
pSequence[nRootFolderPropIndex].getLength(); j < nNum; j++, pValue++)
+for (const beans::PropertyValue& rValue : *pRootFolderPropSeq)
 {
-if (pValue->Name == sKeyInfo )
-pKeyInfoProperty = >Value;
+if (rValue.Name == sKeyInfo )
+pKeyInfoProperty = 
 }
 
 if ( pKeyInfoProperty )
@@ -242,22 +237,19 @@ ManifestExport::ManifestExport( uno::Reference< 
xml::sax::XDocumentHandler > con
 
 uno::Sequence< uno::Sequence < beans::NamedValue > > 
aKeyInfoSequence;
 *pKeyInfoProperty >>= aKeyInfoSequence;
-const uno::Sequence < beans::NamedValue > *pKeyInfoSequence = 
aKeyInfoSequence.getConstArray();
-const sal_uInt32 nKeyInfoLength = aKeyInfoSequence.getLength();
-for (sal_uInt32 nInd = 0; nInd < nKeyInfoLength ; nInd++ )
+for (const uno::Sequence& rKeyInfoSequence : 
std::as_const(aKeyInfoSequence))
 {
 uno::Sequence 

[Libreoffice-bugs] [Bug 127103] New: Typo in tip of the day

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127103

Bug ID: 127103
   Summary: Typo in tip of the day
   Product: LibreOffice
   Version: 6.3.0.0.alpha1+
  Hardware: All
OS: Windows (All)
Status: UNCONFIRMED
  Severity: trivial
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: emo_asse...@yahoo.com

Created attachment 153584
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153584=edit
screenshot

Install the product on a clean machine where there has not been any previous
installation
Open Calc

Issue: Grammar/typo in tip of the day
"To temporarily starts with a fresh user profile"

Should be 
"To temporarily start with a fresh user profile"

Version: 6.3.0.4 (x64)
Build ID: 057fc023c990d676a43019934386b85b21a9ee99
CPU threads: 1; OS: Windows 6.3; UI render: default; VCL: win; 
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: sw/qa

2019-08-22 Thread Tamás Zolnai (via logerrit)
 sw/qa/extras/ooxmlexport/ooxmlexport13.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3a2eeafc30b6b6cf3dafc6d3b94144ba54a8b48b
Author: Tamás Zolnai 
AuthorDate: Thu Aug 22 15:55:54 2019 +0200
Commit: Tamás Zolnai 
CommitDate: Thu Aug 22 17:37:32 2019 +0200

This is an export test.

Change-Id: Idad907f192c7a4e536bd1b8ca899fc6f9d55bbdf
Reviewed-on: https://gerrit.libreoffice.org/77962
Tested-by: Jenkins
Reviewed-by: Tamás Zolnai 

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
index 5666f4fae730..f8bafc76d47c 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
@@ -502,7 +502,7 @@ DECLARE_OOXMLEXPORT_EXPORTONLY_TEST(testTdf121663, 
"tdf121663.docx")
 assertXPath(pXmlDoc, "//w:lnNumType", "distance", "283");
 }
 
-DECLARE_OOXMLIMPORT_TEST(testInvalidDateFormField, 
"invalid_date_form_field.docx")
+DECLARE_OOXMLEXPORT_TEST(testInvalidDateFormField, 
"invalid_date_form_field.docx")
 {
 
 SwXTextDocument* pTextDoc = dynamic_cast(mxComponent.get());
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 127102] FILEOPEN Combined Chart of type stacked area + stacked column loses column stacking

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127102

--- Comment #1 from Gabor Kelemen  ---
Created attachment 153583
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153583=edit
Screenshot of the original document side by side in Word and Writer

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127102] New: FILEOPEN Combined Chart of type stacked area + stacked column loses column stacking

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127102

Bug ID: 127102
   Summary: FILEOPEN Combined Chart of type stacked area + stacked
column loses column stacking
   Product: LibreOffice
   Version: 6.4.0.0.alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Chart
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: kelem...@ubuntu.com
CC: libreoff...@nisz.hu, nem...@numbertext.org
Blocks: 75057

Created attachment 153582
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153582=edit
Example file from Word

Opening a simple combined chart with stacked area and stacked columns loses the
stacked setting for the columns, displaying them grouped.


Steps to reproduce:
1.  Insert a chart in Word
2.  Change its type to Combined, select one data series to be stacked area
and two to be stacked column
3.  Save as docx, open it in Writer

Actual results:
The column components appear grouped instead of stacked.

Expected results:
Stacking remains unchanged.

LibreOffice details:
Version: 6.4.0.0.alpha0+ (x86)
Build ID: 382aceb80d3c0f420d51762e1f88eee2334ff11b
CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-07-21_23:45:18
Locale: hu-HU (hu_HU); UI-Language: en-US
Calc: CL


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=75057
[Bug 75057] [META] OOXML Chart object issues
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 75057] [META] OOXML Chart object issues

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=75057

Gabor Kelemen  changed:

   What|Removed |Added

 Depends on||127102


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=127102
[Bug 127102] FILEOPEN Combined Chart of type stacked area + stacked column
loses column stacking
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: oox/source sw/qa

2019-08-22 Thread Tamás Zolnai (via logerrit)
 oox/source/export/drawingml.cxx  |   10 +++---
 sw/qa/extras/ooxmlexport/data/tdf127085.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport13.cxx   |7 +++
 3 files changed, 14 insertions(+), 3 deletions(-)

New commits:
commit 505cbb9c3d2771a12c989515663cc1eb73dd0c2f
Author: Tamás Zolnai 
AuthorDate: Thu Aug 22 15:57:41 2019 +0200
Commit: Tamás Zolnai 
CommitDate: Thu Aug 22 17:21:19 2019 +0200

tdf#127085: FILESAVE: Transparency is lost after RT

Change-Id: I22ba83b5cc00f84112a3755898ee2be58337afd6
Reviewed-on: https://gerrit.libreoffice.org/77963
Tested-by: Jenkins
Reviewed-by: Tamás Zolnai 

diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index d4b35c8988f6..6f975a298e50 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -1274,14 +1274,18 @@ void 
DrawingML::WriteImageBrightnessContrastTransparence(uno::Reference();
 if (GetProperty(rXPropSet, "AdjustContrast"))
 nContrast = mAny.get();
-if (GetProperty(rXPropSet, "Transparency"))
-nTransparence = mAny.get();
+// Used for shapes with picture fill
+if (GetProperty(rXPropSet, "FillTransparence"))
+nTransparence = mAny.get();
+// Used for pictures
+if (nTransparence == 0 && GetProperty(rXPropSet, "Transparency"))
+nTransparence = static_cast(mAny.get());
 
 
 if (nBright || nContrast)
diff --git a/sw/qa/extras/ooxmlexport/data/tdf127085.docx 
b/sw/qa/extras/ooxmlexport/data/tdf127085.docx
new file mode 100644
index ..65556fe38abd
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/tdf127085.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
index e59e65e19312..5666f4fae730 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
@@ -579,6 +579,13 @@ DECLARE_OOXMLIMPORT_TEST(testInvalidDateFormField, 
"invalid_date_form_field.docx
 CPPUNIT_ASSERT_EQUAL(int(3), nIndex);
 }
 
+DECLARE_OOXMLEXPORT_TEST(tdf127085, "tdf127085.docx")
+{
+// Fill transparency was lost during export
+uno::Reference xShape(getShape(1), uno::UNO_QUERY);
+CPPUNIT_ASSERT_EQUAL(sal_Int16(50), getProperty(xShape, 
"FillTransparence"));
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 127101] EDITING Rejecting direct formatting removes former direct formatting

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127101

--- Comment #2 from Gabor Kelemen  ---
Created attachment 153580
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153580=edit
The example file with changed character formatting and joined paragraphs

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127101] EDITING Rejecting direct formatting removes former direct formatting

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127101

--- Comment #3 from Gabor Kelemen  ---
Created attachment 153581
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153581=edit
The example file after rejecting the changes

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127101] EDITING Rejecting direct formatting removes former direct formatting

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127101

--- Comment #1 from Gabor Kelemen  ---
Created attachment 153579
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153579=edit
The example file with character formatting and paragraph formatting applied

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 83946] [META] Tracking changes issues

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=83946

Gabor Kelemen  changed:

   What|Removed |Added

 Depends on||127101


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=127101
[Bug 127101] EDITING Rejecting direct formatting removes former direct
formatting
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127101] New: EDITING Rejecting direct formatting removes former direct formatting

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127101

Bug ID: 127101
   Summary: EDITING Rejecting direct formatting removes former
direct formatting
   Product: LibreOffice
   Version: 6.4.0.0.alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: kelem...@ubuntu.com
CC: libreoff...@nisz.hu, nem...@numbertext.org
Blocks: 83946

Created attachment 153578
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153578=edit
Example file from Writer

If a paragraph has direct text formatting then we add another formatting as
tracked change and reject it, then the text formatting is reset to the default
settings instead of the formatting set before the tracked format change.
The same happens when we reject a paragraph join that was consequence of a
deletion and has modified the paragraph style. The second paragraph gets the
style of the first one, but upon rejecting the change does not get its former
style back.


Steps to reproduce:
1. Open attached file. It has some character formatting set for the first
paragraph: bold, underline, superscript, highlight color, hyperlink. Change
tracking is enabled.
2. Set the whole first paragraph as underlined.
3. Reject this change.

For the paragraph join part:
1. Delete the end of the paragraph formatted as Heading 1 and the beginning
of the paragraph formatted as Quotations
2. The style of the second paragraph becomes Heading 1
3. Reject this change. The second paragraphs style remains Heading 1
instead of reverting to Quotations.

Actual results:
The original character format settings are not restored.

Expected results:
Rejecting Character formatting should restore former character formatting.

LibreOffice details:
Version: 6.4.0.0.alpha0+ (x64)
Build ID: 3e64065612acec2eb29aa21e2b515953422256d7
CPU threads: 4; OS: Windows 6.3; UI render: GL; VCL: win; 
TinderBox: Win-x86_64@62-TDF, Branch:master, Time: 2019-08-15_22:57:26
Locale: hu-HU (hu_HU); UI-Language: en-US
Calc: CL


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=83946
[Bug 83946] [META] Tracking changes issues
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127100] New: Formula font size cannot be changed when using new Tabbed UI in Impress

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127100

Bug ID: 127100
   Summary: Formula font size cannot be changed when using new
Tabbed UI in Impress
   Product: LibreOffice
   Version: 6.3.0.4 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Formula Editor
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: 2xb.cod...@wwu.de

Open LibreOffice Impress, enable Tabbed UI (View -> User Interface -> Tabbed),
add a formula and try changing the font size of that formula (i.e. navigate to
the tab "Home" and change the font size). The Tab UI is then greyed out instead
of the font size being changed.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 122218] After Update to 6.1.4 on macOS fonts are blurred on retina display

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122218

--- Comment #33 from di...@gelios.net ---
Fonts are not blurry with this build for me.

https://dev-builds.libreoffice.org/daily/libreoffice-6-3/MacOSX-x86_64@49-TDF/2019-08-21_17.28.20/libreoffice-6-3~2019-08-21_17.28.20_LibreOfficeDev_6.3.2.0.0_MacOS_x86-64.dmg

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127099] CppUnitTest_sc_dataprovider fails on ppc64le

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127099

--- Comment #3 from Noel Grandin  ---
This looks like some kind of shutdown/C++ destructor problem, probably because
the order of destruction of C++ global objects is undefined, and the order
slightly different on your machine, and some global object is attempting to
access another global object during  shutdown, but the accessee is already
dead.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127099] CppUnitTest_sc_dataprovider fails on ppc64le

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127099

--- Comment #2 from Caolán McNamara  ---
yeah, crash on cleanup of globals on exit, I doubt its a specific LibreOffice
bug personally.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Re: Reverting patch

2019-08-22 Thread Regina Henschel

Hi Michael,

Michael Weghorn schrieb am 22-Aug-19 um 13:21:

Hi Regina,

On 22/08/2019 13.12, Regina Henschel wrote:

Hi Michael,

I need help with some basics.

Michael Weghorn schrieb am 22-Aug-19 um 08:41:


If you want to explicitly test this actually works as expected, you can
build the respective branch


How to "build the respective branch" ?


After you have checked out the gerrit patch, you can build it the same
way as you build master, i.e. run './autogen.sh' and then 'make'.

The checkout will be the branch that the gerrit patch has been
cherry-picked to, e.g. the libreoffice-6-3 branch for [1] + your patch
on top, so you can test whether this works as expected for LibreOffice 6.3.


Thank you. I have realized now, that I have to use "check-out" and not 
"cherry-pick". I have finally got a build 6.3.2.0 and have tested the patch.


Kind regards
Regina
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice

[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - vcl/inc vcl/qt5 vcl/unx

2019-08-22 Thread Jan-Marek Glogowski (via logerrit)
 vcl/inc/qt5/Qt5DragAndDrop.hxx  |   21 +
 vcl/inc/qt5/Qt5Frame.hxx|   10 +-
 vcl/inc/qt5/Qt5Tools.hxx|3 
 vcl/inc/qt5/Qt5Transferable.hxx |2 
 vcl/inc/qt5/Qt5Widget.hxx   |2 
 vcl/qt5/Qt5DragAndDrop.cxx  |   93 
 vcl/qt5/Qt5Frame.cxx|  153 
 vcl/qt5/Qt5Tools.cxx|   12 +++
 vcl/qt5/Qt5Widget.cxx   |   32 +---
 vcl/unx/kde5/KDE5SalFrame.hxx   |4 -
 10 files changed, 174 insertions(+), 158 deletions(-)

New commits:
commit 658ab1db3abce8941352d8d9def1468496985a60
Author: Jan-Marek Glogowski 
AuthorDate: Thu Aug 8 17:59:20 2019 +
Commit: Michael Weghorn 
CommitDate: Thu Aug 22 16:38:04 2019 +0200

tdf#126560 Qt5 fix D'n'D key-modifier handling

The patch has still one problem: the key-modifier state isn't
reflected by the cursor, unless the user moves the mouse. There is
an upstream Qt bug, reported in 2016-09 against Qt 5.6.1! It is
supposed to be fixed in Qt 5.12, according to the bug report at
https://bugreports.qt.io/browse/QTBUG-56218, which is still open.

I thought about adding a configure test, but I couldn't imagine
any realistic way to write it. And after Michael Weghorn found the
bug is actually not fixed, as claimed in one of the comments, I
decided to drop the warning.

Change-Id: Ice8ebc4ea149282b4c1551e755efe3d4856cf782
Reviewed-on: https://gerrit.libreoffice.org/77174
Reviewed-by: Michael Weghorn 
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski 
(cherry picked from commit 3355be0616c24c5e44b71e7623c4191ed9c69074)
Reviewed-on: https://gerrit.libreoffice.org/77956

diff --git a/vcl/inc/qt5/Qt5DragAndDrop.hxx b/vcl/inc/qt5/Qt5DragAndDrop.hxx
index dcd6cb5e4048..0ec9ce5bbcb2 100644
--- a/vcl/inc/qt5/Qt5DragAndDrop.hxx
+++ b/vcl/inc/qt5/Qt5DragAndDrop.hxx
@@ -25,7 +25,6 @@ class Qt5DragSource
 osl::Mutex m_aMutex;
 Qt5Frame* m_pFrame;
 css::uno::Reference 
m_xListener;
-css::uno::Reference m_xTrans;
 
 public:
 Qt5DragSource()
@@ -55,17 +54,7 @@ public:
 
 css::uno::Sequence SAL_CALL getSupportedServiceNames() override;
 
-void dragFailed();
-void fire_dragEnd(sal_Int8 nAction);
-
-static Qt5DragSource* m_ActiveDragSource;
-static bool m_bDropSuccessSet;
-static bool m_bDropSuccess;
-
-css::uno::Reference const& 
GetTransferable() const
-{
-return m_xTrans;
-}
+void fire_dragEnd(sal_Int8 nAction, bool bSuccessful);
 };
 
 class Qt5DropTarget
@@ -76,11 +65,11 @@ class Qt5DropTarget
 {
 osl::Mutex m_aMutex;
 Qt5Frame* m_pFrame;
-sal_Int8 mnDragAction;
-sal_Int8 mnDropAction;
+sal_Int8 m_nDropAction;
 bool m_bActive;
 sal_Int8 m_nDefaultActions;
 
std::vector> 
m_aListeners;
+bool m_bDropSuccessful;
 
 public:
 Qt5DropTarget();
@@ -115,10 +104,12 @@ public:
 css::uno::Sequence SAL_CALL getSupportedServiceNames() override;
 
 void fire_dragEnter(const 
css::datatransfer::dnd::DropTargetDragEnterEvent& dtde);
+void fire_dragExit(const css::datatransfer::dnd::DropTargetEvent& dte);
 void fire_dragOver(const css::datatransfer::dnd::DropTargetDragEnterEvent& 
dtde);
 void fire_drop(const css::datatransfer::dnd::DropTargetDropEvent& dtde);
 
-sal_Int8 proposedDragAction() const { return mnDragAction; }
+sal_Int8 proposedDropAction() const { return m_nDropAction; }
+bool dropSuccessful() const { return m_bDropSuccessful; }
 };
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/vcl/inc/qt5/Qt5Frame.hxx b/vcl/inc/qt5/Qt5Frame.hxx
index 4c94b846bc93..e296ce92db29 100644
--- a/vcl/inc/qt5/Qt5Frame.hxx
+++ b/vcl/inc/qt5/Qt5Frame.hxx
@@ -56,6 +56,8 @@ class Qt5MainWindow;
 class Qt5Menu;
 class Qt5SvpGraphics;
 
+class QDragMoveEvent;
+class QDropEvent;
 class QImage;
 class QMimeData;
 class QPaintDevice;
@@ -159,10 +161,10 @@ public:
 virtual void deregisterDragSource(Qt5DragSource const* pDragSource);
 virtual void registerDropTarget(Qt5DropTarget* pDropTarget);
 virtual void deregisterDropTarget(Qt5DropTarget const* pDropTarget);
-void draggingStarted(const int x, const int y, Qt::DropActions eActions,
- Qt::KeyboardModifiers eKeyMod, const QMimeData* 
pQMimeData);
-void dropping(const int x, const int y, Qt::KeyboardModifiers eKeyMod,
-  const QMimeData* pQMimeData);
+
+void handleDragLeave();
+void handleDragMove(QDragMoveEvent* pEvent);
+void handleDrop(QDropEvent* pEvent);
 
 virtual void SetExtendedFrameStyle(SalExtStyle nExtStyle) override;
 virtual void Show(bool bVisible, bool bNoActivate = false) override;
diff --git a/vcl/inc/qt5/Qt5Tools.hxx b/vcl/inc/qt5/Qt5Tools.hxx
index 6b1fb1adcc7e..697b703e28f4 100644
--- a/vcl/inc/qt5/Qt5Tools.hxx
+++ b/vcl/inc/qt5/Qt5Tools.hxx
@@ -70,6 +70,7 @@ inline QColor toQColor(const Color& rColor)
 

[Libreoffice-commits] core.git: cui/source dbaccess/source dbaccess/uiconfig dbaccess/UIConfig_dbaccess.mk include/svtools include/tools include/vcl vcl/source vcl/unx

2019-08-22 Thread Caolán McNamara (via logerrit)
 cui/source/dialogs/about.cxx  |   28 -
 cui/source/inc/about.hxx  |   11 
 dbaccess/UIConfig_dbaccess.mk |1 
 dbaccess/source/ui/dlg/adminpages.cxx |4 
 dbaccess/source/ui/dlg/dbwizsetup.cxx |   88 +--
 dbaccess/source/ui/dlg/generalpage.cxx|   44 -
 dbaccess/source/ui/dlg/generalpage.hxx|6 
 dbaccess/source/ui/dlg/sqlmessage.cxx |2 
 dbaccess/source/ui/inc/dbwizsetup.hxx |5 
 dbaccess/source/ui/uno/DBTypeWizDlgSetup.cxx  |6 
 dbaccess/uiconfig/ui/authentificationpage.ui  |4 
 dbaccess/uiconfig/ui/databasewizard.ui|   18 
 dbaccess/uiconfig/ui/dbasepage.ui |4 
 dbaccess/uiconfig/ui/dbwizconnectionpage.ui   |5 
 dbaccess/uiconfig/ui/dbwizmysqlintropage.ui   |5 
 dbaccess/uiconfig/ui/dbwizmysqlnativepage.ui  |8 
 dbaccess/uiconfig/ui/dbwizspreadsheetpage.ui  |   39 -
 dbaccess/uiconfig/ui/dbwiztextpage.ui |8 
 dbaccess/uiconfig/ui/finalpagewizard.ui   |   23 
 dbaccess/uiconfig/ui/generalpagewizard.ui |   67 +-
 dbaccess/uiconfig/ui/jdbcconnectionpage.ui|3 
 dbaccess/uiconfig/ui/ldapconnectionpage.ui|4 
 dbaccess/uiconfig/ui/specialjdbcconnectionpage.ui |5 
 include/svtools/helpids.h |3 
 include/tools/wintypes.hxx|5 
 include/vcl/roadmapwizard.hxx |  146 +
 include/vcl/vclenum.hxx   |   15 
 include/vcl/weld.hxx  |   42 +
 include/vcl/wizardmachine.hxx |  251 -
 vcl/source/app/salvtables.cxx |  293 ++
 vcl/source/control/button.cxx |3 
 vcl/source/control/roadmapwizard.cxx  |  356 -
 vcl/source/control/wizardmachine.cxx  |  598 +-
 vcl/unx/gtk3/gtk3gtkinst.cxx  |  399 --
 34 files changed, 2219 insertions(+), 280 deletions(-)

New commits:
commit 69057064d8957804c76e623d57c103c3413b7cbc
Author: Caolán McNamara 
AuthorDate: Mon Aug 5 17:26:53 2019 +0100
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 16:37:16 2019 +0200

weld ODbTypeWizDialogSetup

split up RoadmapWizard to sit its wizard logic on top of a a native 
GtkAssistant

a) awkwardly GtkAssistant is not a GtkDialog, but derives directly from
GtkWindow so some shuffling around required due to that

b) hidden/unused pages are shuffled to the end of the list of pages and
their titles turned off in order to hide them from the roadmap

c) some nonstandard hackery required to get the gtk roadmap titles to wrap

Change-Id: I0d2346c489fef744136a2785f33c846d97bd8dc6
Reviewed-on: https://gerrit.libreoffice.org/76876
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/cui/source/dialogs/about.cxx b/cui/source/dialogs/about.cxx
index a9617f7a8381..da2e4d2b4e2d 100644
--- a/cui/source/dialogs/about.cxx
+++ b/cui/source/dialogs/about.cxx
@@ -68,13 +68,13 @@ AboutDialog::AboutDialog(weld::Window* pParent)
 m_xDialog->add_button(CuiResId(RID_SVXSTR_ABOUT_WEBSITE), 102);
 m_xDialog->add_button(CuiResId(RID_SVXSTR_ABOUT_RELEASE_NOTES), 103);
 
-m_pCreditsButton = m_xDialog->get_widget_for_response(101);
-m_pCreditsButton->set_secondary(true);
-m_pWebsiteButton = m_xDialog->get_widget_for_response(102);
-m_pWebsiteButton->set_secondary(true);
-m_pReleaseNotesButton = m_xDialog->get_widget_for_response(103);
-m_pReleaseNotesButton->set_secondary(true);
-m_pCloseButton = m_xDialog->get_widget_for_response(RET_CLOSE);
+m_xCreditsButton.reset(m_xDialog->weld_widget_for_response(101));
+m_xCreditsButton->set_secondary(true);
+m_xWebsiteButton.reset(m_xDialog->weld_widget_for_response(102));
+m_xWebsiteButton->set_secondary(true);
+m_xReleaseNotesButton.reset(m_xDialog->weld_widget_for_response(103));
+m_xReleaseNotesButton->set_secondary(true);
+m_xCloseButton.reset(m_xDialog->weld_widget_for_response(RET_CLOSE));
 
 m_buildIdLinkString = m_xDialog->get_website_label();
 
@@ -88,10 +88,10 @@ AboutDialog::AboutDialog(weld::Window* pParent)
 m_xDialog->connect_size_allocate(LINK(this, AboutDialog, SizeAllocHdl));
 
 // Connect all handlers
-m_pCreditsButton->connect_clicked( LINK( this, AboutDialog, HandleClick ) 
);
-m_pWebsiteButton->connect_clicked( LINK( this, AboutDialog, HandleClick ) 
);
-m_pReleaseNotesButton->connect_clicked( LINK( this, AboutDialog, 
HandleClick ) );
-m_pCloseButton->grab_focus();
+m_xCreditsButton->connect_clicked( LINK( this, AboutDialog, HandleClick ) 
);
+m_xWebsiteButton->connect_clicked( LINK( this, AboutDialog, HandleClick ) 
);
+

[Libreoffice-qa] ESC meeting 2019-08-22 minutes

2019-08-22 Thread Miklos Vajna

* Present:
   + Michael W, Olivier, Christian, Stephan, Caolan, Sophie, Thorsten, Miklos, 
Heiko

* Completed Action Items:
   + Package Silesian (szl) - ready to be used as UI language (Cloph)
 [ done locally, but not yet pushed ]
   + Emoji toolbar control out of experimental (Heiko)
 + https://bugs.documentfoundation.org/show_bug.cgi?id=105689#c28
 [ looked, something wrong on Windows – will investigate more ]
 [ but punted the idea for later ]

* Pending Action Items:
   + Change behavior of Previous/Next Change commands:
 look at https://gerrit.libreoffice.org/#/c/76719/, decide if it’s safe
 or not (Kendy)
   + Talk to Markus about showing just dll name in crash reporting (Xisco)
 [ mail sent, but the issue not sorted out yet ]
   + Check with Italo wrt. LibOCon Hackfest wiki page (Sophie)
   + 6.4 release plan (Christian)

* Release Engineering update (Christian)
   + 6.2 – no releases end of Sep
 + would prefer a next bugfix release earlier than 60 days (Caolan)
 AI: came up with a proposal (Cloph)
   + 6.3
 + 6.3.1 RC1 is late, RC2 will be delayed a bit (Cloph)
   + Remotes
   + Android viewer
   + Online
   + daily builds – people are asking (Thorsten)
   + Xisco now has the know-how (Cloph)
   + Linux and macOS is sorted out, but Windows is a bit more complicated
   + bibisect is also an option

* Documentation (Olivier)
   + New Help
   + Investigating xapian-omega for search engine in online version
   + Helpcontents2
   + Several fixes in Help (ohallot)
   + QR Code Help page (S. Goyal)
   + Many refactoring in pages after bug triaging
   + Guides
   + More work on Calc and Draw guide
   + GSDOC
   + Tech writer bonding phase – Start work next week

* UX Update (Heiko)
   + Bugzilla (topicUI) statistics
   238(238) (topicUI) bugs open, 204(204) (needsUXEval) needs to be 
evaluated by the UXteam
   + Updates:
   BZ changes   1 week1 month   3 months   12 months
added  6(-2) 17(-1)34(0)   77(-1)
commented 64(-89)   282(-4)   552(4) 1583(30)
  removed  1(1)   2(0)  3(1)   21(1)
 resolved  7(-11)36(5) 48(4)  149(5)
   + top 10 contributors:
 Heiko Tietze made 189 changes in 1 month, and 299 changes in 1 year
 Foote, V Stuart made 81 changes in 1 month, and 260 changes in 1 year
 Kainz, Andreas made 48 changes in 1 month, and 235 changes in 1 year
 Todd made 42 changes in 1 month, and 42 changes in 1 year
 Dieter Praas made 31 changes in 1 month, and 149 changes in 1 year
 Cor Nouws made 22 changes in 1 month, and 130 changes in 1 year
 Thomas Lendo made 21 changes in 1 month, and 249 changes in 1 year
 Xisco Faulí made 21 changes in 1 month, and 277 changes in 1 year
 Kaganski, Mike made 20 changes in 1 month, and 40 changes in 1 year
 Henschel, Regina made 19 changes in 1 month, and 39 changes in 1 year
   + New needsUXEval between Aug/15-22

 * Deleting the source CSV file of an Address List should delete it
   from data sources

+ https://bugs.documentfoundation.org/show_bug.cgi?id=125331

 * Searching in comments doesn't work

+ https://bugs.documentfoundation.org/show_bug.cgi?id=125974

 => NAB/WF/WFM







 * More explanatory title for "Compare documents" file select dialog

   + https://bugs.documentfoundation.org/show_bug.cgi?id=126618

 ->  * FONT EFFECTS: strikethrough enhancements

   + https://bugs.documentfoundation.org/show_bug.cgi?id=127053

 => NEW

   + be careful, perhaps the actual rendering is not trivial (Miklos)
   + it’s in ODF after all (Heiko)







 * Sheet tab list position should be remembered across sessions

   + https://bugs.documentfoundation.org/show_bug.cgi?id=125112

 * Paragraph Style Inheritance Bug: Derived styles of Heading 1..10
   missing "Chapter Numbering" List Style used by Chapter Numbering Feature

   + https://bugs.documentfoundation.org/show_bug.cgi?id=126887

 * Add DRAW templates to LibreOffice

   + https://bugs.documentfoundation.org/show_bug.cgi?id=127028

 * Cannot find special character if does not know character name but number

   + https://bugs.documentfoundation.org/show_bug.cgi?id=111816

 * Font size in input line (formular bar) lager than in other UI elements

   + https://bugs.documentfoundation.org/show_bug.cgi?id=127066

 => unconfirmed/in discussion

* Crash Testing (Caolan)
   + 0(-1) import failure, 1(-6) export failures
   + 51 (+2) coverity issues
   + other eyes welcome now
   + Google / ossfuzz
   + 15 issues (15 Timeouts, 1 OOM)

* Crash Reporting (Xisco)
   + https://crashreport.libreoffice.org/stats/version/6.2.5.2
   + 1865 (+57) 1808
   + https://crashreport.libreoffice.org/stats/version/6.2.6.2
   + 246 (+234) 12
   + 

[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - svx/source

2019-08-22 Thread Regina Henschel (via logerrit)
 svx/source/svdraw/svdopath.cxx |   71 +
 1 file changed, 44 insertions(+), 27 deletions(-)

New commits:
commit 6f9e19c85ffc290e600084b33d9a929db28c144d
Author: Regina Henschel 
AuthorDate: Wed Aug 21 14:11:23 2019 +0200
Commit: Caolán McNamara 
CommitDate: Thu Aug 22 16:36:14 2019 +0200

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

This reverts commit 26001f4f84b0d921c8612d56613735ba5314.

Change-Id: Ib4df476b405e65be9203ab376c9c68b03198c040
Reviewed-on: https://gerrit.libreoffice.org/77882
Tested-by: Regina Henschel 
Reviewed-by: Regina Henschel 
(cherry picked from commit 59189f3e02acbdf4e5bbd35fc278252b30022f43)
Reviewed-on: https://gerrit.libreoffice.org/77939
Tested-by: Jenkins

diff --git a/svx/source/svdraw/svdopath.cxx b/svx/source/svdraw/svdopath.cxx
index 12a7b2c1af39..9b4e5f060310 100644
--- a/svx/source/svdraw/svdopath.cxx
+++ b/svx/source/svdraw/svdopath.cxx
@@ -2008,45 +2008,62 @@ void SdrPathObj::AddToHdlList(SdrHdlList& rHdlList) 
const
 
 void SdrPathObj::AddToPlusHdlList(SdrHdlList& rHdlList, SdrHdl& rHdl) const
 {
-// exclude some error situations
-const XPolyPolygon aPathPolyPolygon(GetPathPoly());
+// keep old stuff to be able to keep old SdrHdl stuff, too
+const XPolyPolygon aOldPathPolygon(GetPathPoly());
+sal_uInt16 nPnt = static_cast(rHdl.GetPointNum());
 sal_uInt16 nPolyNum = static_cast(rHdl.GetPolyNum());
-if (nPolyNum>=aPathPolyPolygon.Count())
-return;
 
-const XPolygon& rXPoly = aPathPolyPolygon[nPolyNum];
-sal_uInt16 nPntCount = rXPoly.GetPointCount();
-if (nPntCount<=0)
+if (nPolyNum>=aOldPathPolygon.Count())
 return;
 
-sal_uInt16 nPnt = static_cast(rHdl.GetPointNum());
-if (nPnt>=nPntCount)
-return;
+const XPolygon& rXPoly = aOldPathPolygon[nPolyNum];
+sal_uInt16 nPntMax = rXPoly.GetPointCount();
 
-if (rXPoly.IsControl(nPnt))
+if (nPntMax<=0)
+return;
+nPntMax--;
+if (nPnt>nPntMax)
 return;
 
-// segment before
-if (nPnt==0 && IsClosed())
-nPnt=nPntCount-1;
-if (nPnt>0 && rXPoly.IsControl(nPnt-1))
+// calculate the number of plus points
+sal_uInt16 nCnt = 0;
+if (rXPoly.GetFlags(nPnt)!=PolyFlags::Control)
 {
-std::unique_ptr pHdl(new SdrHdlBezWgt());
-pHdl->SetPos(rXPoly[nPnt-1]);
-pHdl->SetPointNum(nPnt-1);
-pHdl->SetSourceHdlNum(rHdl.GetSourceHdlNum());
-pHdl->SetPlusHdl(true);
-rHdlList.AddHdl(std::move(pHdl));
+if (nPnt==0 && IsClosed())
+nPnt=nPntMax;
+if (nPnt>0 && rXPoly.GetFlags(nPnt-1)==PolyFlags::Control)
+nCnt++;
+if (nPnt==nPntMax && IsClosed())
+nPnt=0;
+if (nPnt(rHdl.GetPointNum());
 std::unique_ptr pHdl(new SdrHdlBezWgt());
-pHdl->SetPos(rXPoly[nPnt+1]);
-pHdl->SetPointNum(nPnt+1);
+pHdl->SetPolyNum(rHdl.GetPolyNum());
+
+if (nPnt==0 && IsClosed())
+nPnt=nPntMax;
+if (nPnt>0 && rXPoly.GetFlags(nPnt-1)==PolyFlags::Control && 
nPlusNum==0)
+{
+pHdl->SetPos(rXPoly[nPnt-1]);
+pHdl->SetPointNum(nPnt-1);
+}
+else
+{
+if (nPnt==nPntMax && IsClosed())
+nPnt=0;
+if (nPntSetPos(rXPoly[nPnt+1]);
+pHdl->SetPointNum(nPnt+1);
+}
+}
+
 pHdl->SetSourceHdlNum(rHdl.GetSourceHdlNum());
 pHdl->SetPlusHdl(true);
 rHdlList.AddHdl(std::move(pHdl));
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 120965] Phantom control point line in Bézier curve

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120965

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:6.2.0 target:6.4.0   |target:6.2.0 target:6.4.0
   ||target:6.3.2

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126271] Selecting of control points of curves not working correctly for combined polygons

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126271

--- Comment #12 from Commit Notification 
 ---
Regina Henschel committed a patch related to this issue.
It has been pushed to "libreoffice-6-3":

https://git.libreoffice.org/core/+/6f9e19c85ffc290e600084b33d9a929db28c144d%5E%21

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

It will be available in 6.3.2.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 120965] Phantom control point line in Bézier curve

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120965

--- Comment #11 from Commit Notification 
 ---
Regina Henschel committed a patch related to this issue.
It has been pushed to "libreoffice-6-3":

https://git.libreoffice.org/core/+/6f9e19c85ffc290e600084b33d9a929db28c144d%5E%21

tdf#126271 Revert "tdf120965 Simplify SdrPathObj::AddToHdlList()"

It will be available in 6.3.2.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126271] Selecting of control points of curves not working correctly for combined polygons

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126271

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:6.4.0|target:6.4.0 target:6.3.2

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 122218] After Update to 6.1.4 on macOS fonts are blurred on retina display

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122218

--- Comment #32 from Christian Lohmaier  ---
To rule out (or confirm) XCode 10 being the issue or not:
could you please try with the build from
https://dev-builds.libreoffice.org/daily/libreoffice-6-3/MacOSX-x86_64@49-TDF/2019-08-21_17.28.20/
- that was done with XCode 9

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

--- Comment #1 from Thorsten Wagner  ---
Calc input line is a special widget which is used in Calc only. In relation to
other UI widgets this widget may contain a lot of text like direct input to
Calc cells.

It's only a small effort to revert font size equal to the other UI elements,
but this will restore worse readability especially with larger amounts of text.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-ux-advise] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127066

--- Comment #1 from Thorsten Wagner  ---
Calc input line is a special widget which is used in Calc only. In relation to
other UI widgets this widget may contain a lot of text like direct input to
Calc cells.

It's only a small effort to revert font size equal to the other UI elements,
but this will restore worse readability especially with larger amounts of text.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3-desktop' - sw/source

2019-08-22 Thread Andras Timar (via logerrit)
 sw/source/filter/ww8/ww8par.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit d8458569709e5bf5a05630b735f6e6d4b9b135ff
Author: Andras Timar 
AuthorDate: Thu Aug 22 16:32:18 2019 +0200
Commit: Andras Timar 
CommitDate: Thu Aug 22 16:32:18 2019 +0200

build fix

Change-Id: Iecb4be601fe1cb4d3774795be66a1a0f8c33ca3b

diff --git a/sw/source/filter/ww8/ww8par.cxx b/sw/source/filter/ww8/ww8par.cxx
index afdacbdcb732..b304a40367af 100644
--- a/sw/source/filter/ww8/ww8par.cxx
+++ b/sw/source/filter/ww8/ww8par.cxx
@@ -5490,8 +5490,8 @@ sal_uLong SwWW8ImplReader::CoreLoad(WW8Glossary *pGloss)
 
 UpdateFields();
 
-m_xWFlyPara.reset();
-m_xSFlyPara.reset();
+DELETEZ(m_pWFlyPara);
+DELETEZ(m_pSFlyPara);
 
 // delete the pam before the call for hide all redlines (Bug 73683)
 if (m_bNewDoc)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126968] Save as PDF

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126968

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:6.4.0

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126968] Save as PDF

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126968

--- Comment #2 from Commit Notification 
 ---
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/online/+/634be102c71305f0f35fe86e980ccabb381ae766%5E%21

tdf#126968: Implement "Save as PDF" in the iOS app

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] online.git: kit/ChildSession.cpp loleaflet/src

2019-08-22 Thread Tor Lillqvist (via logerrit)
 kit/ChildSession.cpp |   10 ++
 loleaflet/src/control/Control.Menubar.js |6 ++
 loleaflet/src/control/Toolbar.js |3 ++-
 3 files changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 634be102c71305f0f35fe86e980ccabb381ae766
Author: Tor Lillqvist 
AuthorDate: Thu Aug 22 15:47:12 2019 +0300
Commit: Tor Lillqvist 
CommitDate: Thu Aug 22 16:25:14 2019 +0200

tdf#126968: Implement "Save as PDF" in the iOS app

For now, add an entry "Save as PDF" to the File menu in the iOS app.
Handle in an iOS-specific way in ChildSession::downloadAs(). The PDF
is saved in the app's document directory on the device. The name of
the PDF is the basename of the document being edited with the "pdf"
extension.

Change-Id: Ib0059a86b03b978996eb4cadf230ba7f0abcba62
Reviewed-on: https://gerrit.libreoffice.org/77961
Reviewed-by: Tor Lillqvist 
Tested-by: Tor Lillqvist 

diff --git a/kit/ChildSession.cpp b/kit/ChildSession.cpp
index fadbca1ea..6d7751fdc 100644
--- a/kit/ChildSession.cpp
+++ b/kit/ChildSession.cpp
@@ -879,6 +879,15 @@ bool ChildSession::downloadAs(const char* /*buffer*/, int 
/*length*/, const std:
 filterOptions += std::string(",Watermark=") + getWatermarkText() + 
std::string("WATERMARKEND");
 }
 
+#ifdef IOS
+NSArray *pathComponents = [[NSURL URLWithString:[NSString 
stringWithUTF8String:getDocURL().c_str()]] pathComponents];
+NSString *baseName = [[pathComponents lastObject] 
stringByDeletingPathExtension];
+NSURL *documentDirectory = [NSFileManager.defaultManager 
URLsForDirectory:NSDocumentDirectory inDomains:NSUserDomainMask][0];
+NSURL *pdfURL = [documentDirectory URLByAppendingPathComponent:[baseName 
stringByAppendingString:@".pdf"]];
+getLOKitDocument()->saveAs([[pdfURL absoluteString] UTF8String],
+   format.empty() ? nullptr : format.c_str(),
+   filterOptions.empty() ? nullptr : 
filterOptions.c_str());
+#else
 // The file is removed upon downloading.
 const std::string tmpDir = FileUtil::createRandomDir(JAILED_DOCUMENT_ROOT);
 // Prevent user inputting anything funny here.
@@ -898,6 +907,7 @@ bool ChildSession::downloadAs(const char* /*buffer*/, int 
/*length*/, const std:
 
 sendTextFrame("downloadas: jail=" + _jailId + " dir=" + tmpDir + " name=" 
+ name +
   " port=" + std::to_string(ClientPortNumber) + " id=" + id);
+#endif
 return true;
 }
 
diff --git a/loleaflet/src/control/Control.Menubar.js 
b/loleaflet/src/control/Control.Menubar.js
index 41120cfc2..c71fee473 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -27,6 +27,7 @@ L.Control.Menubar = L.Control.extend({
{name: _('Word 2003 Document (.doc)'), 
id: 'downloadas-doc', type: 'action'},
{name: _('Word Document (.docx)'), id: 
'downloadas-docx', type: 'action'},
{name: _('Rich Text (.rtf)'), id: 
'downloadas-rtf', type: 'action'}]},
+   {name: _('Save as PDF'), id: 'downloadas-pdf', 
mobileapp: true, type: 'action'},
{name: _('Sign document'), id: 'signdocument', 
type: 'action'},
{type: 'separator'},
{name: _('Close document'), id: 
'closedocument', type: 'action'}
@@ -250,6 +251,7 @@ L.Control.Menubar = L.Control.extend({
{name: _('ODF presentation (.odp)'), 
id: 'downloadas-odp', type: 'action'},
{name: _('PowerPoint 2003 Presentation 
(.ppt)'), id: 'downloadas-ppt', type: 'action'},
{name: _('PowerPoint Presentation 
(.pptx)'), id: 'downloadas-pptx', type: 'action'}]},
+   {name: _('Save as PDF'), id: 'downloadas-pdf', 
mobileapp: true, type: 'action'},
{type: 'separator'},
{name: _('Close document'), id: 
'closedocument', type: 'action'}
]},
@@ -338,6 +340,7 @@ L.Control.Menubar = L.Control.extend({
{name: _('ODF spreadsheet (.ods)'), id: 
'downloadas-ods', type: 'action'},
{name: _('Excel 2003 Spreadsheet 
(.xls)'), id: 'downloadas-xls', type: 'action'},
{name: _('Excel Spreadsheet (.xlsx)'), 
id: 'downloadas-xlsx', type: 'action'}]},
+   {name: _('Save as PDF'), id: 'downloadas-pdf', 
mobileapp: true, type: 'action'},
{type: 'separator'},
{name: _('Close document'), id: 
'closedocument', type: 'action'}
]},
@@ -996,6 +999,9 @@ L.Control.Menubar = 

[Libreoffice-bugs] [Bug 125514] Bad text rendering within data source view

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125514

--- Comment #8 from Thorsten Wagner  ---
Created attachment 153577
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153577=edit
Screenshot

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 125514] Bad text rendering within data source view

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125514

Thorsten Wagner  changed:

   What|Removed |Added

 Resolution|WORKSFORME  |---
 Status|RESOLVED|UNCONFIRMED

--- Comment #7 from Thorsten Wagner  ---
Unfortunately issue still exists (see screenshot attached). Retest has been
taken place on macOS 10.14.6 with following LO release:

Version: 6.3.0.4
Build ID: 057fc023c990d676a43019934386b85b21a9ee99
CPU threads: 8; OS: Mac OS X 10.14.6; UI render: default; VCL: osx; 
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded

BTW: Close button closes whole Writer application. Probably this is not
intended. Furthermore horizontal scrolling moves to the wrong direction now.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127099] CppUnitTest_sc_dataprovider fails on ppc64le

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127099

--- Comment #1 from Jonathan Brielmaier  ---
Created attachment 153576
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153576=edit
Valgrind log for CppUnitTest_sc_dataprovider_test

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127099] New: CppUnitTest_sc_dataprovider fails on ppc64le

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127099

Bug ID: 127099
   Summary: CppUnitTest_sc_dataprovider fails on ppc64le
   Product: LibreOffice
   Version: 6.3.0.4 release
  Hardware: PowerPC
OS: Linux (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: jbrielma...@suse.de

Created attachment 153575
  --> https://bugs.documentfoundation.org/attachment.cgi?id=153575=edit
Debug log for CppUnitTest_sc_dataprovider_test

I'm running the unit tests on a Tumbleweed ppc64le machine and the 
CppUnitTest_sc_dataprovider_test test fails:

"""
ScDataProvidersTest::testCSVImport finished in: 1810ms
[New Thread 0x7fffac59f160 (LWP 22385)]
[Thread 0x7fffac59f160 (LWP 22385) exited]
ScDataProvidersTest::testDataLargerThanDB finished in: 4ms
[New Thread 0x7fffac59f160 (LWP 22386)]
[Thread 0x7fffac59f160 (LWP 22386) exited]
ScDataProvidersTest::testHTMLImport finished in: 4ms
[New Thread 0x7fffac59f160 (LWP 22387)]
[Thread 0x7fffac59f160 (LWP 22387) exited]
ScDataProvidersTest::testXMLImport finished in: 139ms
OK (4)

Thread 1 "cppunittester" received signal SIGSEGV, Segmentation fault.
0x7fffad6d42bc in ?? ()
#0  0x7fffad6d42bc in  ()
#1  0x7fffb785b114 in __run_exit_handlers () at /lib64/libc.so.6
#2  0x7fffb785b308 in exit () at /lib64/libc.so.6
#3  0x7fffb78382a0 in generic_start_main.isra () at /lib64/libc.so.6
#4  0x7fffb7838484 in __libc_start_main () at /lib64/libc.so.6
#5  0x in  ()
A debugging session is active.

Inferior 1 [process 22381] will be killed.

Quit anyway? (y or n) [answered Y; input not from terminal]
"""

For me it looks like the actual tests passes, but the unittester segfaults.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3-desktop' - writerfilter/source

2019-08-22 Thread Andras Timar (via logerrit)
 writerfilter/source/ooxml/OOXMLStreamImpl.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1d2fa036f92938dbb386f2301fd0f8eab385761f
Author: Andras Timar 
AuthorDate: Thu Aug 22 16:04:27 2019 +0200
Commit: Andras Timar 
CommitDate: Thu Aug 22 16:04:27 2019 +0200

build fix

Change-Id: I82ae070fd1597fcdeae420b28f506347eb47b005

diff --git a/writerfilter/source/ooxml/OOXMLStreamImpl.cxx 
b/writerfilter/source/ooxml/OOXMLStreamImpl.cxx
index 3398c216093f..0ee4e73dd7f2 100644
--- a/writerfilter/source/ooxml/OOXMLStreamImpl.cxx
+++ b/writerfilter/source/ooxml/OOXMLStreamImpl.cxx
@@ -319,7 +319,7 @@ bool OOXMLStreamImpl::lcl_getTarget(const 
uno::Reference xPart = 
xFac->parse(sMyTarget);
 uno::Reference xAbs = 
xFac->makeAbsolute(xBase, xPart, true, 
uri::RelativeUriExcessParentSegments_RETAIN);
-if (!xAbs)
+if (!xAbs.is())
 {
 //it was invalid gibberish
 bFound = false;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 127085] PPTX: FILESAVE: Transparency is lost after RT

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127085

Tamás Zolnai  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |zolnaitamas2...@gmail.com
   |desktop.org |

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - sc/source

2019-08-22 Thread Julien Nabet (via logerrit)
 sc/source/ui/docshell/docsh.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit bc19862ab3b13f55e9364ba5cfe8547bf30adac7
Author: Julien Nabet 
AuthorDate: Wed Aug 21 22:13:58 2019 +0200
Commit: Kohei Yoshida 
CommitDate: Thu Aug 22 15:55:17 2019 +0200

Fix full url for calc/styles by adding missing '/'

warn:sc:1272:1272:sc/source/filter/orcus/orcusfiltersimpl.cxx:149: Unable 
to load styles from xml file! boost::filesystem::file_size: Aucun fichier ou 
dossier de ce type
Thread 1 "soffice.bin" hit Breakpoint 2, (anonymous namespace)::lookup 
(file=0x556435a0, mode=(anonymous namespace)::LOOKUP_MODE_NORMAL, 
override=false, key="BRAND_BASE_DIRshare", requestStack=0x0)
at /home/julien/lo/libreoffice/sal/rtl/bootstrap.cxx:860
860OUString v;
(gdb) bt
0  0x77f2ae45 in (anonymous namespace)::lookup(Bootstrap_Impl 
const*, (anonymous namespace)::LookupMode, bool, rtl::OUString const&, 
(anonymous namespace)::ExpandRequestLink const*)
(file=0x556435a0, mode=(anonymous namespace)::LOOKUP_MODE_NORMAL, 
override=false, key="BRAND_BASE_DIRshare", requestStack=0x0) at 
/home/julien/lo/libreoffice/sal/rtl/bootstrap.cxx:860
1  0x77f2b7f6 in (anonymous namespace)::expandMacros(Bootstrap_Impl 
const*, rtl::OUString const&, (anonymous namespace)::LookupMode, (anonymous 
namespace)::ExpandRequestLink const*)
(file=0x556435a0, text="$BRAND_BASE_DIRshare/calc/styles.xml", 
mode=(anonymous namespace)::LOOKUP_MODE_NORMAL, requestStack=0x0) at 
/home/julien/lo/libreoffice/sal/rtl/bootstrap.cxx:995
2  0x77f2aa0d in 
rtl_bootstrap_expandMacros_from_handle(rtlBootstrapHandle, rtl_uString**) 
(handle=0x556435a0, macro=0x7ffefb90) at 
/home/julien/lo/libreoffice/sal/rtl/bootstrap.cxx:791
3  0x77f2aa60 in rtl_bootstrap_expandMacros(rtl_uString**) 
(macro=0x7ffefb90) at /home/julien/lo/libreoffice/sal/rtl/bootstrap.cxx:797
4  0x7fffde89e675 in rtl::Bootstrap::expandMacros(rtl::OUString&) 
(macro="$BRAND_BASE_DIRshare/calc/styles.xml") at 
/home/julien/lo/libreoffice/include/rtl/bootstrap.hxx:131
5  0x7fffde890d5f in ScDocShell::Load(SfxMedium&) (this=0x58034760, 
rMedium=...) at /home/julien/lo/libreoffice/sc/source/ui/docshell/docsh.cxx:613

Change-Id: I5e4019c730f25d32b431dbd01c5668189be6cc66
Reviewed-on: https://gerrit.libreoffice.org/77936
Tested-by: Jenkins
Reviewed-by: Kohei Yoshida 
(cherry picked from commit 86d55aaad49a3f83b61894afd81f195b37a142af)
Reviewed-on: https://gerrit.libreoffice.org/77955

diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx
index 48b0532f73a7..372686daacbc 100644
--- a/sc/source/ui/docshell/docsh.cxx
+++ b/sc/source/ui/docshell/docsh.cxx
@@ -604,7 +604,7 @@ bool ScDocShell::Load( SfxMedium& rMedium )
 {
 /* Create styles that are imported through Orcus */
 
-OUString aURL("$BRAND_BASE_DIR" LIBO_SHARE_FOLDER 
"/calc/styles.xml");
+OUString aURL("$BRAND_BASE_DIR/" LIBO_SHARE_FOLDER 
"/calc/styles.xml");
 rtl::Bootstrap::expandMacros(aURL);
 
 OUString aPath;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3-desktop' - 3 commits - filter/source sd/source

2019-08-22 Thread Caolán McNamara (via logerrit)
 filter/source/msfilter/msdffimp.cxx |   26 ++
 sd/source/filter/ppt/pptin.cxx  |2 +-
 2 files changed, 19 insertions(+), 9 deletions(-)

New commits:
commit c61ed488f9ed8375e02d8fd41fbe3a0901cd311a
Author: Caolán McNamara 
AuthorDate: Sun Nov 26 20:45:10 2017 +
Commit: Andras Timar 
CommitDate: Thu Aug 22 15:54:41 2019 +0200

ofz#4436 check if seek succeeded

Change-Id: I56d9692647b28c706b56ccacf08d494b3d830d94
Reviewed-on: https://gerrit.libreoffice.org/45296
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 
(cherry picked from commit 765ff606426251f90aff5d1fc89f01ed7594ed59)

diff --git a/filter/source/msfilter/msdffimp.cxx 
b/filter/source/msfilter/msdffimp.cxx
index 1e4023c4d021..a3d1b2909fb6 100644
--- a/filter/source/msfilter/msdffimp.cxx
+++ b/filter/source/msfilter/msdffimp.cxx
@@ -6046,7 +6046,10 @@ bool SvxMSDffManager::GetShapeGroupContainerData( 
SvStream& rSt,
 return false;
 }
 else
-rSt.SeekRel( nLength );
+{
+if (!checkSeek(rSt, rSt.Tell() + nLength))
+return false;
+}
 nReadSpGrCont += nLength;
 }
 while( nReadSpGrCont < nLenShapeGroupCont );
diff --git a/sd/source/filter/ppt/pptin.cxx b/sd/source/filter/ppt/pptin.cxx
index f54b6afe0d2e..b29156594855 100644
--- a/sd/source/filter/ppt/pptin.cxx
+++ b/sd/source/filter/ppt/pptin.cxx
@@ -854,7 +854,7 @@ bool ImplSdPPTImport::Import()
 bool bSuccess = aHd.SeekToEndOfRecord(rStCtrl);
 if (!bSuccess)
 {
-SAL_WARN("filter.ms", "Count not seek to end of 
record");
+SAL_WARN("filter.ms", "Could not seek to end of 
record");
 break;
 }
 }
commit 74d737aecd0821a255f5ba98be3239a6ff1df984
Author: Caolán McNamara 
AuthorDate: Sat Apr 8 20:31:35 2017 +0100
Commit: Andras Timar 
CommitDate: Thu Aug 22 15:54:21 2019 +0200

limit symbol visibility

Change-Id: I54d5fbe1cd9d5ae94cb134f08fc00593c802ebee
(cherry picked from commit 79890a6d1bccbba8c40e42566c3d437d7a1aaace)

diff --git a/filter/source/msfilter/msdffimp.cxx 
b/filter/source/msfilter/msdffimp.cxx
index 720fac503635..1e4023c4d021 100644
--- a/filter/source/msfilter/msdffimp.cxx
+++ b/filter/source/msfilter/msdffimp.cxx
@@ -222,10 +222,13 @@ DffPropertyReader::DffPropertyReader( const 
SvxMSDffManager& rMan )
 InitializePropSet( DFF_msofbtOPT );
 }
 
-bool checkSeek(SvStream , sal_uInt32 nOffset)
+namespace
 {
-const sal_uInt64 nMaxSeek(rSt.Tell() + rSt.remainingSize());
-return (nOffset <= nMaxSeek && rSt.Seek(nOffset) == nOffset);
+bool checkSeek(SvStream , sal_uInt32 nOffset)
+{
+const sal_uInt64 nMaxSeek(rSt.Tell() + rSt.remainingSize());
+return (nOffset <= nMaxSeek && rSt.Seek(nOffset) == nOffset);
+}
 }
 
 void DffPropertyReader::SetDefaultPropSet( SvStream& rStCtrl, sal_uInt32 
nOffsDgg ) const
commit 17afdae82f7c9968a372cb70e5792547b50c10ac
Author: Caolán McNamara 
AuthorDate: Fri Apr 7 20:49:52 2017 +0100
Commit: Andras Timar 
CommitDate: Thu Aug 22 15:54:06 2019 +0200

check escher seeks

Change-Id: I4cb523e8654defa25e75ebd54f1e125ea02e1cf9
Reviewed-on: https://gerrit.libreoffice.org/36281
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 
(cherry picked from commit a1147a34d85742fd5d9410efac6fc59ac2fd1574)

diff --git a/filter/source/msfilter/msdffimp.cxx 
b/filter/source/msfilter/msdffimp.cxx
index 508bee068d02..720fac503635 100644
--- a/filter/source/msfilter/msdffimp.cxx
+++ b/filter/source/msfilter/msdffimp.cxx
@@ -222,13 +222,20 @@ DffPropertyReader::DffPropertyReader( const 
SvxMSDffManager& rMan )
 InitializePropSet( DFF_msofbtOPT );
 }
 
+bool checkSeek(SvStream , sal_uInt32 nOffset)
+{
+const sal_uInt64 nMaxSeek(rSt.Tell() + rSt.remainingSize());
+return (nOffset <= nMaxSeek && rSt.Seek(nOffset) == nOffset);
+}
+
 void DffPropertyReader::SetDefaultPropSet( SvStream& rStCtrl, sal_uInt32 
nOffsDgg ) const
 {
 delete pDefaultPropSet;
 sal_uInt32 nMerk = rStCtrl.Tell();
-rStCtrl.Seek( nOffsDgg );
+bool bOk = checkSeek(rStCtrl, nOffsDgg);
 DffRecordHeader aRecHd;
-bool bOk = ReadDffRecordHeader( rStCtrl, aRecHd );
+if (bOk)
+bOk = ReadDffRecordHeader( rStCtrl, aRecHd );
 if (bOk && aRecHd.nRecType == DFF_msofbtDggContainer)
 {
 if ( SvxMSDffManager::SeekToRec( rStCtrl, DFF_msofbtOPT, 
aRecHd.GetRecEndFilePos() ) )
@@ -5839,13 +5846,10 @@ void SvxMSDffManager::CheckTxBxStoryChain()
 and remembering the File-Offsets for each Blip

 **/
-void SvxMSDffManager::GetCtrlData( sal_uInt32 nOffsDgg_ )
+void 

[Libreoffice-commits] core.git: 2 commits - include/test include/toolkit include/unotest toolkit/source

2019-08-22 Thread Noel Grandin (via logerrit)
 include/test/sheet/xcellrangereferrer.hxx |2 +-
 include/test/sheet/xformulaquery.hxx  |2 +-
 include/test/table/xtablecolumns.hxx  |2 +-
 include/toolkit/awt/vclxwindows.hxx   |4 ++--
 include/unotest/bootstrapfixturebase.hxx  |4 ++--
 toolkit/source/awt/vclxwindow.cxx |4 ++--
 toolkit/source/awt/vclxwindows.cxx|4 ++--
 7 files changed, 11 insertions(+), 11 deletions(-)

New commits:
commit e724f245e9652230d4c1f58c353be150006affcd
Author: Noel Grandin 
AuthorDate: Thu Aug 22 13:41:58 2019 +0200
Commit: Noel Grandin 
CommitDate: Thu Aug 22 15:46:24 2019 +0200

loplugin:constmethod in test

Change-Id: I15aa90d962b9d428a911f4b47a06d55318b42ba4
Reviewed-on: https://gerrit.libreoffice.org/77953
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/test/sheet/xcellrangereferrer.hxx 
b/include/test/sheet/xcellrangereferrer.hxx
index f840a6bd1355..949f6b12eb7e 100644
--- a/include/test/sheet/xcellrangereferrer.hxx
+++ b/include/test/sheet/xcellrangereferrer.hxx
@@ -24,7 +24,7 @@ public:
 virtual css::uno::Reference< css::uno::XInterface > init() = 0;
 
 void setCellRange(css::table::CellRangeAddress aCellRange) { m_aCellRange 
= aCellRange; }
-const css::table::CellRangeAddress & getCellRange() { return m_aCellRange; 
}
+const css::table::CellRangeAddress & getCellRange() const { return 
m_aCellRange; }
 void testGetReferredCells();
 
 protected:
diff --git a/include/test/sheet/xformulaquery.hxx 
b/include/test/sheet/xformulaquery.hxx
index 7beb944e192c..662bc6c90fc6 100644
--- a/include/test/sheet/xformulaquery.hxx
+++ b/include/test/sheet/xformulaquery.hxx
@@ -35,7 +35,7 @@ public:
 virtual css::uno::Reference init() = 0;
 virtual css::uno::Reference getXSpreadsheet() = 0;
 void setXCell(css::uno::Reference xCell) { m_xCell = 
xCell; }
-css::uno::Reference const& getXCell() { return m_xCell; 
}
+css::uno::Reference const& getXCell() const { return 
m_xCell; }
 
 void testQueryDependents();
 void testQueryPrecedents();
diff --git a/include/test/table/xtablecolumns.hxx 
b/include/test/table/xtablecolumns.hxx
index 204fb2860aa2..169eac9dbc77 100644
--- a/include/test/table/xtablecolumns.hxx
+++ b/include/test/table/xtablecolumns.hxx
@@ -25,7 +25,7 @@ class OOO_DLLPUBLIC_TEST XTableColumns
 public:
 virtual css::uno::Reference init() = 0;
 
-void setXSpreadsheet(css::uno::Reference& 
r_xSheet)
+void setXSpreadsheet(const css::uno::Reference& 
r_xSheet)
 {
 m_xSheet = r_xSheet;
 }
diff --git a/include/unotest/bootstrapfixturebase.hxx 
b/include/unotest/bootstrapfixturebase.hxx
index 0a967d34d736..85f270b02c2c 100644
--- a/include/unotest/bootstrapfixturebase.hxx
+++ b/include/unotest/bootstrapfixturebase.hxx
@@ -59,9 +59,9 @@ public:
   virtual ~BootstrapFixtureBase() override;
 
   const css::uno::Reference&
-  getComponentContext() { return m_xContext; }
+  getComponentContext() const { return m_xContext; }
   const css::uno::Reference&
-  getMultiServiceFactory() { return m_xSFactory; }
+  getMultiServiceFactory() const { return m_xSFactory; }
 
   virtual void setUp() override;
   virtual void tearDown() override;
commit 8a82dca5bd43080f0f5597b915d1433b1c0f141a
Author: Noel Grandin 
AuthorDate: Thu Aug 22 13:41:08 2019 +0200
Commit: Noel Grandin 
CommitDate: Thu Aug 22 15:46:16 2019 +0200

loplugin:constmethod in toolkit

Change-Id: Id80ee487d7083da5c1f1c4d012d6339700471204
Reviewed-on: https://gerrit.libreoffice.org/77952
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/toolkit/awt/vclxwindows.hxx 
b/include/toolkit/awt/vclxwindows.hxx
index 202d5974c8f9..eba13e505a41 100644
--- a/include/toolkit/awt/vclxwindows.hxx
+++ b/include/toolkit/awt/vclxwindows.hxx
@@ -608,7 +608,7 @@ public:
 /// @throws css::uno::RuntimeException
 void setMinimum( sal_Int32 n );
 /// @throws css::uno::RuntimeException
-sal_Int32 getMinimum(  );
+sal_Int32 getMinimum(  ) const;
 
 // css::awt::VclWindowPeer
 void SAL_CALL setProperty( const OUString& PropertyName, const 
css::uno::Any& Value ) override;
@@ -889,7 +889,7 @@ public:
 voidSetFormatter( FormatterBase* pFormatter ) { mpFormatter = 
pFormatter; }
 
 voidsetStrictFormat( bool bStrict );
-boolisStrictFormat();
+boolisStrictFormat() const;
 
 // css::awt::VclWindowPeer
 void SAL_CALL setProperty( const OUString& PropertyName, const 
css::uno::Any& Value ) override;
diff --git a/toolkit/source/awt/vclxwindow.cxx 
b/toolkit/source/awt/vclxwindow.cxx
index d632fc13036b..58afca500b80 100644
--- a/toolkit/source/awt/vclxwindow.cxx
+++ b/toolkit/source/awt/vclxwindow.cxx
@@ -156,11 +156,11 @@ public:
 /** synchronously mbEnableVisible
 */
 voidsetEnableVisible( bool bEnableVisible ) { 

[Libreoffice-ux-advise] [Bug 122007] Support for multiple selection of list entries in data series selection

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122007

Buovjaga  changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|REOPENED|RESOLVED
 CC||todven...@suomi24.fi

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-bugs] [Bug 122007] Support for multiple selection of list entries in data series selection

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122007

Buovjaga  changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|REOPENED|RESOLVED
 CC||todven...@suomi24.fi

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 108538] [META] Chart feature enhancements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108538
Bug 108538 depends on bug 122007, which changed state.

Bug 122007 Summary: Support for multiple selection of list entries in data 
series selection
https://bugs.documentfoundation.org/show_bug.cgi?id=122007

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 125834] Extend context menu in a pivot table with mark actions

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125834

Buovjaga  changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|REOPENED|RESOLVED
 CC||todven...@suomi24.fi

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 103381] [META] Pivot table (aka Data Pilot) bugs and enhancements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103381
Bug 103381 depends on bug 125834, which changed state.

Bug 125834 Summary: Extend context menu in a pivot table with mark actions
https://bugs.documentfoundation.org/show_bug.cgi?id=125834

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 86349] [META] Context menu bugs and enhancements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=86349
Bug 86349 depends on bug 125834, which changed state.

Bug 125834 Summary: Extend context menu in a pivot table with mark actions
https://bugs.documentfoundation.org/show_bug.cgi?id=125834

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 103381] [META] Pivot table (aka Data Pilot) bugs and enhancements

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103381
Bug 103381 depends on bug 125864, which changed state.

Bug 125864 Summary: Safer application of options for each filter criterium in a 
pivot table
https://bugs.documentfoundation.org/show_bug.cgi?id=125864

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 125864] Safer application of options for each filter criterium in a pivot table

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125864

Buovjaga  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
   Keywords|needsUXEval |
 CC||todven...@suomi24.fi
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 127098] New: Functions to Track and View Changes should really become Useful

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127098

Bug ID: 127098
   Summary: Functions to Track and View Changes should really
become Useful
   Product: LibreOffice
   Version: 6.4.0.0.alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: enhancement
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: adalbert.hans...@gmx.de

This proposal refers to experiences with LibreOfficeWriter Development up to 

Version: 6.4.0.0.alpha0+
Build ID: 8387a6db641b29e6ff3c2f4cdc4688f538cbe35f
CPU threads: 4; OS: Linux 4.4; UI render: default; VCL: gtk3; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time:
2019-08-09_06:28:42
Locale: de-DE (de_DE.UTF-8); UI-Language: en-US
Calc: threaded

and earlier versions. The functions to track and view changes in a document are
not really practical. Therefore they should be improved.

Steps to reproduce
1. provide a document of a few pages length.
2. Switch on Edit>Track Changes>Record but do not switch on View>Track Changes
at this point.
3. Apply a quite few changes in the document.
4. Apply some more changes to the document, but do this out of order from top
to bottom of the document, just one change here, the other one there and so on.
5. Correct some typos in the document.
6. Add some format changes to it, e.g. adding tabs, deleting multiple white
spaces.
7. Make some format changes. e.g. setting some words to italics, underlines or
bold.
8. Mark some lines as headlines of different order. If you have the impression
that you have touched in total 20-30% of the text somehow, this is a good
example.
9. Now switch on View>Track Changes.

If you have applied enough changes, the whole document looks to be completely
changed and you have difficulties to keep an overview of what has happened to
it.

Therefore I propose to enhance the already existing functions of the track
changes, namely View>Track Changes and Edit>Track Changes>Manage Changes, which
already has a tab Filter. It gives access to select changes in its List tab.
Only changes meeting certain criteria dealing with Author, Action or Comment
defined in the Filter tab are shown in the List tab. 

Using the category Comment (which has nothing to do to add Comments to the text
itself during the revision process) might be useful if there is a strict
agreement among the revisors to categorize changes by type and assign e.g.
letters to the different changes (e.g. O=optional, B=might be left out for
brevity, S plus a number=superfluous, the number characterizing the duplicate
where the topic is also dealt with). Using Comment requires very much
discipline among the involved people however.

The Action tab lets one choose between Insertion/Deletion/Format/Table Changes.
Unfortunately, selection only has an influence on the list shown in the List
tab of Edit>Track Changes>Manage Changes, not on the document window. That
would really provide a great benefit! The Filter criteria should also influence
the display of changes in the document window, this would be the right place to
deselect all format changes in View>Track Changes. 

It does not make much sense to select either Insertions or Deletions. At least
one should have a chance to select Insertions or Deletions. Often one wants to
view the document with the text changes emphasized (by color). Selecting
Insertions or Deletions would be the right thing then. However, corrections of
typos (typically done by using the spell check functions or by typing over a
marked part in a word and only changing a few characters, namely transposing a
few of them or adding missing ones or deleting duplicatd ones - but not
exchanging digits in a number! - would also be something which should be
possible to select or deselect for View>Track Changes. I propose to categorize
all this as “typos”. Of course, proceed to the previous or next correction or
accept all corrections should only operate on those which are selected
(presently it is that way, but they are only highlighted in Edit>Track
Changes>Manage Changes List tab and one can click on them to get to the proper
place in the document window). Of course currently you can proceed to the next
or previous meeting the filter criterion and accept the current one or reject
it or accept all or reject all selected changes.

Expected result: 
9. View>Track Changes should have two basic choices:
simple
Text changes (insertions or deletions)
typos
white spaces (insertions or deletions)
format changes

complex (see Edit>Track Changes>Manage Changes>Filter)
If one checks simple, one should be able to select among the four subcategories
(0 to 4 check marks).

If one selects complex, one should be able to select like in the current
Edit>Track Changes>Manage Changes>Filter tab, however with the addition of
being able 

[Libreoffice-ux-advise] [Bug 125864] Safer application of options for each filter criterium in a pivot table

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125864

Buovjaga  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
   Keywords|needsUXEval |
 CC||todven...@suomi24.fi
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-commits] core.git: package/source

2019-08-22 Thread Ashod Nakashian (via logerrit)
 package/source/zippackage/ZipPackage.cxx |9 -
 1 file changed, 4 insertions(+), 5 deletions(-)

New commits:
commit 0ec0ba24f0f28deef7993e430682bf4a5d9f3271
Author: Ashod Nakashian 
AuthorDate: Mon Mar 25 10:29:38 2019 -0400
Commit: Ashod Nakashian 
CommitDate: Thu Aug 22 15:33:15 2019 +0200

oox: typos, comments, formatting

Change-Id: I6bb138be2a8da91a85248e85d408f54a9b885f9b
Reviewed-on: https://gerrit.libreoffice.org/77686
Tested-by: Jenkins
Reviewed-by: Ashod Nakashian 

diff --git a/package/source/zippackage/ZipPackage.cxx 
b/package/source/zippackage/ZipPackage.cxx
index f5d8ca351ba9..574b7a434f5c 100644
--- a/package/source/zippackage/ZipPackage.cxx
+++ b/package/source/zippackage/ZipPackage.cxx
@@ -713,9 +713,9 @@ void SAL_CALL ZipPackage::initialize( const uno::Sequence< 
Any >& aArguments )
 }
 else if ( aNamedValue.Value >>= nFormatID )
 {
-if ( nFormatID != embed::StorageFormats::PACKAGE
-  && nFormatID != embed::StorageFormats::ZIP
-  && nFormatID != embed::StorageFormats::OFOPXML )
+if (nFormatID != embed::StorageFormats::PACKAGE
+&& nFormatID != embed::StorageFormats::ZIP
+&& nFormatID != embed::StorageFormats::OFOPXML)
 throw lang::IllegalArgumentException(THROW_WHERE, 
uno::Reference< uno::XInterface >(), 1 );
 
 m_nFormat = nFormatID;
@@ -1088,8 +1088,7 @@ void ZipPackage::WriteContentTypes( ZipOutputStream& 
aZipOut, const vector< uno:
 pEntry->nSize = pEntry->nCompressedSize = -1;
 pEntry->nTime = ZipOutputStream::getCurrentDosTime();
 
-// Convert vector into a uno::Sequence
-// TODO/LATER: use Default entries in future
+// Add default entries, the count must be updated manually when appending.
 uno::Sequence< beans::StringPair > aDefaultsSequence(4);
 // Add at least the standard default entries.
 aDefaultsSequence[0].First = "xml";
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 124365] FORMATTING Opening 2 separate "Cell Style Modify" windows disables input on all windows

2019-08-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=124365

Buovjaga  changed:

   What|Removed |Added

 Resolution|FIXED   |WORKSFORME

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3-desktop' - filter/source

2019-08-22 Thread Andras Timar (via logerrit)
 filter/source/xsltfilter/OleHandler.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3bf20599bb3afe03086e602ccebabd77b8896f71
Author: Andras Timar 
AuthorDate: Thu Aug 22 15:31:45 2019 +0200
Commit: Andras Timar 
CommitDate: Thu Aug 22 15:31:45 2019 +0200

build fix

Change-Id: Ic0be0c6fd0d498b3f0512461480f166ad3f7b5f8

diff --git a/filter/source/xsltfilter/OleHandler.cxx 
b/filter/source/xsltfilter/OleHandler.cxx
index 057b69ada620..807a8a04fed1 100644
--- a/filter/source/xsltfilter/OleHandler.cxx
+++ b/filter/source/xsltfilter/OleHandler.cxx
@@ -97,7 +97,7 @@ namespace XSLT
 OString SAL_CALL
 OleHandler::encodeSubStorage(const OUString& streamName)
 {
-if (!m_storage || !m_storage->hasByName(streamName))
+if (!m_storage.is() || !m_storage->hasByName(streamName))
 {
 return "Not Found:";// + streamName;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

  1   2   3   >