[PUSHED libreoffice-4-1-0] fdo#67093 Change traduction of Title for some more languages

2013-07-23 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/5044

Approvals:
  Petr Mladek: Verified; Looks good to me, but someone else must approve
  Thorsten Behrens: Looks good to me, but someone else must approve
  Bosdonnat Cedric: Verified; Looks good to me, approved
  Fridrich Strba: Looks good to me, but someone else must approve


-- 
To view, visit https://gerrit.libreoffice.org/5044
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibcd7d03c9ea7608c8963d1cb9b7c2d7cbe2415d5
Gerrit-PatchSet: 2
Gerrit-Project: translations
Gerrit-Branch: libreoffice-4-1-0
Gerrit-Owner: Petr Mladek pmla...@suse.cz
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Christian Lohmaier lohmaier+libreoff...@googlemail.com
Gerrit-Reviewer: Fridrich Strba fridr...@documentfoundation.org
Gerrit-Reviewer: Petr Mladek pmla...@suse.cz
Gerrit-Reviewer: Thorsten Behrens tbehr...@suse.com

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


[PATCH libreoffice-4-1] n#820504: default color hidden by Default style in writerfil...

2013-06-18 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/4329

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/29/4329/1

n#820504: default color hidden by Default style in writerfilter

Setting the font color of the default paragraph style to Auto makes the
color set in the Writer defaults (from rPrDefault) ignored.

Change-Id: I1dcd92bcd0774f5229785373242899a492ab7b7c
(cherry picked from commit d7acb5ba0924f848f1ae96eceab50a0169d18445)
(cherry picked from commit 122db845e057cfd83bb2b5fde442200bc742eb8d)
---
M writerfilter/source/dmapper/StyleSheetTable.cxx
1 file changed, 1 insertion(+), 2 deletions(-)



diff --git a/writerfilter/source/dmapper/StyleSheetTable.cxx 
b/writerfilter/source/dmapper/StyleSheetTable.cxx
index 74adb0b..8352b1b 100644
--- a/writerfilter/source/dmapper/StyleSheetTable.cxx
+++ b/writerfilter/source/dmapper/StyleSheetTable.cxx
@@ -786,8 +786,7 @@
 pEntry-pProperties-Insert(PROP_PARA_ORPHANS, true, 
aTwo, false);
 //  Left-to-right direction if not already set
 pEntry-pProperties-Insert(PROP_WRITING_MODE, true, 
uno::makeAny( sal_Int16(text::WritingMode_LR_TB) ), false);
-//  font color COL_AUTO if not already set
-pEntry-pProperties-Insert(PROP_CHAR_COLOR, true, 
uno::makeAny( sal_Int32(0x) ), false);
+// Don't set font color to Auto if not already set: 
this could hide the default font color setting
 }
 
 uno::Sequence beans::PropertyValue  aPropValues = 
pEntry-pProperties-GetPropertyValues();

-- 
To view, visit https://gerrit.libreoffice.org/4329
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1dcd92bcd0774f5229785373242899a492ab7b7c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-1
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-1] added unit test for n#820504

2013-06-18 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/4330

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/30/4330/1

added unit test for n#820504

Change-Id: I804e081a564d24d3896160f817051b19da884c34
(cherry picked from commit f9610ed634318c27382d79089d022fb8175eb90a)
---
A sw/qa/extras/ooxmlimport/data/n820504.docx
M sw/qa/extras/ooxmlimport/ooxmlimport.cxx
2 files changed, 12 insertions(+), 0 deletions(-)



diff --git a/sw/qa/extras/ooxmlimport/data/n820504.docx 
b/sw/qa/extras/ooxmlimport/data/n820504.docx
new file mode 100644
index 000..36279a9
--- /dev/null
+++ b/sw/qa/extras/ooxmlimport/data/n820504.docx
Binary files differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index ce66980..c895db2 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -119,6 +119,7 @@
 void testPageBorderShadow();
 void testN820509();
 void testN820788();
+void testN820504();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX)  !defined(WNT)
@@ -205,6 +206,7 @@
 {page-border-shadow.docx, Test::testPageBorderShadow},
 {n820509.docx, Test::testN820509},
 {n820788.docx, Test::testN820788},
+{n820504.docx, Test::testN820504},
 };
 header();
 for (unsigned int i = 0; i  SAL_N_ELEMENTS(aMethods); ++i)
@@ -1457,6 +1459,16 @@
 CPPUNIT_ASSERT_EQUAL(text::SizeType::MIN, getPropertysal_Int16(xFrame, 
SizeType));
 }
 
+void Test::testN820504()
+{
+uno::Referencestyle::XStyleFamiliesSupplier 
xFamiliesSupplier(mxComponent, uno::UNO_QUERY);
+uno::Referencecontainer::XNameAccess 
xFamiliesAccess(xFamiliesSupplier-getStyleFamilies(), uno::UNO_QUERY);
+uno::Referencecontainer::XNameAccess 
xStylesAccess(xFamiliesAccess-getByName(ParagraphStyles), uno::UNO_QUERY);
+uno::Referencebeans::XPropertySet 
xStyle(xStylesAccess-getByName(Default Style), uno::UNO_QUERY);
+// The problem was that the CharColor was set to AUTO (-1) even if we have 
some default char color set
+CPPUNIT_ASSERT_EQUAL(sal_Int32(4040635), getPropertysal_Int32(xStyle, 
CharColor));
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();

-- 
To view, visit https://gerrit.libreoffice.org/4330
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I804e081a564d24d3896160f817051b19da884c34
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-1
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Template Manager: Use actual template icons if no thumbnail ...

2013-06-17 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3278

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3278
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1ee86a180701355886a340ea97940303f527422
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Stefan Knorr heinzless...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Fixed libcmis handling of related multipart content types

2013-06-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/4241

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/41/4241/1

Fixed libcmis handling of related multipart content types

Change-Id: I8468286be1affc635838b5329d2df72b53f5c9de
(cherry picked from commit 6b10219eefc0d6bd0e871976d6fe781a26020c8b)
(cherry picked from commit 6ed3ef87d2472bad71b719e9ec927e72acfd2850)
---
M libcmis/libcmis-0.3.0.patch
1 file changed, 14 insertions(+), 0 deletions(-)



diff --git a/libcmis/libcmis-0.3.0.patch b/libcmis/libcmis-0.3.0.patch
index 37644c8..772c828 100644
--- a/libcmis/libcmis-0.3.0.patch
+++ b/libcmis/libcmis-0.3.0.patch
@@ -33,3 +33,17 @@
  contentLength = it-second-getLongs( ).front( );
  return contentLength;
  }
+diff --git src/libcmis/ws-relatedmultipart.cxx 
src/libcmis/ws-relatedmultipart.cxx
+index ef91b4d..3a990e7 100644
+--- src/libcmis/ws-relatedmultipart.cxx
 src/libcmis/ws-relatedmultipart.cxx
+@@ -93,6 +93,8 @@ RelatedMultipart::RelatedMultipart( const string body, 
const string contentTyp
+ if ( value[0] == ''  value[value.length() - 1] == '' )
+ value = value.substr( 1, value.length( ) - 2 );
+ 
++name = libcmis::trim( name );
++
+ if ( name == start )
+ {
+ m_startId = value;
+-- 

-- 
To view, visit https://gerrit.libreoffice.org/4241
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8468286be1affc635838b5329d2df72b53f5c9de
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] fdo#64577, CMIS: fixed crasher in libcmis

2013-05-16 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3921

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/3921/1

fdo#64577, CMIS: fixed crasher in libcmis

The crash is handled, but the huge files' size won't be properly handled
on 32bits builds: would require a libcmis API change.

Change-Id: I8ef1190a4d1de7d91a67ec20330db9e1747dfdc2
(cherry picked from commit 7a6f5186a8e4089181f57af44fa4654fe692856b)
---
M libcmis/libcmis-0.3.0.patch
1 file changed, 13 insertions(+), 0 deletions(-)



diff --git a/libcmis/libcmis-0.3.0.patch b/libcmis/libcmis-0.3.0.patch
index d9dc70f..37644c8 100644
--- a/libcmis/libcmis-0.3.0.patch
+++ b/libcmis/libcmis-0.3.0.patch
@@ -20,3 +20,16 @@
  xmlTextWriterEndElement( writer ); // End of Expires
  xmlTextWriterEndElement( writer ); // End of Timestamp
  
+diff --git src/libcmis/document.cxx src/libcmis/document.cxx
+index 7c2a8e6..e7014cb 100644
+--- src/libcmis/document.cxx
 src/libcmis/document.cxx
+@@ -78,7 +78,7 @@ namespace libcmis
+ {
+ long contentLength = 0;
+ map string, libcmis::PropertyPtr ::const_iterator it = 
getProperties( ).find( string( cmis:contentStreamLength ) );
+-if ( it != getProperties( ).end( )  !it-second-getStrings( 
).empty( ) )
++if ( it != getProperties( ).end( )  !it-second-getLongs( ).empty( 
) )
+ contentLength = it-second-getLongs( ).front( );
+ return contentLength;
+ }

-- 
To view, visit https://gerrit.libreoffice.org/3921
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8ef1190a4d1de7d91a67ec20330db9e1747dfdc2
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fix for fdo#53031

2013-05-16 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3914

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3914
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I137f2446d08b7b536ded26676132e63489e26e48
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Manal Alhassoun malhass...@kacst.edu.sa
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] fdo#64249: make sure we have matching start/end level for ta...

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3907

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/07/3907/1

fdo#64249: make sure we have matching start/end level for table in shapes

Change-Id: I4928f6a54e6cca9b26c0d2391fc1545688be2375
(cherry picked from commit 9ceda3013a6b7e60e0e56ba540e217fd36a34a0f)
---
M writerfilter/source/dmapper/DomainMapper_Impl.cxx
1 file changed, 6 insertions(+), 7 deletions(-)



diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx 
b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index deda4f1..48e40dea 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -1627,6 +1627,10 @@
 if (m_aTextAppendStack.empty())
 return;
 uno::Referencetext::XTextAppend xTextAppend = 
m_aTextAppendStack.top().xTextAppend;
+
+appendTableManager( );
+appendTableHandler( );
+getTableManager().startLevel();
 try
 {
 uno::Reference lang::XServiceInfo  xSInfo( xShape, 
uno::UNO_QUERY_THROW );
@@ -1685,10 +1689,6 @@
 xProps-setPropertyValue( rPropNameSupplier.GetName( 
PROP_ANCHOR_TYPE ), bIsGraphic  ?  uno::makeAny( 
text::TextContentAnchorType_AS_CHARACTER ) : uno::makeAny( 
text::TextContentAnchorType_AT_PARAGRAPH ) );
 }
 }
-
-appendTableManager( );
-appendTableHandler( );
-getTableManager().startLevel();
 }
 catch ( const uno::Exception e )
 {
@@ -1700,11 +1700,10 @@
 
 void DomainMapper_Impl::PopShapeContext()
 {
+getTableManager().endLevel();
+popTableManager();
 if ( m_aAnchoredStack.size()  0 )
 {
-getTableManager().endLevel();
-popTableManager();
-
 // For OLE object replacement shape, the text append context was 
already removed
 // or the OLE object couldn't be inserted.
 if ( !m_aAnchoredStack.top().bToRemove )

-- 
To view, visit https://gerrit.libreoffice.org/3907
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4928f6a54e6cca9b26c0d2391fc1545688be2375
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] coverity#982969 Resource leak

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3902

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3902
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iabf637be4b704de4165c5cefb6a19687a1b5637a
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Require at least gio 2.26 for the dbus support.

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3904

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3904
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6276d55081098f567104f61fde896d1b66591522
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Mark Wielaard m...@klomp.org
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] coverity#705731 Resource leak

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3901

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3901
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idbe476534025017d21e8af353e92a15cee902cca
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] coverity#1000721 Resource Leak

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3897

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3897
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I59161a08eee0406ac71b1212a0a8697edf27911f
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] coverity#1019413: Unitialized scalar variable

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3895

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3895
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0a2a1a137176f2f2176b2c16cd7a747d9bc9b52
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/ribbar...

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3893

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3893
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I048ff445fa04b12cb86ae580370bfe444041d41f
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matthias Freund matti...@yahoo.de
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/utlui/

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3891

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3891
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie150e8812b674a9fdd0cb904489dfb561bdcf7a2
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matthias Freund matti...@yahoo.de
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#39468 Comment translation GER-ENG in sw/source/ui/shells...

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3890

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3890
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I94cc2d50c85acd7434f12766094d3edd0cf9d71d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matthias Freund matti...@yahoo.de
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3771

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3771
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I80aca85562e4a43fded5c37f9895e51f79e42c14
Gerrit-PatchSet: 8
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ioan Radu ioan.rad...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Chris Sherlock chris.sherloc...@gmail.com
Gerrit-Reviewer: David Tardon dtar...@redhat.com
Gerrit-Reviewer: Norbert Thiebaud nthieb...@gmail.com

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


[PUSHED] fdo#55582 calc part

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3872

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3872
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I01ce4a7a7738304f47efc718c382a5b1dee68ea2
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Lennard Wasserthal wassert...@nefkom.net
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] cppcheck: fix arrayIndexThenCheck/optimize loops

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3869

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3869
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If399abf5fb3eaea91f4f5d5e86b86b37790d3c9a
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet serval2...@yahoo.fr
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#32368 - Link both Ctrl+Y as Ctrl+Shift+Z as shortcut key...

2013-05-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3853

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3853
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d766b300d79e9ebd85dc5ae7bb327cf9c1a7c41
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Joren De Cuyper joren.libreoff...@telenet.be
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Related fdo#35546: Simplyfy Code in Photo Album Dialog

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3855

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3855
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie40a4f3bb4402b988071b315617404c802b7e92a
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Samuel Mehrbrodt s.mehrbr...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Related fdo#35546: Further improve Layout of Photo Album Dia...

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3852

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3852
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icfb24a8678d096bb596bcdeabbf539e9fe1835db
Gerrit-PatchSet: 4
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Samuel Mehrbrodt s.mehrbr...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Related fdo#35546: Remove 1 Image with Title Layout for Ph...

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3858

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3858
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I891511d6cf8183e45b41a9358f4821f7de551836
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Samuel Mehrbrodt s.mehrbr...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Get rid of GlobalEditData::GetStdRefDevice()

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3886

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3886
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf3ae3c28da0583a44472f36312b07766734bf7b
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Tor Lillqvist t...@iki.fi
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] pastespecial.ui widget

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3790

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3790
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1e26bfe16a64aee8d94dc6a8042a8a4cd722ade1
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Aliah Almusaireae almusair...@kacst.edu.sa
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] zoomandviewlayout.ui widget

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3794

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3794
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d607a5960ffc3c69ffa97ca3e2ebae555044581
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Aliah Almusaireae almusair...@kacst.edu.sa
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#64531 : Tab-stops exported to DOCX added too much indent...

2013-05-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3884

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3884
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I12747006c8aac663f9529b495914f4a08301c66f
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Adam CloudOn rattles2...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED libreoffice-4-0] fdo#42144 WW8 export: avoid writing pointless empty footers

2013-05-06 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3797

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3797
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ied55dc2a9f0faef87689c07935a47db384d37e1a
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED libreoffice-4-0] SwPostItMgr::Delete: fix crash when layout is not yet ready

2013-05-03 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3757

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3757
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5fd5c908bc7b1feba926214863e84391b9037484
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] make parent style for 'frame contents' be 'standard' rather ...

2013-05-02 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3697

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3697
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe7bfd6a2820cff5cf29c669fd60b575c63500b0
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Luboš Luňák l.lu...@suse.cz
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Luboš Luňák l.lu...@suse.cz
Gerrit-Reviewer: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[PUSHED libreoffice-4-0] Fix out-of-bonds Sequence access in NumberingManager with pa...

2013-05-02 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3739

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3739
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib35fcee1605215fb22ae9838b29fe1d8be5eaf58
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Fridrich Strba fridr...@documentfoundation.org
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fix fdo#40496: don't reset RES_PARATR_ADJUST RES_FRAMEDIR ...

2013-04-02 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3146

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3146
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I69298632d56decbce3eb7058b982d845cb3e7240
Gerrit-PatchSet: 4
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: navin patidar pati...@kacst.edu.sa
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Lior Kaplan kaplanl...@gmail.com

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


[PATCH libreoffice-4-0] Templates Manager: merged the Double Click Handling into Tem...

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3087

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/87/3087/1

Templates Manager: merged the Double Click Handling into TemplateAbstractView

(cherry picked from commit 8a2bfe697bf71c8a2ca3c6ba1ef5e6324d4163aa)

Conflicts:
sfx2/inc/sfx2/templateview.hxx
sfx2/source/control/templateabstractview.cxx
sfx2/source/control/templatelocalview.cxx
sfx2/source/control/templateview.cxx

Change-Id: Iafc1650d57d29fbc24921498f18da6c0e510b37a
---
M sfx2/inc/sfx2/templateabstractview.hxx
M sfx2/inc/sfx2/templatelocalview.hxx
M sfx2/inc/sfx2/templateremoteview.hxx
M sfx2/inc/sfx2/templateview.hxx
M sfx2/source/control/templateabstractview.cxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/control/templateremoteview.cxx
M sfx2/source/control/templateview.cxx
M sfx2/source/doc/templatedlg.cxx
9 files changed, 39 insertions(+), 68 deletions(-)



diff --git a/sfx2/inc/sfx2/templateabstractview.hxx 
b/sfx2/inc/sfx2/templateabstractview.hxx
index ca51181..a5fef71 100644
--- a/sfx2/inc/sfx2/templateabstractview.hxx
+++ b/sfx2/inc/sfx2/templateabstractview.hxx
@@ -87,7 +87,7 @@
 
 virtual void filterTemplatesByApp (const FILTER_APPLICATION eApp);
 
-virtual void showOverlay (bool bVisible) = 0;
+void showOverlay (bool bVisible);
 
 void setItemDimensions (long ItemWidth, long ThumbnailHeight, long 
DisplayHeight, int itemPadding);
 
@@ -109,7 +109,7 @@
 
 void setOverlayItemStateHdl (const Link aLink) { maOverlayItemStateHdl = 
aLink; }
 
-void setOverlayDblClickHdl (const Link rLink);
+void setOpenHdl (const Link rLink);
 
 void setOverlayCloseHdl (const Link rLink);
 
@@ -127,10 +127,13 @@
 
 DECL_LINK(OverlayItemStateHdl, const ThumbnailViewItem*);
 
+virtual void OnItemDblClicked(ThumbnailViewItem *pItem);
+
 protected:
 
 TemplateView *mpItemView;
 Link maOverlayItemStateHdl;
+Link maOpenHdl;
 
 bool mbFilteredResults; // Flag keep track if overlay has been 
filtered so folders can get filtered too afterwards
 FILTER_APPLICATION meFilterOption;
diff --git a/sfx2/inc/sfx2/templatelocalview.hxx 
b/sfx2/inc/sfx2/templatelocalview.hxx
index 78cf547..432b008 100644
--- a/sfx2/inc/sfx2/templatelocalview.hxx
+++ b/sfx2/inc/sfx2/templatelocalview.hxx
@@ -39,8 +39,6 @@
 
 std::vectorOUString getFolderNames ();
 
-virtual void showOverlay (bool bVisible);
-
 std::vectorTemplateItemProperties
 getFilteredItems (const boost::functionbool (const 
TemplateItemProperties)  rFunc) const;
 
@@ -72,10 +70,6 @@
 bool isTemplateNameUnique (const sal_uInt16 nRegionItemId, const OUString 
rName) const;
 
 virtual bool renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
-
-private:
-
-virtual void OnItemDblClicked (ThumbnailViewItem *pRegionItem);
 
 private:
 
diff --git a/sfx2/inc/sfx2/templateremoteview.hxx 
b/sfx2/inc/sfx2/templateremoteview.hxx
index bd1bf5c..b760f00 100644
--- a/sfx2/inc/sfx2/templateremoteview.hxx
+++ b/sfx2/inc/sfx2/templateremoteview.hxx
@@ -25,8 +25,6 @@
 
 virtual ~TemplateRemoteView ();
 
-virtual void showOverlay (bool bVisible);
-
 bool loadRepository (TemplateRepository* pRepository, bool bRefresh);
 
 private:
diff --git a/sfx2/inc/sfx2/templateview.hxx b/sfx2/inc/sfx2/templateview.hxx
index e219905..aaa7334 100644
--- a/sfx2/inc/sfx2/templateview.hxx
+++ b/sfx2/inc/sfx2/templateview.hxx
@@ -36,26 +36,28 @@
 
 void InsertItems (const std::vectorTemplateItemProperties rTemplates);
 
-void setDblClickHdl (const Link rLink) { maDblClickHdl = rLink; }
-
 void setCloseHdl (const Link rLink) { maAllButton.SetClickHdl(rLink); }
 void setMasterView(TemplateAbstractView* pMasterView) { mpMasterView = 
pMasterView; }
 virtual bool renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+
+
+// FIXME Kept only during the refactoring
+void setOpenHdl (const Link rLink) { maOpenHdl = rLink; }
+virtual void OnItemDblClicked(ThumbnailViewItem *pItem) { 
maOpenHdl.Call(pItem); }
 
 protected:
 
 virtual void Resize ();
 
-virtual void OnItemDblClicked (ThumbnailViewItem *pItem);
-
 private:
 TemplateAbstractView* mpMasterView;
+
+Link maOpenHdl;
 
 PushButton maAllButton;
 FixedText  maFTName;
 sal_uInt16 mnId;
 OUString maName;
-Link maDblClickHdl;
 };
 
 #endif // TEMPLATEVIEW_HXX
diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index 14ab811..43efcbb 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -217,9 +217,10 @@
 mpItemView-filterItems(ViewFilter_Keyword(rKeyword));
 }
 
-void TemplateAbstractView::setOverlayDblClickHdl(const Link rLink)
+void TemplateAbstractView::setOpenHdl(const Link rLink)
 {
-mpItemView-setDblClickHdl(rLink);
+

[PATCH libreoffice-4-0] Deselect thumbnailview items when clearing the view.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3089

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/89/3089/1

Deselect thumbnailview items when clearing the view.

Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
(cherry picked from commit 1c9f13f921cb5bf955b8568ffb5d14047738eef9)
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 10 insertions(+), 0 deletions(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index ca685cc..b3b1251 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -114,6 +114,16 @@
 for ( size_t i = 0; i  n; ++i )
 {
 ThumbnailViewItem *const pItem = mItemList[i];
+
+// deselect all current selected items and fire events
+if (pItem-isSelected())
+{
+pItem-setSelection(false);
+maItemStateHdl.Call(pItem);
+
+// fire accessible event???
+}
+
 if ( pItem-isVisible()  ImplHasAccessibleListeners() )
 {
 ::com::sun::star::uno::Any aOldAny, aNewAny;

-- 
To view, visit https://gerrit.libreoffice.org/3089
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Rafael Dominguez venccsra...@gmail.com

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


[PATCH libreoffice-4-0] Fix template manager item selection behaviour with mouse.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3090

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/90/3090/1

Fix template manager item selection behaviour with mouse.

Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
(cherry picked from commit 28d5ced7f9982257d263ffc3e550bd479a9542f8)
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 14 insertions(+), 9 deletions(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index b3b1251..8b2840d 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -609,15 +609,22 @@
 {
 if ( rMEvt.GetClicks() == 1 )
 {
-if (pItem-isSelected()  rMEvt.IsMod1())
-pItem-setSelection(false);
+if (rMEvt.IsMod1())
+{
+//Keep selected item group state and just invert current 
desired one state
+pItem-setSelection(!pItem-isSelected());
+}
 else
 {
-if (!pItem-isSelected()  !rMEvt.IsMod1())
-deselectItems( );
-
+//If we got a group of selected items deselect the rest 
and only keep the desired one
+//mark items as not selected to not fire unnecessary 
change state events.
+pItem-setSelection(false);
+deselectItems();
 pItem-setSelection(true);
+}
 
+if (pItem-isSelected())
+{
 bool bClickOnTitle = 
pItem-getTextArea().IsInside(rMEvt.GetPosPixel());
 pItem-setEditTitle(bClickOnTitle);
 }
@@ -626,13 +633,11 @@
 DrawItem(pItem);
 
 maItemStateHdl.Call(pItem);
+
+//fire accessible event??
 }
 else if ( rMEvt.GetClicks() == 2 )
 {
-// The mouse button down event 1 click right before is 
pointless
-pItem-setSelection(false);
-maItemStateHdl.Call(pItem);
-
 Rectangle aRect(pItem-getDrawArea());
 
 if (aRect.IsInside(rMEvt.GetPosPixel()))

-- 
To view, visit https://gerrit.libreoffice.org/3090
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Rafael Dominguez venccsra...@gmail.com

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


[PATCH libreoffice-4-0] Syncronize thumbnails ids with SfxDocumentTemplates when mov...

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3091

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/91/3091/1

Syncronize thumbnails ids with SfxDocumentTemplates when moving.

Change-Id: I9ee17e00f769ca16f566b1397d09b76a09273912
(cherry picked from commit dda5fe60ff6dfd61f5f262c61e1d538d20e9c978)
---
M sfx2/inc/sfx2/templatelocalview.hxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/doc/templatedlg.cxx
3 files changed, 62 insertions(+), 47 deletions(-)



diff --git a/sfx2/inc/sfx2/templatelocalview.hxx 
b/sfx2/inc/sfx2/templatelocalview.hxx
index 3bcacc4..68b7dc7 100644
--- a/sfx2/inc/sfx2/templatelocalview.hxx
+++ b/sfx2/inc/sfx2/templatelocalview.hxx
@@ -56,9 +56,9 @@
 bool removeTemplate (const sal_uInt16 nItemId, const sal_uInt16 
nSrcItemId);
 
 bool moveTemplate (const ThumbnailViewItem* pItem, const sal_uInt16 
nSrcItem,
-   const sal_uInt16 nTargetItem, bool bCopy);
+   const sal_uInt16 nTargetItem);
 
-bool moveTemplates (const std::setconst 
ThumbnailViewItem*,selection_cmp_fn rItems, const sal_uInt16 nTargetItem, 
bool bCopy);
+bool moveTemplates (const std::setconst 
ThumbnailViewItem*,selection_cmp_fn rItems, const sal_uInt16 nTargetItem);
 
 bool copyFrom (const sal_uInt16 nRegionItemId, const BitmapEx rThumbnail, 
const OUString rPath);
 
diff --git a/sfx2/source/control/templatelocalview.cxx 
b/sfx2/source/control/templatelocalview.cxx
index ab55ca4..429b4dc 100644
--- a/sfx2/source/control/templatelocalview.cxx
+++ b/sfx2/source/control/templatelocalview.cxx
@@ -289,11 +289,8 @@
 }
 
 bool TemplateLocalView::moveTemplate (const ThumbnailViewItem *pItem, const 
sal_uInt16 nSrcItem,
-   const sal_uInt16 nTargetItem, bool 
bCopy)
+   const sal_uInt16 nTargetItem)
 {
-bool bRet = true;
-bool bRefresh = false;
-
 TemplateContainerItem *pTarget = NULL;
 TemplateContainerItem *pSrc = NULL;
 
@@ -313,16 +310,13 @@
 
 const TemplateViewItem *pViewItem = static_castconst 
TemplateViewItem*(pItem);
 
-bool bOK;
+bool bCopy = 
!mpDocTemplates-Move(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem-mnDocId);
 
 if (bCopy)
-bOK = 
mpDocTemplates-Copy(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem-mnId-1);
-else
-bOK = 
mpDocTemplates-Move(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem-mnId-1);
-
-if (!bOK)
-return false;
-
+{
+if 
(!mpDocTemplates-Copy(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem-mnDocId))
+return false;
+}
 // move template to destination
 
 TemplateItemProperties aTemplateItem;
@@ -340,37 +334,45 @@
 // remove template from region cached data
 
 std::vectorTemplateItemProperties::iterator aIter;
-for (aIter = pSrc-maTemplates.begin(); aIter != 
pSrc-maTemplates.end(); ++aIter)
+for (aIter = pSrc-maTemplates.begin(); aIter != 
pSrc-maTemplates.end();)
 {
-if (aIter-nId == pViewItem-mnId)
+if (aIter-nDocId == pViewItem-mnDocId)
 {
-pSrc-maTemplates.erase(aIter);
-
-RemoveItem(pViewItem-mnId);
-break;
+aIter = pSrc-maTemplates.erase(aIter);
 }
+else
+{
+// Keep region document id syncronized with 
SfxDocumentTemplates
+if (aIter-nDocId  pViewItem-mnDocId)
+--aIter-nDocId;
+
+++aIter;
+}
+}
+
+// Keep view document id syncronized with SfxDocumentTemplates
+std::vectorThumbnailViewItem*::iterator pItemIter = 
mItemList.begin();
+for (; pItemIter != mItemList.end(); ++pItemIter)
+{
+if (static_castTemplateViewItem*(*pItemIter)-mnDocId  
pViewItem-mnDocId)
+--static_castTemplateViewItem*(*pItemIter)-mnDocId;
 }
 }
 
-bRefresh = true;
-}
-else
-bRet = false;
-
-if (bRefresh)
-{
 lcl_updateThumbnails(pSrc);
 lcl_updateThumbnails(pTarget);
 
 CalculateItemPositions();
 Invalidate();
+
+return true;
 }
 
-return bRet;
+return false;
 }
 
 bool TemplateLocalView::moveTemplates(const std::setconst ThumbnailViewItem*, 
selection_cmp_fn rItems,
-  const sal_uInt16 nTargetItem, bool bCopy)
+  const sal_uInt16 nTargetItem)
 {
 bool ret = true;
 bool refresh = false;
@@ -400,17 +402,15 @@
 {
 const TemplateViewItem *pViewItem = static_castconst 
TemplateViewItem*(*aSelIter);
 
-bool bOK;
+  

[PUSHED libreoffice-4-0] fdo#61429: Stylist: save selection of Hierarchical entry

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3056

Approvals:
  Christoph Brill: Looks good to me, but someone else must approve
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3056
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie2bc9ce2690fbc742b4510e453ca1ef310614256
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Christoph Brill egore...@gmail.com

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


[PUSHED] Separate template region ids from thumbnail ids.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3075

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3075
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I22da25a4bbba4604f5df42f440e6532076d39457
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Fix deleting templates through Template Manager.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3076

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3076
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic59d3d666ae5f3808ef7dd7324589a731a841186
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Delete the correct template from the selected search results...

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3077

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3077
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2fe82b20e9b4a78a3e8e587a5720054f845d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Fix exporting selected search results in Template Manager.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3078

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3078
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0716b3f2833bdcc78a67f1b0874b92a59df705f7
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Fix moving selected search results in Template Manager.

2013-03-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3079

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/3079
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I51ae74f91845255dc1f332e33c8e708e5af1ead0
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Fix template manager item selection behaviour with mouse.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3053


-- 
To view, visit https://gerrit.libreoffice.org/3053
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Syncronize thumbnails ids with SfxDocumentTemplates when mov...

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3054


-- 
To view, visit https://gerrit.libreoffice.org/3054
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9ee17e00f769ca16f566b1397d09b76a09273912
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Merge TemplateView into TemplateAbstractView class.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3051


-- 
To view, visit https://gerrit.libreoffice.org/3051
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iafc246eeff4c50c12ac9ebec9fe95dcc84991e9e
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Deselect thumbnailview items when clearing the view.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/3052


-- 
To view, visit https://gerrit.libreoffice.org/3052
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] fdo#61390: allow arrow keys to show hidden thumbnail items

2013-03-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3033

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/33/3033/1

fdo#61390: allow arrow keys to show hidden thumbnail items

Change-Id: I70924b4c9578122a1c5a5b6c8a67889d6d003a2e
(cherry picked from commit cc599e914e7ade9f10da460fc1c77ea07454d7ff)
---
M sfx2/inc/sfx2/thumbnailview.hxx
M sfx2/source/control/thumbnailview.cxx
M sfx2/source/doc/templatedlg.cxx
3 files changed, 31 insertions(+), 3 deletions(-)



diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 95e6b53..9e29c9a 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -277,6 +277,7 @@
 using Window::ImplInit;
 
 void CalculateItemPositions ();
+void MakeItemVisible( sal_uInt16 nId );
 
 SFX2_DLLPRIVATE void ImplInit();
 SFX2_DLLPRIVATE void ImplInitSettings( bool bFont, bool 
bForeground, bool bBackground );
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index f78a308..08be7cb 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -553,11 +553,38 @@
 Control::KeyInput( rKEvt );
 }
 
-if ( pNext  pNext-isVisible() )
+if ( pNext )
 {
 deselectItems();
 SelectItem(pNext-mnId);
+MakeItemVisible(pNext-mnId);
 }
+}
+
+void ThumbnailView::MakeItemVisible( sal_uInt16 nItemId )
+{
+// Get the item row
+size_t nPos = 0;
+bool bFound = false;
+for ( size_t i = 0; !bFound  i  mFilteredItemList.size(); ++i )
+{
+ThumbnailViewItem* pItem = mFilteredItemList[i];
+if ( pItem-mnId == nItemId )
+{
+nPos = i;
+bFound = true;
+}
+}
+sal_uInt16 nRow = nPos / mnCols;
+
+// Move the visible rows as little as possible to include that one
+if ( nRow  mnFirstLine )
+mnFirstLine = nRow;
+else if ( nRow  mnFirstLine + mnVisLines )
+mnFirstLine = nRow - mnVisLines;
+
+CalculateItemPositions();
+Invalidate();
 }
 
 void ThumbnailView::MouseButtonDown( const MouseEvent rMEvt )
@@ -703,11 +730,11 @@
 
 void ThumbnailView::Resize()
 {
+Control::Resize();
 CalculateItemPositions();
 
 if ( IsReallyVisible()  IsUpdateMode() )
 Invalidate();
-Control::Resize();
 }
 
 void ThumbnailView::StateChanged( StateChangedType nType )
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index b9d3945..e5e4014 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -331,7 +331,7 @@
 Point aViewPos = maView-GetPosPixel();
 aViewPos.setY(nToolbarsHeight);
 aViewPos.setX(0);
-Size aThumbSize(aWinSize.getWidth(), 
maTabControl.GetTabPageSizePixel().getWidth() - aViewPos.getY());
+Size aThumbSize(aWinSize.getWidth(), 
maTabControl.GetTabPageSizePixel().getHeight() - aViewPos.getY());
 maView-SetPosSizePixel(aViewPos, aThumbSize);
 
 if (aWinSize.getHeight()  aViewPos.getY() + aThumbSize.getHeight() + 
PADDING_DLG_BORDER)

-- 
To view, visit https://gerrit.libreoffice.org/3033
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I70924b4c9578122a1c5a5b6c8a67889d6d003a2e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] fdo#62659: only select items in ThumbnailView on TabStop

2013-03-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3034

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/34/3034/1

fdo#62659: only select items in ThumbnailView on TabStop

Change-Id: I024df8520f0daf2cc2e9e5f041e1ff18cb3689ec
(cherry picked from commit f8a3582d4cbcf6a705f724b921e370c739ecee35)
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 13 insertions(+), 9 deletions(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 08be7cb..0da633f 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -126,6 +126,7 @@
 }
 
 mItemList.clear();
+mFilteredItemList.clear();
 }
 
 void ThumbnailView::ImplInitSettings( bool bFont, bool bForeground, bool 
bBackground )
@@ -680,17 +681,20 @@
 
 void ThumbnailView::GetFocus()
 {
-// Select the first item if nothing selected
-int nSelected = -1;
-for (size_t i = 0, n = mItemList.size(); i  n  nSelected == -1; ++i)
+if(GETFOCUS_TAB  GetGetFocusFlags())
 {
-if (mItemList[i]-isSelected())
-nSelected = i;
-}
+// Select the first item if nothing selected
+int nSelected = -1;
+for (size_t i = 0, n = mItemList.size(); i  n  nSelected == -1; ++i)
+{
+if (mItemList[i]-isSelected())
+nSelected = i;
+}
 
-if ( nSelected == -1  mItemList.size( )  0 )
-{
-SelectItem( 1 );
+if ( nSelected == -1  mItemList.size( )  0 )
+{
+SelectItem( 1 );
+}
 }
 
 // Tell the accessible object that we got the focus.

-- 
To view, visit https://gerrit.libreoffice.org/3034
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I024df8520f0daf2cc2e9e5f041e1ff18cb3689ec
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Template Manager: fixed control WinBits reset... was breakin...

2013-03-22 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2909

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/09/2909/1

Template Manager: fixed control WinBits reset... was breaking tabstop

Change-Id: I6e542ecc0e0f5a2b3e0ca59a52bb809b1a23284f
(cherry picked from commit 5fa5f95612c53511fb951660b4ad397321e8)
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index e9f85eb..b9d3945 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -163,7 +163,7 @@
 
mpSearchEdit-SetUpdateDataHdl(LINK(this,SfxTemplateManagerDlg,SearchUpdateHdl));
 mpSearchEdit-EnableUpdateData();
 
-maView-SetStyle(WB_VSCROLL);
+maView-SetStyle(maView-GetStyle() | WB_VSCROLL);
 maView-setItemMaxTextLength(TEMPLATE_ITEM_MAX_TEXT_LENGTH);
 
 
maView-setItemDimensions(TEMPLATE_ITEM_MAX_WIDTH,TEMPLATE_ITEM_THUMBNAIL_MAX_HEIGHT,

-- 
To view, visit https://gerrit.libreoffice.org/2909
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6e542ecc0e0f5a2b3e0ca59a52bb809b1a23284f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Template Manager: select first item when getting focus if no...

2013-03-22 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2910

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/10/2910/1

Template Manager: select first item when getting focus if no selection

Change-Id: Ie012e010ca92728725535345c76fc9f5eb31a254
(cherry picked from commit ec2f84b376f595b0553297d25b7716f6b97e1b87)
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 18 insertions(+), 1 deletion(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index c996f0f..57dba07 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -579,12 +579,29 @@
 
 void ThumbnailView::GetFocus()
 {
-Control::GetFocus();
+// Select the first item if nothing selected
+int nSelected = -1;
+for (size_t i = 0, n = mItemList.size(); i  n  nSelected == -1; ++i)
+{
+if (mItemList[i]-isSelected())
+nSelected = i;
+}
+
+if ( nSelected == -1  mItemList.size( )  0 )
+{
+mItemList[0]-setSelection(true);
+maItemStateHdl.Call(mItemList[0]);
+
+if (IsReallyVisible()  IsUpdateMode())
+Invalidate();
+}
 
 // Tell the accessible object that we got the focus.
 ThumbnailViewAcc* pAcc = ThumbnailViewAcc::getImplementation( 
GetAccessible( sal_False ) );
 if( pAcc )
 pAcc-GetFocus();
+
+Control::GetFocus();
 }
 
 void ThumbnailView::LoseFocus()

-- 
To view, visit https://gerrit.libreoffice.org/2910
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie012e010ca92728725535345c76fc9f5eb31a254
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Template Manager: removed some dead selection code

2013-03-22 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2911

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/11/2911/1

Template Manager: removed some dead selection code

(cherry picked from commit bfd1bcb204ed3ca35df1455a346a0ee7254a1191)

Conflicts:
sfx2/source/control/thumbnailview.cxx

Change-Id: Ib5ebcd928e77a115f4f62a50724656c33ae13c61
---
M sfx2/inc/sfx2/thumbnailview.hxx
M sfx2/source/control/thumbnailview.cxx
M sfx2/source/control/thumbnailviewacc.cxx
3 files changed, 24 insertions(+), 111 deletions(-)



diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 8eb9426..efc5e78 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -202,10 +202,7 @@
 
 void SelectItem( sal_uInt16 nItemId );
 
-sal_uInt16 GetSelectItemId() const { return mnSelItemId; }
-
-bool IsItemSelected( sal_uInt16 nItemId ) const
-{ return nItemId == mnSelItemId; }
+bool IsItemSelected( sal_uInt16 nItemId ) const;
 
 void deselectItem (const sal_uInt16 nItemId);
 
@@ -258,8 +255,6 @@
 
 virtual void DataChanged( const DataChangedEvent rDCEvt );
 
-virtual bool StartDrag( const CommandEvent rCEvt, Region rRegion );
-
 virtual ::com::sun::star::uno::Reference 
::com::sun::star::accessibility::XAccessible  CreateAccessible();
 
 protected:
@@ -310,7 +305,6 @@
 long mnVisLines;
 long mnLines;
 sal_uInt16 mnScrBarOffset;
-sal_uInt16 mnSelItemId;
 sal_uInt16 mnHighItemId;
 sal_uInt16 mnCols;
 sal_uInt16 mnFirstLine;
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 57dba07..63ba189 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -92,7 +92,6 @@
 mnLines = 0;
 mnFirstLine = 0;
 mnScrBarOffset = 1;
-mnSelItemId = 0;
 mnHighItemId= 0;
 mnCols  = 0;
 mnSpacing   = 0;
@@ -589,11 +588,7 @@
 
 if ( nSelected == -1  mItemList.size( )  0 )
 {
-mItemList[0]-setSelection(true);
-maItemStateHdl.Call(mItemList[0]);
-
-if (IsReallyVisible()  IsUpdateMode())
-Invalidate();
+SelectItem( 1 );
 }
 
 // Tell the accessible object that we got the focus.
@@ -710,10 +705,9 @@
 }
 
 // reset variables
-if ( (mnHighItemId == nItemId) || (mnSelItemId == nItemId) )
+if ( (mnHighItemId == nItemId) )
 {
 mnHighItemId= 0;
-mnSelItemId = 0;
 }
 
 CalculateItemPositions();
@@ -729,7 +723,6 @@
 // reset variables
 mnFirstLine = 0;
 mnHighItemId= 0;
-mnSelItemId = 0;
 
 CalculateItemPositions();
 
@@ -777,19 +770,18 @@
 
 void ThumbnailView::SelectItem( sal_uInt16 nItemId )
 {
-size_t nItemPos = 0;
+size_t nItemPos = GetItemPos( nItemId );
+if ( nItemPos == THUMBNAILVIEW_ITEM_NOTFOUND )
+return;
 
-if ( nItemId )
+ThumbnailViewItem* pItem = mItemList[nItemPos];
+if (!pItem-isSelected())
 {
-nItemPos = GetItemPos( nItemId );
-if ( nItemPos == THUMBNAILVIEW_ITEM_NOTFOUND )
-return;
-}
+mItemList[nItemPos]-setSelection(true);
+maItemStateHdl.Call(mItemList[nItemPos]);
 
-if ( mnSelItemId != nItemId)
-{
-sal_uInt16 nOldItem = mnSelItemId ? mnSelItemId : 1;
-mnSelItemId = nItemId;
+if (IsReallyVisible()  IsUpdateMode())
+Invalidate();
 
 bool bNewOut = IsReallyVisible()  IsUpdateMode();
 
@@ -815,44 +807,8 @@
 
 if( ImplHasAccessibleListeners() )
 {
-// focus event (deselect)
-if( nOldItem )
-{
-const size_t nPos = GetItemPos( nItemId );
-
-if( nPos != THUMBNAILVIEW_ITEM_NOTFOUND )
-{
-ThumbnailViewAcc* pItemAcc = 
ThumbnailViewAcc::getImplementation(
-mItemList[nPos]-GetAccessible( 
mbIsTransientChildrenDisabled ) );
-
-if( pItemAcc )
-{
-::com::sun::star::uno::Any aOldAny, aNewAny;
-if( !mbIsTransientChildrenDisabled )
-{
-aOldAny = ::com::sun::star::uno::Reference 
::com::sun::star::uno::XInterface (
-static_cast ::cppu::OWeakObject* ( pItemAcc 
));
-ImplFireAccessibleEvent 
(::com::sun::star::accessibility::AccessibleEventId::ACTIVE_DESCENDANT_CHANGED, 
aOldAny, aNewAny );
-}
-else
-{
-aOldAny = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
-pItemAcc-FireAccessibleEvent( 

[PATCH libreoffice-4-0] fdo#61390: simple keybard support in TemplateManager

2013-03-22 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2912

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/12/2912/1

fdo#61390: simple keybard support in TemplateManager

Adds support for the UP, DOWN, LEFT, RIGHT and RETURN keys in the
thumbnails view but doesn't handle the modifiers yet. There are still
some problems with the focus and key input outside the top level

(cherry picked from commit ee819bdd2dab5756cc3bad74f24e50bd7409f308)

Conflicts:
sfx2/source/control/thumbnailview.cxx

Change-Id: I5ba67583c835bcc00b075071411c0d6590a07f9a
---
M sfx2/inc/sfx2/thumbnailview.hxx
M sfx2/source/control/thumbnailview.cxx
2 files changed, 104 insertions(+), 7 deletions(-)



diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index efc5e78..95e6b53 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -202,6 +202,8 @@
 
 void SelectItem( sal_uInt16 nItemId );
 
+void DeselectItem( sal_uInt16 nItemId );
+
 bool IsItemSelected( sal_uInt16 nItemId ) const;
 
 void deselectItem (const sal_uInt16 nItemId);
@@ -239,6 +241,8 @@
 
 protected:
 
+virtual void KeyInput( const KeyEvent rKEvt );
+
 virtual void MouseButtonDown( const MouseEvent rMEvt );
 
 virtual void MouseButtonUp( const MouseEvent rMEvt );
@@ -272,8 +276,6 @@
 using Control::ImplInitSettings;
 using Window::ImplInit;
 
-void calculateColumnsRows ();
-
 void CalculateItemPositions ();
 
 SFX2_DLLPRIVATE void ImplInit();
@@ -294,6 +296,7 @@
 protected:
 
 ValueItemList mItemList;
+ValueItemList mFilteredItemList; /// Cache to store the filtered items
 ScrollBar* mpScrBar;
 Rectangle maItemListRect;
 long mnHeaderHeight;
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 63ba189..f78a308 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -214,6 +214,8 @@
 WinBits nStyle = GetStyle();
 ScrollBar*  pDelScrBar = NULL;
 
+mFilteredItemList.clear();
+
 // consider the scrolling
 if ( nStyle  WB_VSCROLL )
 ImplInitScrollBar();
@@ -298,6 +300,7 @@
 
 if (maFilterFunc(pItem))
 {
+mFilteredItemList.push_back(pItem);
 if ((nCurCount = nFirstItem)  (nCurCount  nLastItem))
 {
 if( !pItem-isVisible())
@@ -490,6 +493,73 @@
 return 0;
 }
 
+void ThumbnailView::KeyInput( const KeyEvent rKEvt )
+{
+// Get the last selected item in the list
+size_t nLastPos = 0;
+bool bFoundLast = false;
+for ( long i = mFilteredItemList.size() - 1; !bFoundLast  i = 0; --i )
+{
+ThumbnailViewItem* pItem = mFilteredItemList[i];
+if ( pItem-isSelected() )
+{
+nLastPos = i;
+bFoundLast = true;
+}
+}
+
+KeyCode aKeyCode = rKEvt.GetKeyCode();
+ThumbnailViewItem* pNext = NULL;
+switch ( aKeyCode.GetCode() )
+{
+case KEY_RIGHT:
+{
+size_t nNextPos = nLastPos;
+if ( bFoundLast  nLastPos  mFilteredItemList.size( ) - 1 )
+nNextPos = nLastPos + 1;
+pNext = mFilteredItemList[nNextPos];
+}
+break;
+case KEY_LEFT:
+{
+size_t nNextPos = nLastPos;
+if ( nLastPos  0 )
+nNextPos = nLastPos - 1;
+pNext = mFilteredItemList[nNextPos];
+}
+break;
+case KEY_DOWN:
+{
+size_t nNextPos = nLastPos;
+if ( bFoundLast  nLastPos  mFilteredItemList.size( ) - 
mnCols )
+nNextPos = nLastPos + mnCols;
+pNext = mFilteredItemList[nNextPos];
+}
+break;
+case KEY_UP:
+{
+size_t nNextPos = nLastPos;
+if ( nLastPos = mnCols )
+nNextPos = nLastPos - mnCols;
+pNext = mFilteredItemList[nNextPos];
+}
+break;
+case KEY_RETURN:
+{
+if ( bFoundLast )
+OnItemDblClicked( mFilteredItemList[nLastPos] );
+}
+default:
+Control::KeyInput( rKEvt );
+}
+
+if ( pNext  pNext-isVisible() )
+{
+deselectItems();
+SelectItem(pNext-mnId);
+}
+}
+
 void ThumbnailView::MouseButtonDown( const MouseEvent rMEvt )
 {
 if ( rMEvt.IsLeft() )
@@ -500,12 +570,17 @@
 {
 if ( rMEvt.GetClicks() == 1 )
 {
-if (!pItem-isSelected()  !rMEvt.IsMod1())
-deselectItems( );
-pItem-setSelection(true);
+if (pItem-isSelected()  rMEvt.IsMod1())
+DeselectItem( pItem-mnId );
+else
+ 

[PATCH libreoffice-4-0] fdo#61390: Fixed the tabstop in the TemplateView.

2013-03-22 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2913

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/13/2913/1

fdo#61390: Fixed the tabstop in the TemplateView.

Change-Id: I887b11646efb9a839da11cd3e20cdf7e532caf3e
(cherry picked from commit 7a25aa2821ccf7318ce4a13efe171763989009d5)
---
M sfx2/inc/sfx2/templateview.hxx
M sfx2/source/control/templateabstractview.cxx
M sfx2/source/control/templateview.cxx
M sfx2/source/control/templateview.src
4 files changed, 13 insertions(+), 20 deletions(-)



diff --git a/sfx2/inc/sfx2/templateview.hxx b/sfx2/inc/sfx2/templateview.hxx
index 3e2a7c8..e219905 100644
--- a/sfx2/inc/sfx2/templateview.hxx
+++ b/sfx2/inc/sfx2/templateview.hxx
@@ -51,7 +51,6 @@
 private:
 TemplateAbstractView* mpMasterView;
 
-ControlmaButtons;
 PushButton maAllButton;
 FixedText  maFTName;
 sal_uInt16 mnId;
diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index de2091f..14ab811 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -114,7 +114,7 @@
 
 TemplateAbstractView::TemplateAbstractView (Window *pParent, WinBits 
nWinStyle, bool bDisableTransientChildren)
 : ThumbnailView(pParent,nWinStyle,bDisableTransientChildren),
-  mpItemView(new TemplateView(this)),
+  mpItemView(new TemplateView(pParent)),
   mbFilteredResults(false),
   meFilterOption(FILTER_APP_WRITER)
 {
@@ -123,7 +123,7 @@
 
 TemplateAbstractView::TemplateAbstractView(Window *pParent, const ResId 
rResId, bool bDisableTransientChildren)
 : ThumbnailView(pParent,rResId,bDisableTransientChildren),
-  mpItemView(new TemplateView(this)),
+  mpItemView(new TemplateView(pParent)),
   mbFilteredResults(false),
   meFilterOption(FILTER_APP_WRITER)
 {
@@ -190,8 +190,14 @@
 
 void TemplateAbstractView::showOverlay (bool bVisible)
 {
+Show(!bVisible);
 mpItemView-Show(bVisible);
 
+mpItemView-SetPosSizePixel(GetPosPixel(), GetSizePixel());
+mpItemView-SetStyle(GetStyle());
+
+mpItemView-GrabFocus();
+
 // Clear items is the overlay is closed.
 if (!bVisible)
 {
diff --git a/sfx2/source/control/templateview.cxx 
b/sfx2/source/control/templateview.cxx
index 615465d..46ebeaa 100644
--- a/sfx2/source/control/templateview.cxx
+++ b/sfx2/source/control/templateview.cxx
@@ -34,14 +34,13 @@
 using namespace drawinglayer::primitive2d;
 
 TemplateView::TemplateView (Window *pParent)
-: ThumbnailView(pParent,WB_VSCROLL),
+: ThumbnailView(pParent,WB_VSCROLL | WB_TABSTOP),
   mpMasterView(NULL),
-  maButtons(this, SfxResId(CONTROL_BUTTONS)),
-  maAllButton(maButtons, SfxResId(BTN_ALL_TEMPLATES)),
-  maFTName(maButtons, SfxResId(FT_NAME)),
+  maAllButton(this, SfxResId(BTN_ALL_TEMPLATES)),
+  maFTName(this, SfxResId(FT_NAME)),
   mnId(0)
 {
-mnHeaderHeight = maButtons.GetSizePixel().getHeight();
+mnHeaderHeight = maAllButton.GetSizePixel().getHeight() + 
maAllButton.GetPosPixel().Y() * 2;
 maAllButton.SetStyle(maAllButton.GetStyle() | WB_FLATBUTTON);
 }
 
@@ -88,13 +87,8 @@
 Size aWinSize = GetOutputSize();
 
 // Set the buttons panel and buttons size
-Size aPanelSize = maButtons.GetSizePixel();
-int nDeltaW = aWinSize.getWidth() - aPanelSize.getWidth();
-aPanelSize.setWidth(aWinSize.getWidth());
-maButtons.SetSizePixel(aPanelSize);
-
 Size aNameSize = maFTName.GetSizePixel();
-aNameSize.setWidth(aNameSize.getWidth() + nDeltaW);
+aNameSize.setWidth( aWinSize.getWidth() - maFTName.GetPosPixel().X());
 maFTName.SetSizePixel(aNameSize);
 
 ThumbnailView::Resize();
diff --git a/sfx2/source/control/templateview.src 
b/sfx2/source/control/templateview.src
index db4d575..8549c96 100644
--- a/sfx2/source/control/templateview.src
+++ b/sfx2/source/control/templateview.src
@@ -8,12 +8,6 @@
 
 #include templateview.hrc
 
-Control CONTROL_BUTTONS
-{
-Size = MAP_APPFONT( 290, 17 );
-TabStop = False;
-};
-
 PushButton BTN_ALL_TEMPLATES
 {
 Pos = MAP_APPFONT( 1, 1 );

-- 
To view, visit https://gerrit.libreoffice.org/2913
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I887b11646efb9a839da11cd3e20cdf7e532caf3e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Template manager: moved the controls into a proper TabPage t...

2013-03-19 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2843

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/43/2843/1

Template manager: moved the controls into a proper TabPage to avoid problems

Conflicts:
sfx2/inc/templatedlg.hxx
sfx2/source/doc/templatedlg.src

Change-Id: Ibff35bf24873a38c9a09d71fb3630a57e06f1a3c
---
M sfx2/inc/templatedlg.hxx
M sfx2/source/control/templateview.src
M sfx2/source/doc/doc.hrc
M sfx2/source/doc/templatedlg.cxx
M sfx2/source/doc/templatedlg.src
5 files changed, 98 insertions(+), 74 deletions(-)



diff --git a/sfx2/inc/templatedlg.hxx b/sfx2/inc/templatedlg.hxx
index 47c1989..095292e 100644
--- a/sfx2/inc/templatedlg.hxx
+++ b/sfx2/inc/templatedlg.hxx
@@ -15,6 +15,7 @@
 #include vcl/dialog.hxx
 #include vcl/button.hxx
 #include vcl/tabctrl.hxx
+#include vcl/tabpage.hxx
 
 class Edit;
 class PopupMenu;
@@ -126,6 +127,7 @@
 private:
 
 TabControl maTabControl;
+TabPage maTabPage;
 Control *mpToolbars;
 
 Edit *mpSearchEdit;
diff --git a/sfx2/source/control/templateview.src 
b/sfx2/source/control/templateview.src
index d2711a7..db4d575 100644
--- a/sfx2/source/control/templateview.src
+++ b/sfx2/source/control/templateview.src
@@ -11,7 +11,6 @@
 Control CONTROL_BUTTONS
 {
 Size = MAP_APPFONT( 290, 17 );
-Border = True;
 TabStop = False;
 };
 
@@ -20,6 +19,7 @@
 Pos = MAP_APPFONT( 1, 1 );
 Size = MAP_APPFONT( 50, 14 );
 Text [ en-US ] = All Templates;
+TabStop = True;
 };
 
 FixedText FT_NAME
diff --git a/sfx2/source/doc/doc.hrc b/sfx2/source/doc/doc.hrc
index f3edfa7..b56a473 100644
--- a/sfx2/source/doc/doc.hrc
+++ b/sfx2/source/doc/doc.hrc
@@ -39,6 +39,7 @@
 #define DLG_DOC_TEMPLATE(RID_SFX_DOC_START+2)
 #define DLG_ORGANIZE(RID_SFX_DOC_START+3)
 #define DLG_TEMPLATE_MANAGER(RID_SFX_DOC_START+4)
+#define TAB_TEMPLATE_MANAGER(RID_SFX_DOC_START+5)
 
 #define BMP_STYLES_CLOSED   (RID_SFX_DOC_START+ 0)
 #define BMP_STYLES_OPENED   (RID_SFX_DOC_START+ 1)
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 7041966..1d52d45 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -99,22 +99,38 @@
 OUString maKeyword;
 };
 
+class TemplateManagerPage : public TabPage
+{
+private:
+FixedText maFixedText;
+
+public:
+TemplateManagerPage( Window* pParent );
+~TemplateManagerPage( ) { };
+};
+
 SfxTemplateManagerDlg::SfxTemplateManagerDlg (Window *parent)
 : ModelessDialog(parent, SfxResId(DLG_TEMPLATE_MANAGER)),
   maTabControl(this,SfxResId(TAB_CONTROL)),
-  mpToolbars( new Control(maTabControl,SfxResId(TOOLBARS))),
+  maTabPage(maTabControl, SfxResId(TAB_TEMPLATE_MANAGER)),
+  mpToolbars( new Control(maTabPage,SfxResId(TOOLBARS))),
   mpSearchEdit(new Edit(this,WB_HIDE | WB_BORDER)),
   mpViewBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_VIEW))),
   mpActionBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_ACTION))),
   mpTemplateBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_TEMPLATES))),
-  mpSearchView(new TemplateSearchView(this)),
-  maView(new TemplateLocalView(this,SfxResId(TEMPLATE_VIEW))),
-  mpOnlineView(new TemplateRemoteView(this, WB_VSCROLL,false)),
+  mpSearchView(new TemplateSearchView(maTabPage)),
+  maView(new TemplateLocalView(maTabPage,SfxResId(TEMPLATE_VIEW))),
+  mpOnlineView(new TemplateRemoteView(maTabPage, WB_VSCROLL,false)),
   mbIsSaveMode(false),
   mxDesktop(comphelper::getProcessServiceFactory()-createInstance( 
com.sun.star.frame.Desktop ),uno::UNO_QUERY ),
   mbIsSynced(false),
   maRepositories()
 {
+maTabControl.SetTabPage( FILTER_DOCS, maTabPage );
+maTabControl.SetTabPage( FILTER_SHEETS, maTabPage );
+maTabControl.SetTabPage( FILTER_PRESENTATIONS, maTabPage );
+maTabControl.SetTabPage( FILTER_DRAWS, maTabPage );
+
 // Create popup menus
 mpActionMenu = new PopupMenu;
 
mpActionMenu-InsertItem(MNI_ACTION_SORT_NAME,SfxResId(STR_ACTION_SORT_NAME).toString(),SfxResId(IMG_ACTION_SORT));
@@ -293,20 +309,17 @@
 Size aTabSize = maTabControl.GetSizePixel();
 aTabSize.setWidth(aWinSize.getWidth());
 maTabControl.SetSizePixel(aTabSize);
-Size aTabPageSize = maTabControl.GetTabPageSizePixel();
-Point aToolbarsPos(0, aTabSize.getHeight() - aTabPageSize.getHeight());
-mpToolbars-SetPosPixel(aToolbarsPos);
-aTabPageSize.setHeight(mpToolbars-GetSizePixel().getHeight() + 3);
-maTabControl.SetTabPageSizePixel(aTabPageSize);
-
-Size aToolbarsSize = mpToolbars-GetSizePixel();
-aToolbarsSize.setWidth(aWinSize.getWidth());
-mpToolbars-SetSizePixel(aToolbarsSize);
+maTabControl.SetTabPageSizePixel(aWinSize);
 
 // Calculate toolboxes size and positions
 Size aViewSize = 

[PATCH libreoffice-4-0] Template Manager: fixed TabStops for the toolbars

2013-03-19 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2845

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/2845/1

Template Manager: fixed TabStops for the toolbars

Change-Id: Iab2dd8ef71fd84e9d147d90e9cad46d69a8fe650
(cherry picked from commit eff94128b8434501a6f9c26a5a47cfd7d408f472)
---
M sfx2/inc/templatedlg.hxx
M sfx2/source/doc/templatedlg.cxx
M sfx2/source/doc/templatedlg.src
3 files changed, 4 insertions(+), 16 deletions(-)



diff --git a/sfx2/inc/templatedlg.hxx b/sfx2/inc/templatedlg.hxx
index 095292e..0d7eb78 100644
--- a/sfx2/inc/templatedlg.hxx
+++ b/sfx2/inc/templatedlg.hxx
@@ -128,7 +128,6 @@
 
 TabControl maTabControl;
 TabPage maTabPage;
-Control *mpToolbars;
 
 Edit *mpSearchEdit;
 ToolBox *mpViewBar;
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index f0a16b35..e9f85eb 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -113,11 +113,10 @@
 : ModelessDialog(parent, SfxResId(DLG_TEMPLATE_MANAGER)),
   maTabControl(this,SfxResId(TAB_CONTROL)),
   maTabPage(maTabControl, SfxResId(TAB_TEMPLATE_MANAGER)),
-  mpToolbars( new Control(maTabPage,SfxResId(TOOLBARS))),
   mpSearchEdit(new Edit(maTabPage,WB_HIDE | WB_BORDER)),
-  mpViewBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_VIEW))),
-  mpActionBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_ACTION))),
-  mpTemplateBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_TEMPLATES))),
+  mpViewBar( new ToolBox(maTabPage, SfxResId(TBX_ACTION_VIEW))),
+  mpActionBar( new ToolBox(maTabPage, SfxResId(TBX_ACTION_ACTION))),
+  mpTemplateBar( new ToolBox(maTabPage, SfxResId(TBX_ACTION_TEMPLATES))),
   mpSearchView(new TemplateSearchView(maTabPage)),
   maView(new TemplateLocalView(maTabPage,SfxResId(TEMPLATE_VIEW))),
   mpOnlineView(new TemplateRemoteView(maTabPage, WB_VSCROLL,false)),
@@ -239,7 +238,6 @@
 delete mpTemplateDefaultMenu;
 delete mpActionMenu;
 delete mpRepositoryMenu;
-delete mpToolbars;
 }
 
 void SfxTemplateManagerDlg::setSaveMode(bool bMode)
@@ -318,9 +316,6 @@
 
 long nToolbarsHeight = std::max(std::max(aViewSize.getHeight(), 
aActionSize.getHeight()), aTemplateSize.getHeight());
 
-Size aToolbarsSize (aWinSize.getWidth(), nToolbarsHeight);
-mpToolbars-SetSizePixel(aToolbarsSize);
-
 aActionSize.setWidth(3*aActionSize.getWidth());
 
aViewSize.setWidth(aWinSize.getWidth()-aActionSize.getWidth()-mpViewBar-GetPosPixel().X());
 aTemplateSize.setWidth(aWinSize.getWidth());
@@ -334,7 +329,7 @@
 
 // Set view position below toolbox
 Point aViewPos = maView-GetPosPixel();
-aViewPos.setY(aToolbarsSize.getHeight());
+aViewPos.setY(nToolbarsHeight);
 aViewPos.setX(0);
 Size aThumbSize(aWinSize.getWidth(), 
maTabControl.GetTabPageSizePixel().getWidth() - aViewPos.getY());
 maView-SetPosSizePixel(aViewPos, aThumbSize);
diff --git a/sfx2/source/doc/templatedlg.src b/sfx2/source/doc/templatedlg.src
index 6928ef1..60c2857 100644
--- a/sfx2/source/doc/templatedlg.src
+++ b/sfx2/source/doc/templatedlg.src
@@ -106,12 +106,6 @@
 Size = MAP_APPFONT( 290, 220 );
 Hide = TRUE;
 
-Control TOOLBARS
-{
-Size = MAP_APPFONT( 290 , 20 );
-TabStop = False;
-};
-
 Control TEMPLATE_VIEW
 {
 Pos = MAP_APPFONT( 0, 20 );

-- 
To view, visit https://gerrit.libreoffice.org/2845
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iab2dd8ef71fd84e9d147d90e9cad46d69a8fe650
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] Template Manager: fixed search edit field position

2013-03-19 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2844

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/44/2844/1

Template Manager: fixed search edit field position

Change-Id: Ic5ef64ba3ed4c98397b0c4911de491df4c79946a
(cherry picked from commit b6ba04639b9922f6717f79ac4be215e09691d7a9)
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 1d52d45..f0a16b35 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -114,7 +114,7 @@
   maTabControl(this,SfxResId(TAB_CONTROL)),
   maTabPage(maTabControl, SfxResId(TAB_TEMPLATE_MANAGER)),
   mpToolbars( new Control(maTabPage,SfxResId(TOOLBARS))),
-  mpSearchEdit(new Edit(this,WB_HIDE | WB_BORDER)),
+  mpSearchEdit(new Edit(maTabPage,WB_HIDE | WB_BORDER)),
   mpViewBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_VIEW))),
   mpActionBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_ACTION))),
   mpTemplateBar( new ToolBox(mpToolbars, SfxResId(TBX_ACTION_TEMPLATES))),

-- 
To view, visit https://gerrit.libreoffice.org/2844
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic5ef64ba3ed4c98397b0c4911de491df4c79946a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Remove rtl:: namespace, fix minor spelling error in comment

2013-03-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2726

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2726
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd04bfa61209277c57afe313d679b253aa241b7d
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Chris Sherlock chris.sherloc...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED libreoffice-3-6] rhbz#895690: Make GIO UCP less brittle, so saving docs works...

2013-03-13 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2697

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2697
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a9c0c094f5320456940e3af4c802711828ab5ac
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#38246 Comment control on Writer ruler feature

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1785


-- 
To view, visit https://gerrit.libreoffice.org/1785
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib065043d05fe56fbfe6d00e0bb654966f046129b
Gerrit-PatchSet: 6
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


[PUSHED] fdo#61458, index should be flat

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2603


-- 
To view, visit https://gerrit.libreoffice.org/2603
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I46db120fc5e7d616cd4c9eadee7fc4c502569e86
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ádám Király kiraly.adam.cs...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] use startsWith() instead of compareToAscii()

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2621


-- 
To view, visit https://gerrit.libreoffice.org/2621
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4dc63c7346f724eded9ac7b82cda25c2bb60beff
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Thomas Arnhold tho...@arnhold.org
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Chris Sherlock chris.sherloc...@gmail.com
Gerrit-Reviewer: Thomas Arnhold tho...@arnhold.org

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


[PUSHED] fdo#57950: Removed some chained appends from sw

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2643


-- 
To view, visit https://gerrit.libreoffice.org/2643
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5b78b4a8f2fc0fb0c4a67042ac4924e375c2d63a
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza marcos.souza@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Korrawit Pruegsanusak detective.conan.1...@gmail.com
Gerrit-Reviewer: Thomas Arnhold tho...@arnhold.org

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


[PUSHED] Ask for confirmation when deleting a template, fdo#61466

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2667


-- 
To view, visit https://gerrit.libreoffice.org/2667
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa499bfa5536d8df1d4e6d2aed3c350c588a0c46
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez venccsra...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#47011 autosave feature

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2663

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2663
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I473154b21bab53bf595a5a59e87dc16e472dcbf9
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Krisztian Pinter pin.termina...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] pptx import:supporting FilterData properties:PageRange and I...

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2665


-- 
To view, visit https://gerrit.libreoffice.org/2665
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9da8455bb10e93f1ae6c4894d4f15ff7fbcdfa07
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Sven Jacobi sven-jac...@gmx.de
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#47011 added autosave help description

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2668

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2668
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a0233ef834f55c4cd701f1d7292ada184abff2e
Gerrit-PatchSet: 3
Gerrit-Project: help
Gerrit-Branch: master
Gerrit-Owner: Krisztian Pinter pin.termina...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED libreoffice-4-0] rhbz#915743: Avoid potentially blocking autocompletion for n...

2013-03-11 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2669

Approvals:
  Petr Mladek: Verified; Looks good to me, approved
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2669
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie82b9a70285b30fe88cf4c40992710087e86815e
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Petr Mladek pmla...@suse.cz

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


[PUSHED] Remove RTL_CONST* macros from codemaker - pt2

2013-03-08 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2600


-- 
To view, visit https://gerrit.libreoffice.org/2600
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If55046a5a9ceb6c8c84f3fa190f26cc9e1dde352
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza marcos.souza@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Chris Sherlock chris.sherloc...@gmail.com
Gerrit-Reviewer: Marcos Souza marcos.souza@gmail.com

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


[PUSHED] It is safer to uncomment these checks in the postithelper

2013-03-06 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/2567

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/2567
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I84d96224dbe51ff25fcb6d96e0be9a8dff89b0a1
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Joren De Cuyper joren.libreoff...@telenet.be
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH libreoffice-4-0] CMIS: URL-decode username or it won't be shown properly in d...

2013-02-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2147

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/47/2147/1

CMIS: URL-decode username or it won't be shown properly in dialog

Change-Id: Icb8eaf5d841cba0d0d7efc4146b33995cdbf46f3
---
M ucb/source/ucp/cmis/cmis_url.cxx
1 file changed, 3 insertions(+), 3 deletions(-)



diff --git a/ucb/source/ucp/cmis/cmis_url.cxx b/ucb/source/ucp/cmis/cmis_url.cxx
index eb779d8..40d7a25 100644
--- a/ucb/source/ucp/cmis/cmis_url.cxx
+++ b/ucb/source/ucp/cmis/cmis_url.cxx
@@ -52,12 +52,12 @@
 m_sBindingUrl = aHostUrl.GetURLNoMark( );
 m_sRepositoryId = aHostUrl.GetMark( );
 
-m_sUser = aUrl.GetUser( );
-m_sPass = aUrl.GetPass( );
+m_sUser = aUrl.GetUser( INetURLObject::DECODE_WITH_CHARSET );
+m_sPass = aUrl.GetPass( INetURLObject::DECODE_WITH_CHARSET );
 
 // Store the path to the object
 m_sPath = aUrl.GetURLPath( INetURLObject::DECODE_WITH_CHARSET );
-m_sId = aUrl.GetMark( );
+m_sId = aUrl.GetMark( INetURLObject::DECODE_WITH_CHARSET );
 
 if ( !m_sId.isEmpty( ) )
 m_sPath = rtl::OUString( );

-- 
To view, visit https://gerrit.libreoffice.org/2147
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icb8eaf5d841cba0d0d7efc4146b33995cdbf46f3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH libreoffice-4-0] CMIS: made it work with Lotus Live

2013-02-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2148

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/48/2148/1

CMIS: made it work with Lotus Live

Making libcmis and LibreOffice work with Lotus Live service needed a few
hacks to either better implement CMIS or workaround some bad
implementations.

As a general improvement, the CheckOut InfoBar isn't shown if the
document can't be checked out.

Change-Id: I7bb4211db0506998cef40ac1fb6375e647234085
---
M libcmis/UnpackedTarball_cmis.mk
A libcmis/libcmis-0.3.0-lotus-live-fix.patch
M sfx2/source/view/sfxbasecontroller.cxx
M ucb/source/ucp/cmis/cmis_content.cxx
M ucb/source/ucp/cmis/cmis_content.hxx
M ucb/source/ucp/cmis/cmis_url.cxx
6 files changed, 259 insertions(+), 35 deletions(-)



diff --git a/libcmis/UnpackedTarball_cmis.mk b/libcmis/UnpackedTarball_cmis.mk
index a25c316..0a9f9f8 100644
--- a/libcmis/UnpackedTarball_cmis.mk
+++ b/libcmis/UnpackedTarball_cmis.mk
@@ -18,6 +18,7 @@
libcmis/libcmis-0.3.0-win.patch \
libcmis/libcmis-0.3.0.patch \
libcmis/libcmis-0.3.0-proxy.patch \
+   libcmis/libcmis-0.3.0-lotus-live-fix.patch \
 ))
 
 ifeq ($(OS)$(COM),WNTMSC)
diff --git a/libcmis/libcmis-0.3.0-lotus-live-fix.patch 
b/libcmis/libcmis-0.3.0-lotus-live-fix.patch
new file mode 100644
index 000..2aca934
--- /dev/null
+++ b/libcmis/libcmis-0.3.0-lotus-live-fix.patch
@@ -0,0 +1,122 @@
+diff --git src/libcmis/atom-folder.cxx src/libcmis/atom-folder.cxx
+index 68fb124..2756a5d 100644
+--- src/libcmis/atom-folder.cxx
 src/libcmis/atom-folder.cxx
+@@ -57,8 +57,11 @@ vector libcmis::ObjectPtr  AtomFolder::getChildren( ) 
throw ( libcmis::Excepti
+ {
+ AtomLink* childrenLink = getLink( down, 
application/atom+xml;type=feed );
+ 
++// Some servers aren't giving the GetChildren properly... if not defined, 
we need to try
++// as we may have the right to proceed.
+ if ( ( NULL == childrenLink ) || ( getAllowableActions( ).get() 
+-!getAllowableActions()-isAllowed( 
libcmis::ObjectAction::GetChildren ) ) )
++( !getAllowableActions()-isAllowed( 
libcmis::ObjectAction::GetChildren ) 
++  getAllowableActions()-isDefined( 
libcmis::ObjectAction::GetChildren ) ) ) )
+ throw libcmis::Exception( string( GetChildren not allowed on node  
) + getId() );
+ 
+ vector libcmis::ObjectPtr  children;
+@@ -182,7 +185,8 @@ libcmis::DocumentPtr AtomFolder::createDocument( const 
map string, libcmis::Pro
+ AtomLink* childrenLink = getLink( down, 
application/atom+xml;type=feed );
+ 
+ if ( ( NULL == childrenLink ) || ( getAllowableActions( ).get() 
+-!getAllowableActions()-isAllowed( 
libcmis::ObjectAction::CreateDocument ) ) )
++!getAllowableActions()-isAllowed( 
libcmis::ObjectAction::CreateDocument ) 
++getAllowableActions()-isDefined( 
libcmis::ObjectAction::CreateDocument ) ) )
+ throw libcmis::Exception( string( CreateDocument not allowed on 
folder  ) + getId() );
+ 
+ xmlBufferPtr buf = xmlBufferCreate( );
+@@ -210,9 +214,37 @@ libcmis::DocumentPtr AtomFolder::createDocument( const 
map string, libcmis::Pro
+ }
+ 
+ string respBuf = response-getStream( )-str( );
+-xmlDocPtr doc = xmlReadMemory( respBuf.c_str(), respBuf.size(), 
getInfosUrl().c_str(), NULL, 0 );
++xmlDocPtr doc = xmlReadMemory( respBuf.c_str(), respBuf.size(), 
getInfosUrl().c_str(), NULL, XML_PARSE_NOERROR );
+ if ( NULL == doc )
+-throw libcmis::Exception( Failed to parse object infos );
++{
++// We may not have the created document entry in the response body: 
this is
++// the behaviour of some servers, but the standard says we need to 
look for
++// the Location header.
++map string, string  headers = response-getHeaders( );
++map string, string ::iterator it = headers.find( Location );
++
++// Some servers like Lotus Live aren't sending Location header, but 
Content-Location
++if ( it == headers.end( ) )
++it = headers.find( Content-Location );
++
++if ( it != headers.end() )
++{
++try
++{
++response = getSession( )-httpGetRequest( it-second );
++respBuf = response-getStream( )-str( );
++doc = xmlReadMemory( respBuf.c_str(), respBuf.size(), 
getInfosUrl().c_str(), NULL, XML_PARSE_NOERROR );
++}
++catch ( const CurlException e )
++{
++throw e.getCmisException( );
++}
++}
++
++// if doc is still NULL after that, then throw an exception
++if ( NULL == doc )
++throw libcmis::Exception( Missing expected response from server 
);
++}
+ 
+ libcmis::ObjectPtr created = getSession( )-createObjectFromEntryDoc( doc 
);
+ xmlFreeDoc( doc );
+diff --git src/libcmis/atom-object.cxx 

[PATCH libreoffice-4-0] Fixed SfxDocTplService_Impl::setTitleForURL to keep all stre...

2013-02-12 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2107

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/07/2107/1

Fixed SfxDocTplService_Impl::setTitleForURL to keep all streams.

After commit 117fdb98, the setTitleForURL only wrote the meta.xml file
to the odf container, dropping all the other streams. Use storeToStorage
as in the old implementation to fix the problem.

Even though marginal, this problem is important for templates renaming.

Change-Id: Ia9b6abfc112f102e2bd2ae9d3080c4ed827200df
---
M sfx2/source/doc/doctemplates.cxx
1 file changed, 13 insertions(+), 2 deletions(-)



diff --git a/sfx2/source/doc/doctemplates.cxx b/sfx2/source/doc/doctemplates.cxx
index 1313fee..9720fab 100644
--- a/sfx2/source/doc/doctemplates.cxx
+++ b/sfx2/source/doc/doctemplates.cxx
@@ -32,6 +32,7 @@
 #include comphelper/componentcontext.hxx
 #include comphelper/processfactory.hxx
 #include comphelper/sequenceashashmap.hxx
+#include comphelper/storagehelper.hxx
 #include comphelper/string.hxx
 #include com/sun/star/beans/PropertyAttribute.hpp
 #include com/sun/star/beans/XPropertySet.hpp
@@ -646,8 +647,18 @@
 try
 {
 m_xDocProps-loadFromMedium(rURL, SequencePropertyValue());
-m_xDocProps-setTitle(aTitle );
-m_xDocProps-storeToMedium(rURL, SequencePropertyValue());
+m_xDocProps-setTitle(aTitle);
+
+uno::Reference embed::XStorage  xStorage = 
::comphelper::OStorageHelper::GetStorageFromURL(
+rURL, embed::ElementModes::READWRITE);
+
+uno::Sequencebeans::PropertyValue medium(2);
+medium[0].Name = ::rtl::OUString(DocumentBaseURL);
+medium[0].Value = rURL;
+medium[1].Name = ::rtl::OUString(URL);
+medium[1].Value = rURL;
+
+m_xDocProps-storeToStorage(xStorage, medium);
 return true;
 }
 catch ( Exception )

-- 
To view, visit https://gerrit.libreoffice.org/2107
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia9b6abfc112f102e2bd2ae9d3080c4ed827200df
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH libreoffice-4-0] Templates Manager: UI for renaming templates and folders, fd...

2013-02-12 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2108

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/08/2108/1

Templates Manager: UI for renaming templates and folders, fdo#60579

After thinking this was a feature regression, I finally discovered that
the old template manager couldn't rename templates. This commit is
bringing back some previously unused code dropped in an ealier commit.

Even though this is a UI change, no new string is added by this commit.

Conflicts:
sfx2/inc/sfx2/templateview.hxx
sfx2/source/control/templatelocalview.cxx
sfx2/source/control/templateview.cxx

Change-Id: I2e4a89c2e68f7e04b0fca3fc161920505ac715b4
---
M sfx2/inc/sfx2/doctempl.hxx
M sfx2/inc/sfx2/templatelocalview.hxx
M sfx2/inc/sfx2/templateview.hxx
M sfx2/inc/sfx2/templateviewitem.hxx
M sfx2/inc/sfx2/thumbnailview.hxx
M sfx2/inc/sfx2/thumbnailviewitem.hxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/control/templateview.cxx
M sfx2/source/control/templateviewitem.cxx
M sfx2/source/control/thumbnailview.cxx
M sfx2/source/control/thumbnailviewitem.cxx
M sfx2/source/doc/doctempl.cxx
12 files changed, 257 insertions(+), 5 deletions(-)



diff --git a/sfx2/inc/sfx2/doctempl.hxx b/sfx2/inc/sfx2/doctempl.hxx
index 507117c..b9ca56d 100644
--- a/sfx2/inc/sfx2/doctempl.hxx
+++ b/sfx2/inc/sfx2/doctempl.hxx
@@ -96,6 +96,21 @@
 sal_BoolInsertDir(const String rText, sal_uInt16 nRegion);
 sal_BoolSetName(const String rName, sal_uInt16 nRegion, 
sal_uInt16 nIdx);
 
+/** Change the name of an entry or a directory
+
+\param rName
+The new name to set
+\param nRegion
+The id of the region to rename or containing the template to rename
+\param nIdx
+The id of the template to rename or USHRT_MAX to rename the region.
+
+\return
+sal_True if the action could be performed, sal_False otherwise
+
+*/
+sal_BoolSetName(const rtl::OUString rName, sal_uInt16 
nRegion, sal_uInt16 nIdx);
+
 sal_BoolCopyTo(sal_uInt16 nRegion, sal_uInt16 nIdx, const 
String rName) const;
 sal_BoolCopyFrom(sal_uInt16 nRegion, sal_uInt16 nIdx, String 
rName);
 
diff --git a/sfx2/inc/sfx2/templatelocalview.hxx 
b/sfx2/inc/sfx2/templatelocalview.hxx
index e11c42b..dbf00ef 100644
--- a/sfx2/inc/sfx2/templatelocalview.hxx
+++ b/sfx2/inc/sfx2/templatelocalview.hxx
@@ -71,6 +71,8 @@
 
 bool isTemplateNameUnique (const sal_uInt16 nRegionItemId, const OUString 
rName) const;
 
+virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+
 private:
 
 virtual void OnItemDblClicked (ThumbnailViewItem *pRegionItem);
diff --git a/sfx2/inc/sfx2/templateview.hxx b/sfx2/inc/sfx2/templateview.hxx
index 6e81ea4..07b8487 100644
--- a/sfx2/inc/sfx2/templateview.hxx
+++ b/sfx2/inc/sfx2/templateview.hxx
@@ -39,6 +39,8 @@
 void setDblClickHdl (const Link rLink) { maDblClickHdl = rLink; }
 
 void setCloseHdl (const Link rLink) { maAllButton.SetClickHdl(rLink); }
+void setMasterView(TemplateAbstractView* pMasterView) { mpMasterView = 
pMasterView; }
+virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
 
 protected:
 
@@ -47,6 +49,7 @@
 virtual void OnItemDblClicked (ThumbnailViewItem *pItem);
 
 private:
+TemplateAbstractView* mpMasterView;
 
 ControlmaButtons;
 PushButton maAllButton;
diff --git a/sfx2/inc/sfx2/templateviewitem.hxx 
b/sfx2/inc/sfx2/templateviewitem.hxx
index b19a060..0a3cb3e 100644
--- a/sfx2/inc/sfx2/templateviewitem.hxx
+++ b/sfx2/inc/sfx2/templateviewitem.hxx
@@ -42,6 +42,10 @@
 
 virtual void Paint (drawinglayer::processor2d::BaseProcessor2D *pProcessor,
 const ThumbnailItemAttributes *pAttrs);
+
+sal_uInt16mnRegionId;
+sal_uInt16mnDocId;
+
 private:
 
 rtl::OUString maPath;
diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 9548bd1..a33e762 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -238,6 +238,8 @@
 
 virtual void Resize();
 
+virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+
 protected:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
diff --git a/sfx2/inc/sfx2/thumbnailviewitem.hxx 
b/sfx2/inc/sfx2/thumbnailviewitem.hxx
index 18b7eae..2acc8de 100644
--- a/sfx2/inc/sfx2/thumbnailviewitem.hxx
+++ b/sfx2/inc/sfx2/thumbnailviewitem.hxx
@@ -25,6 +25,7 @@
 #include drawinglayer/primitive2d/baseprimitive2d.hxx
 #include osl/mutex.hxx
 #include vcl/bitmapex.hxx
+#include vcl/vclmedit.hxx
 #include sfx2/dllapi.h
 
 #include com/sun/star/accessibility/XAccessible.hpp
@@ -88,12 +89,17 @@
 
 void setHighlight (bool state);
 
+void setEditTitle (bool edit, bool bChangeFocus = true);
+void updateTitleEditSize ();
+virtual void setTitle 

[PATCH libreoffice-4-0] Templates Manager: if the title can't be edited, don't updat...

2013-02-12 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2109

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/09/2109/1

Templates Manager: if the title can't be edited, don't update it in UI

Change-Id: Id16431fb3517c831a5e81ff72fafaa8569f99ef1
---
M sfx2/inc/sfx2/templatelocalview.hxx
M sfx2/inc/sfx2/templateview.hxx
M sfx2/inc/sfx2/thumbnailview.hxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/control/templateview.cxx
M sfx2/source/control/thumbnailview.cxx
M sfx2/source/control/thumbnailviewitem.cxx
7 files changed, 12 insertions(+), 10 deletions(-)



diff --git a/sfx2/inc/sfx2/templatelocalview.hxx 
b/sfx2/inc/sfx2/templatelocalview.hxx
index dbf00ef..78cf547 100644
--- a/sfx2/inc/sfx2/templatelocalview.hxx
+++ b/sfx2/inc/sfx2/templatelocalview.hxx
@@ -71,7 +71,7 @@
 
 bool isTemplateNameUnique (const sal_uInt16 nRegionItemId, const OUString 
rName) const;
 
-virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+virtual bool renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
 
 private:
 
diff --git a/sfx2/inc/sfx2/templateview.hxx b/sfx2/inc/sfx2/templateview.hxx
index 07b8487..3e2a7c8 100644
--- a/sfx2/inc/sfx2/templateview.hxx
+++ b/sfx2/inc/sfx2/templateview.hxx
@@ -40,7 +40,7 @@
 
 void setCloseHdl (const Link rLink) { maAllButton.SetClickHdl(rLink); }
 void setMasterView(TemplateAbstractView* pMasterView) { mpMasterView = 
pMasterView; }
-virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+virtual bool renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
 
 protected:
 
diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index a33e762..8eb9426 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -238,7 +238,7 @@
 
 virtual void Resize();
 
-virtual void renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
+virtual bool renameItem(ThumbnailViewItem* pItem, rtl::OUString sNewTitle);
 
 protected:
 
diff --git a/sfx2/source/control/templatelocalview.cxx 
b/sfx2/source/control/templatelocalview.cxx
index 2bbf0d0..ca8856e 100644
--- a/sfx2/source/control/templatelocalview.cxx
+++ b/sfx2/source/control/templatelocalview.cxx
@@ -605,7 +605,7 @@
 showOverlay(true);
 }
 
-void TemplateLocalView::renameItem(ThumbnailViewItem* pItem, rtl::OUString 
sNewTitle)
+bool TemplateLocalView::renameItem(ThumbnailViewItem* pItem, rtl::OUString 
sNewTitle)
 {
 sal_uInt16 nRegionId = 0;
 sal_uInt16 nDocId = USHRT_MAX;
@@ -620,7 +620,7 @@
 {
 nRegionId = pContainerItem-mnId - 1;
 }
-mpDocTemplates-SetName( sNewTitle, nRegionId, nDocId );
+return mpDocTemplates-SetName( sNewTitle, nRegionId, nDocId );
 }
 
 static void lcl_updateThumbnails (TemplateContainerItem *pItem)
diff --git a/sfx2/source/control/templateview.cxx 
b/sfx2/source/control/templateview.cxx
index 90e86ec..615465d 100644
--- a/sfx2/source/control/templateview.cxx
+++ b/sfx2/source/control/templateview.cxx
@@ -105,10 +105,11 @@
 maDblClickHdl.Call(pItem);
 }
 
-void TemplateView::renameItem(ThumbnailViewItem* pItem, rtl::OUString 
sNewTitle)
+bool TemplateView::renameItem(ThumbnailViewItem* pItem, rtl::OUString 
sNewTitle)
 {
 if (mpMasterView)
-mpMasterView-renameItem(pItem, sNewTitle);
+return mpMasterView-renameItem(pItem, sNewTitle);
+return false;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 9a29381..c996f0f 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -980,9 +980,10 @@
 Invalidate();
 }
 
-void ThumbnailView::renameItem(ThumbnailViewItem*, rtl::OUString)
+bool ThumbnailView::renameItem(ThumbnailViewItem*, rtl::OUString)
 {
 // Do nothing by default
+return false;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/source/control/thumbnailviewitem.cxx 
b/sfx2/source/control/thumbnailviewitem.cxx
index c228bab..2a83bc1 100644
--- a/sfx2/source/control/thumbnailviewitem.cxx
+++ b/sfx2/source/control/thumbnailviewitem.cxx
@@ -187,8 +187,8 @@
 
 void ThumbnailViewItem::setTitle (const rtl::OUString rTitle)
 {
-mrParent.renameItem(this, rTitle);
-maTitle = rTitle;
+if (mrParent.renameItem(this, rTitle))
+maTitle = rTitle;
 }
 
 uno::Reference accessibility::XAccessible  ThumbnailViewItem::GetAccessible( 
bool bIsTransientChildrenDisabled )

-- 
To view, visit https://gerrit.libreoffice.org/2109
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id16431fb3517c831a5e81ff72fafaa8569f99ef1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___

[PATCH libreoffice-4-0] CMIS: proxy was setin to :-1 is no proxy is defined in the...

2013-02-12 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/2120

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/20/2120/1

CMIS: proxy was setin to :-1 is no proxy is defined in the UI

Don't output the port number if that one is not valid.
(cherry picked from commit 12ea70362f9b1234b4b1613872c8f07735dd386b)

Change-Id: Ia533d189117227a25c2df2e639b7fbd854e1bc45
---
M ucb/source/ucp/cmis/cmis_content.cxx
M ucb/source/ucp/cmis/cmis_repo_content.cxx
2 files changed, 6 insertions(+), 2 deletions(-)



diff --git a/ucb/source/ucp/cmis/cmis_content.cxx 
b/ucb/source/ucp/cmis/cmis_content.cxx
index a8b1c86..5f94d7a 100644
--- a/ucb/source/ucp/cmis/cmis_content.cxx
+++ b/ucb/source/ucp/cmis/cmis_content.cxx
@@ -262,7 +262,9 @@
 INetURLObject aBindingUrl( m_aURL.getBindingUrl( ) );
 const ucbhelper::InternetProxyServer rProxy = aProxyDecider.getProxy(
 INetURLObject::GetScheme( aBindingUrl.GetProtocol( ) ), 
aBindingUrl.GetHost(), aBindingUrl.GetPort() );
-rtl::OUString sProxy = rProxy.aName + : + rtl::OUString::valueOf( 
rProxy.nPort );
+rtl::OUString sProxy = rProxy.aName;
+if ( rProxy.nPort  0 )
+sProxy += : + rtl::OUString::valueOf( rProxy.nPort );
 libcmis::SessionFactory::setProxySettings( OUSTR_TO_STDSTR( sProxy ), 
string(), string(), string() );
 
 // Look for a cached session, key is binding url + repo id
diff --git a/ucb/source/ucp/cmis/cmis_repo_content.cxx 
b/ucb/source/ucp/cmis/cmis_repo_content.cxx
index afd503c..b76514c 100644
--- a/ucb/source/ucp/cmis/cmis_repo_content.cxx
+++ b/ucb/source/ucp/cmis/cmis_repo_content.cxx
@@ -144,7 +144,9 @@
 INetURLObject aBindingUrl( m_aURL.getBindingUrl( ) );
 const ucbhelper::InternetProxyServer rProxy = aProxyDecider.getProxy(
 INetURLObject::GetScheme( aBindingUrl.GetProtocol( ) ), 
aBindingUrl.GetHost(), aBindingUrl.GetPort() );
-rtl::OUString sProxy = rProxy.aName + : + rtl::OUString::valueOf( 
rProxy.nPort );
+rtl::OUString sProxy = rProxy.aName;
+if ( rProxy.nPort  0 )
+sProxy += : + rtl::OUString::valueOf( rProxy.nPort );
 libcmis::SessionFactory::setProxySettings( OUSTR_TO_STDSTR( sProxy ), 
string(), string(), string() );
 
 if ( m_aRepositories.empty() )

-- 
To view, visit https://gerrit.libreoffice.org/2120
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia533d189117227a25c2df2e639b7fbd854e1bc45
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Change in core[libreoffice-4-0]: Template Manager: show some icons if there is no thumbnail

2013-01-31 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1948

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/48/1948/1

Template Manager: show some icons if there is no thumbnail

Change-Id: I2686d6f1d29862878b5ee0f6ca0d2e746766ef21
---
M sfx2/inc/sfx2/sfx.hrc
M sfx2/inc/templatedlg.hxx
M sfx2/source/appl/sfx.src
M sfx2/source/control/templateview.cxx
M sfx2/source/doc/templatedlg.cxx
5 files changed, 58 insertions(+), 0 deletions(-)



diff --git a/sfx2/inc/sfx2/sfx.hrc b/sfx2/inc/sfx2/sfx.hrc
index 7173366..92bcf1f 100644
--- a/sfx2/inc/sfx2/sfx.hrc
+++ b/sfx2/inc/sfx2/sfx.hrc
@@ -177,6 +177,11 @@
 #define BT_CHECKOUT (RID_SFX_START+126)
 #define STR_ACCTITLE_PRODUCTIVITYTOOLS  (RID_SFX_START+157)
 
+#define SFX_THUMBNAIL_TEXT  (RID_SFX_START+158)
+#define SFX_THUMBNAIL_SHEET (RID_SFX_START+159)
+#define SFX_THUMBNAIL_PRESENTATION  (RID_SFX_START+160)
+#define SFX_THUMBNAIL_DRAWING   (RID_SFX_START+161)
+
 //=
 
 // group ids
diff --git a/sfx2/inc/templatedlg.hxx b/sfx2/inc/templatedlg.hxx
index 44a6653..47c1989 100644
--- a/sfx2/inc/templatedlg.hxx
+++ b/sfx2/inc/templatedlg.hxx
@@ -49,6 +49,8 @@
 
 DECL_LINK(ActivatePageHdl, void*);
 
+static BitmapEx getDefaultThumbnail( const rtl::OUString rPath );
+
 private:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
diff --git a/sfx2/source/appl/sfx.src b/sfx2/source/appl/sfx.src
index 1103841..bffec2b 100644
--- a/sfx2/source/appl/sfx.src
+++ b/sfx2/source/appl/sfx.src
@@ -37,3 +37,23 @@
 {
 Text [ en-US ] = Password ;
 };
+
+Bitmap ( SFX_THUMBNAIL_TEXT )
+{
+File = odt_48_8.png;
+};
+
+Bitmap ( SFX_THUMBNAIL_SHEET )
+{
+File = ods_48_8.png;
+};
+
+Bitmap ( SFX_THUMBNAIL_PRESENTATION )
+{
+File = odp_48_8.png;
+};
+
+Bitmap ( SFX_THUMBNAIL_DRAWING )
+{
+File = odg_48_8.png;
+};
diff --git a/sfx2/source/control/templateview.cxx 
b/sfx2/source/control/templateview.cxx
index 2efbbda..9758065 100644
--- a/sfx2/source/control/templateview.cxx
+++ b/sfx2/source/control/templateview.cxx
@@ -7,6 +7,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+#include templatedlg.hxx
 #include sfx2/templateview.hxx
 
 #include basegfx/matrix/b2dhommatrixtools.hxx
@@ -63,6 +64,11 @@
 pItem-maTitle = pCur-aName;
 pItem-setPath(pCur-aPath);
 pItem-maPreview1 = pCur-aThumbnail;
+if ( pCur-aThumbnail.IsEmpty() )
+{
+// Use the default thumbnail if we have nothing else
+pItem-maPreview1 = SfxTemplateManagerDlg::getDefaultThumbnail( 
pItem-getPath() );
+}
 pItem-setSelectClickHdl(LINK(this,ThumbnailView,OnItemSelected));
 
 mItemList.push_back(pItem);
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index bf51e37..c14a824 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -1533,6 +1533,31 @@
 }
 }
 
+BitmapEx SfxTemplateManagerDlg::getDefaultThumbnail( const OUString rPath )
+{
+INetURLObject aUrl(rPath);
+OUString aExt = aUrl.getExtension();
+
+BitmapEx aImg;
+if ( aExt == ott || aExt == stw || aExt == oth || aExt == dot || 
aExt == dotx )
+{
+aImg = BitmapEx ( SfxResId( SFX_THUMBNAIL_TEXT ) );
+}
+else if ( aExt == ots || aExt == stc || aExt == xlt || aExt == 
xltm || aExt == xltx )
+{
+aImg = BitmapEx ( SfxResId( SFX_THUMBNAIL_SHEET ) );
+}
+else if ( aExt == otp || aExt == sti || aExt == pot || aExt == 
potm || aExt == potx )
+{
+aImg = BitmapEx ( SfxResId( SFX_THUMBNAIL_PRESENTATION ) );
+}
+else if ( aExt == otg || aExt == std )
+{
+aImg = BitmapEx ( SfxResId( SFX_THUMBNAIL_DRAWING ) );
+}
+return aImg;
+}
+
 static bool lcl_getServiceName ( const OUString rFileURL, OUString rName )
 {
 bool bRet = false;

-- 
To view, visit https://gerrit.libreoffice.org/1948
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2686d6f1d29862878b5ee0f6ca0d2e746766ef21
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Change in core[libreoffice-4-0]: Template Manager: focus in search edit when clicking search ...

2013-01-29 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1916

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/16/1916/1

Template Manager: focus in search edit when clicking search action

Change-Id: I91fa8463442d842ef9929a87fb22e5e1a0a869b5
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 2 insertions(+), 0 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 908fca9..bf51e37 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -963,6 +963,8 @@
 
 mpSearchEdit-Show(!bVisible);
 mpSearchEdit-SetText(OUString());
+if (!bVisible)
+mpSearchEdit-GrabFocus();
 
 // display all templates if we hide the search bar
 if (bVisible  mpCurView-isOverlayVisible())

-- 
To view, visit https://gerrit.libreoffice.org/1916
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I91fa8463442d842ef9929a87fb22e5e1a0a869b5
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Change in core[libreoffice-4-0-0]: fdo#59798: MSI installer spurious space in version number

2013-01-28 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1843

Approvals:
  Andras Timar: Verified; Looks good to me, but someone else must approve
  Eike Rathke: Looks good to me, but someone else must approve
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/1843
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0a94f7265d310f0f5c48c747ffca49a7aeb05a01
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-0
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Andras Timar ati...@suse.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Eike Rathke er...@redhat.com
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Change in core[libreoffice-4-0-0]: fdo#59798: fix SDK MSI version LOdev 4.1 SDK.0.0.alpha0

2013-01-28 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1844

Approvals:
  Andras Timar: Verified; Looks good to me, but someone else must approve
  Eike Rathke: Looks good to me, but someone else must approve
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/1844
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie130dd5be08247ae9202f98cd58a6922fed27f32
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-0
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Andras Timar ati...@suse.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Eike Rathke er...@redhat.com
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Change in core[libreoffice-4-0]: Template Manager: don't show any tab and show all folders wh...

2013-01-28 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1902

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/02/1902/1

Template Manager: don't show any tab and show all folders when saving template

Change-Id: Ib8f8dd66374a56a5c49155cde6420c865aa308f7
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 2 insertions(+), 0 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 833e556..908fca9 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -226,6 +226,8 @@
 void SfxTemplateManagerDlg::setSaveMode(bool bMode)
 {
 mbIsSaveMode = bMode;
+maTabControl.Clear();
+mpCurView-filterTemplatesByApp(FILTER_APP_NONE);
 
 if (bMode)
 {

-- 
To view, visit https://gerrit.libreoffice.org/1902
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib8f8dd66374a56a5c49155cde6420c865aa308f7
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Change in core[libreoffice-4-0-0]: Template Manager: remote repositories only for experimental ...

2013-01-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1847

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/47/1847/1

Template Manager: remote repositories only for experimental mode

Change-Id: I79501bdcd25f362065d9b95efd1a43f6be84577e
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 8 insertions(+), 0 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index f1e69f5..833e556 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -30,6 +30,7 @@
 #include sfx2/thumbnailviewitem.hxx
 #include sot/storage.hxx
 #include svtools/imagemgr.hxx
+#include svtools/miscopt.hxx
 #include svtools/PlaceEditDialog.hxx
 #include tools/urlobj.hxx
 #include unotools/moduleoptions.hxx
@@ -177,6 +178,13 @@
 
 
maTabControl.SetActivatePageHdl(LINK(this,SfxTemplateManagerDlg,ActivatePageHdl));
 
+SvtMiscOptions aMiscOptions;
+if ( !aMiscOptions.IsExperimentalMode() )
+{
+sal_uInt16 nPos = mpViewBar-GetItemId(TBI_TEMPLATE_REPOSITORY);
+mpViewBar-RemoveItem(nPos);
+}
+
 mpViewBar-Show();
 mpActionBar-Show();
 

-- 
To view, visit https://gerrit.libreoffice.org/1847
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I79501bdcd25f362065d9b95efd1a43f6be84577e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0-0]: fdo#59779: Set the first page attributes on pooled styles cr...

2013-01-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1864

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/64/1864/1

fdo#59779: Set the first page attributes on pooled styles creation

Change-Id: If599bc9d9b9a22c5500a0e39bcec76a6162bec3f
---
M sw/source/core/doc/poolfmt.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/sw/source/core/doc/poolfmt.cxx b/sw/source/core/doc/poolfmt.cxx
index 2820663..f33eee8 100644
--- a/sw/source/core/doc/poolfmt.cxx
+++ b/sw/source/core/doc/poolfmt.cxx
@@ -1597,6 +1597,7 @@
 if( bSetLeft )
 pNewPgDsc-GetLeft().SetFmtAttr( aSet );
 pNewPgDsc-GetMaster().SetFmtAttr( aSet );
+pNewPgDsc-GetFirst().SetFmtAttr( aSet );
 }
 }
 return pNewPgDsc;

-- 
To view, visit https://gerrit.libreoffice.org/1864
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If599bc9d9b9a22c5500a0e39bcec76a6162bec3f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Change in core[libreoffice-4-0]: fdo#58204: PPT export: fix text rotation in shapes:

2013-01-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1862

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/1862
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id3dcfb6221d031cdabb7f543b68752b467d9e1a6
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Change in core[libreoffice-4-0-0]: Resolves: fdo#59586 Missing comma in particular PDF file

2013-01-25 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1845

Approvals:
  Caolán McNamara: Verified; Looks good to me, but someone else must approve
  Joren De Cuyper: Looks good to me, but someone else must approve
  Bosdonnat Cedric: Verified; Looks good to me, approved


-- 
To view, visit https://gerrit.libreoffice.org/1845
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5d4407729c46111fc080e24b6a86d6740faa0f81
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-0
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Joren De Cuyper joren.libreoff...@telenet.be
Gerrit-Reviewer: Michael Stahl mst...@redhat.com

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


Resolves fdo#46718 : UI: Delete multiple styles at once

2013-01-25 Thread Bosdonnat Cedric (via Code Review)
Bosdonnat Cedric has abandoned this change.

Change subject: Resolves fdo#46718 : UI: Delete multiple styles at once
..


Patch Set 1: Abandoned

Someone fixed in right before (and better patch quality)

-- 
To view, visit https://gerrit.libreoffice.org/1730
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I88d0113d4d311f98de8e20a5f8c8ebc0e2025b6f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rushi Shukla rushbo...@gmail.com
Gerrit-Reviewer: Joren De Cuyper joren.libreoff...@telenet.be
Gerrit-Reviewer: Noel Power noel.po...@suse.com

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


[PATCH] Change in core[libreoffice-4-0]: curl: patched to read IE proxy settings on Windows

2013-01-21 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1792

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/92/1792/1

curl: patched to read IE proxy settings on Windows

Change-Id: I46605f6c8dfacab0feb5a446db458eed8e5756ee
---
M curl/UnpackedTarball_curl.mk
A curl/curl-7.26.0_win-proxy.patch
2 files changed, 125 insertions(+), 0 deletions(-)



diff --git a/curl/UnpackedTarball_curl.mk b/curl/UnpackedTarball_curl.mk
index 4ff19f7..e6a8cac 100644
--- a/curl/UnpackedTarball_curl.mk
+++ b/curl/UnpackedTarball_curl.mk
@@ -23,6 +23,7 @@
curl/curl-aix.patch \
curl/curl-7.26.0_win.patch \
curl/curl-7.26.0_mingw.patch \
+   curl/curl-7.26.0_win-proxy.patch \
 ))
 ifeq ($(OS),ANDROID)
 $(eval $(call gb_UnpackedTarball_add_patches,curl,\
diff --git a/curl/curl-7.26.0_win-proxy.patch b/curl/curl-7.26.0_win-proxy.patch
new file mode 100644
index 000..766ef16
--- /dev/null
+++ b/curl/curl-7.26.0_win-proxy.patch
@@ -0,0 +1,124 @@
+--- curl-7.26.0/lib/Makefile.vc9
 misc/build/curl-7.26.0/lib/Makefile.vc9
+@@ -116,7 +116,7 @@ LFLAGS = /nologo /machine:$(MACHINE)
+ SSLLIBS= libeay32.lib ssleay32.lib
+ ZLIBLIBSDLL= zdll.lib
+ ZLIBLIBS   = zlib.lib
+-WINLIBS= ws2_32.lib wldap32.lib
++WINLIBS= ws2_32.lib wldap32.lib winhttp.lib
+ CFLAGS = $(CFLAGS) $(EXCFLAGS)
+ 
+ CFGSET = FALSE
+--- curl-7.26.0/lib/url.c
 misc/build/curl-7.26.0/lib/url.c
+@@ -80,6 +80,10 @@ void idn_free (void *ptr);
+ int curl_win32_idn_to_ascii(const char *in, char **out);
+ #endif  /* USE_LIBIDN */
+ 
++#ifdef WIN32
++#include WinHttp.h
++#endif
++
+ #include urldata.h
+ #include netrc.h
+ 
+@@ -4111,6 +4115,21 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
+   return FALSE;
+ }
+ 
++#ifdef WIN32
++static char* wstrToCstr( LPWSTR wStr )
++{
++  int bufSize;
++  char* out = NULL;
++  if ( wStr != NULL )
++  {
++  bufSize = WideCharToMultiByte( CP_ACP,  0, wStr, -1, NULL, 0, 
NULL, NULL );
++  out = ( char* )malloc( bufSize * sizeof(char));
++  WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, 
NULL );
++  }
++  return out;
++}
++#endif
++
+ /
+ * Detect what (if any) proxy to use. Remember that this selects a host
+ * name and is not limited to HTTP proxies only.
+@@ -4119,6 +4138,7 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
+ static char *detect_proxy(struct connectdata *conn)
+ {
+   char *proxy = NULL;
++  char *no_proxy=NULL;
+ 
+ #ifndef CURL_DISABLE_HTTP
+   /* If proxy was not specified, we check for default proxy environment
+@@ -4138,7 +4158,57 @@ static char *detect_proxy(struct connectdata *conn)
+* For compatibility, the all-uppercase versions of these variables are
+* checked if the lowercase versions don't exist.
+*/
+-  char *no_proxy=NULL;
++#ifdef WIN32
++  WINHTTP_CURRENT_USER_IE_PROXY_CONFIG *ieProxyConfig;
++  ieProxyConfig = 
(WINHTTP_CURRENT_USER_IE_PROXY_CONFIG*)malloc(sizeof(WINHTTP_CURRENT_USER_IE_PROXY_CONFIG));
++  if(WinHttpGetIEProxyConfigForCurrentUser(ieProxyConfig)) {
++if(!ieProxyConfig-fAutoDetect) {
++  char *ieProxy;
++  char *ieNoProxy;
++char* pos;
++
++  ieProxy = wstrToCstr(ieProxyConfig-lpszProxy);
++  ieNoProxy = wstrToCstr(ieProxyConfig-lpszProxyBypass);
++
++  /* Convert the ieNoProxy into a proper no_proxy value */
++  no_proxy = strdup(ieNoProxy);
++  pos = strpbrk(no_proxy, ; );
++  while (NULL != pos) {
++no_proxy[pos-no_proxy] = ',';
++pos = strpbrk(no_proxy, ; );
++  }
++
++  if(!check_noproxy(conn-host.name, no_proxy)) {
++/* Look for the http proxy setting */
++char* tok;
++
++tok = strtok(ieProxy, ;);
++if(strchr(tok, '=') == NULL) {
++  proxy = strdup(ieProxy);
++}
++else {
++  do {
++if(strncmp(tok, http=, 5) == 0) {
++  /* We found HTTP proxy value, then use it */
++  proxy = strdup( tok + 5 );
++}
++  }
++  while(NULL != strtok(NULL, ;));
++}
++  }
++
++  free(ieProxy);
++  free(ieNoProxy);
++}
++else {
++  /* TODO Handle the Proxy config Auto Detection case */
++}
++  
++GlobalFree( ieProxyConfig-lpszAutoConfigUrl );
++  GlobalFree( ieProxyConfig-lpszProxy );
++  GlobalFree( ieProxyConfig-lpszProxyBypass );
++  }
++#else /* !WIN32 */
+   char proxy_env[128];
+ 
+   no_proxy=curl_getenv(no_proxy);
+@@ -4189,9 +4259,9 @@ static char *detect_proxy(struct connectdata *conn)
+ }
+   } /* if(!check_noproxy(conn-host.name, no_proxy)) - it wasn't specified
+non-proxy */
++#endif /* WIN32 */
+   if(no_proxy)
+ free(no_proxy);
+-
+ #else /* !CURL_DISABLE_HTTP */
+ 
+   

[PATCH] Change in core[libreoffice-4-0]: Tabs to spaces

2013-01-21 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1799

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/99/1799/1

Tabs to spaces

Change-Id: Ib0d64c9d93f7ae73a160ad242a6731b781edaf0f
---
M curl/curl-7.26.0_win-proxy.patch
1 file changed, 14 insertions(+), 14 deletions(-)



diff --git a/curl/curl-7.26.0_win-proxy.patch b/curl/curl-7.26.0_win-proxy.patch
index 766ef16..cf6a277 100644
--- a/curl/curl-7.26.0_win-proxy.patch
+++ b/curl/curl-7.26.0_win-proxy.patch
@@ -29,15 +29,15 @@
 +#ifdef WIN32
 +static char* wstrToCstr( LPWSTR wStr )
 +{
-+  int bufSize;
-+  char* out = NULL;
-+  if ( wStr != NULL )
-+  {
-+  bufSize = WideCharToMultiByte( CP_ACP,  0, wStr, -1, NULL, 0, 
NULL, NULL );
-+  out = ( char* )malloc( bufSize * sizeof(char));
-+  WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, 
NULL );
-+  }
-+  return out;
++int bufSize;
++char* out = NULL;
++if ( wStr != NULL )
++{
++bufSize = WideCharToMultiByte( CP_ACP,  0, wStr, -1, NULL, 0, NULL, 
NULL );
++out = ( char* )malloc( bufSize * sizeof(char));
++WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, NULL );
++}
++return out;
 +}
 +#endif
 +
@@ -64,12 +64,12 @@
 +if(!ieProxyConfig-fAutoDetect) {
 +  char *ieProxy;
 +  char *ieNoProxy;
-+char* pos;
++  char* pos;
 +
 +  ieProxy = wstrToCstr(ieProxyConfig-lpszProxy);
 +  ieNoProxy = wstrToCstr(ieProxyConfig-lpszProxyBypass);
 +
-+  /* Convert the ieNoProxy into a proper no_proxy value */
++  /* Convert the ieNoProxy into a proper no_proxy value */
 +  no_proxy = strdup(ieNoProxy);
 +  pos = strpbrk(no_proxy, ; );
 +  while (NULL != pos) {
@@ -102,10 +102,10 @@
 +else {
 +  /* TODO Handle the Proxy config Auto Detection case */
 +}
-+  
++
 +GlobalFree( ieProxyConfig-lpszAutoConfigUrl );
-+  GlobalFree( ieProxyConfig-lpszProxy );
-+  GlobalFree( ieProxyConfig-lpszProxyBypass );
++GlobalFree( ieProxyConfig-lpszProxy );
++GlobalFree( ieProxyConfig-lpszProxyBypass );
 +  }
 +#else /* !WIN32 */
char proxy_env[128];

-- 
To view, visit https://gerrit.libreoffice.org/1799
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib0d64c9d93f7ae73a160ad242a6731b781edaf0f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[PATCH] Change in core[libreoffice-4-0]: More massaging to avoid odd layout-related warnings/errors f...

2013-01-21 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1800

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/00/1800/1

More massaging to avoid odd layout-related warnings/errors from GCC

(...whatever it is that causes them to be raised at least in my build).

Change-Id: I52889de876e448ae8e90ec328e65cda2025968e2
---
M curl/curl-7.26.0_win-proxy.patch
1 file changed, 8 insertions(+), 6 deletions(-)



diff --git a/curl/curl-7.26.0_win-proxy.patch b/curl/curl-7.26.0_win-proxy.patch
index cf6a277..78cee02 100644
--- a/curl/curl-7.26.0_win-proxy.patch
+++ b/curl/curl-7.26.0_win-proxy.patch
@@ -22,7 +22,7 @@
  #include urldata.h
  #include netrc.h
  
-@@ -4111,6 +4115,21 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
+@@ -4111,6 +4115,22 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
return FALSE;
  }
  
@@ -31,9 +31,10 @@
 +{
 +int bufSize;
 +char* out = NULL;
-+if ( wStr != NULL )
++if( wStr != NULL )
 +{
-+bufSize = WideCharToMultiByte( CP_ACP,  0, wStr, -1, NULL, 0, NULL, 
NULL );
++bufSize = WideCharToMultiByte(
++CP_ACP,  0, wStr, -1, NULL, 0, NULL, NULL );
 +out = ( char* )malloc( bufSize * sizeof(char));
 +WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, NULL );
 +}
@@ -52,14 +53,15 @@
  
  #ifndef CURL_DISABLE_HTTP
/* If proxy was not specified, we check for default proxy environment
-@@ -4138,7 +4158,57 @@ static char *detect_proxy(struct connectdata *conn)
+@@ -4138,7 +4158,58 @@ static char *detect_proxy(struct connectdata *conn)
 * For compatibility, the all-uppercase versions of these variables are
 * checked if the lowercase versions don't exist.
 */
 -  char *no_proxy=NULL;
 +#ifdef WIN32
 +  WINHTTP_CURRENT_USER_IE_PROXY_CONFIG *ieProxyConfig;
-+  ieProxyConfig = 
(WINHTTP_CURRENT_USER_IE_PROXY_CONFIG*)malloc(sizeof(WINHTTP_CURRENT_USER_IE_PROXY_CONFIG));
++  ieProxyConfig = (WINHTTP_CURRENT_USER_IE_PROXY_CONFIG*)
++malloc(sizeof(WINHTTP_CURRENT_USER_IE_PROXY_CONFIG));
 +  if(WinHttpGetIEProxyConfigForCurrentUser(ieProxyConfig)) {
 +if(!ieProxyConfig-fAutoDetect) {
 +  char *ieProxy;
@@ -72,7 +74,7 @@
 +  /* Convert the ieNoProxy into a proper no_proxy value */
 +  no_proxy = strdup(ieNoProxy);
 +  pos = strpbrk(no_proxy, ; );
-+  while (NULL != pos) {
++  while(NULL != pos) {
 +no_proxy[pos-no_proxy] = ',';
 +pos = strpbrk(no_proxy, ; );
 +  }

-- 
To view, visit https://gerrit.libreoffice.org/1800
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I52889de876e448ae8e90ec328e65cda2025968e2
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[PATCH] Change in core[libreoffice-4-0]: ...and some more curl code layout massaging

2013-01-21 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1801

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/01/1801/1

...and some more curl code layout massaging

Change-Id: Ifbd7dcc3fa8d18192d2ac2fc5faca1b0637c93c2
---
M curl/curl-7.26.0_win-proxy.patch
1 file changed, 10 insertions(+), 11 deletions(-)



diff --git a/curl/curl-7.26.0_win-proxy.patch b/curl/curl-7.26.0_win-proxy.patch
index 78cee02..61badc7 100644
--- a/curl/curl-7.26.0_win-proxy.patch
+++ b/curl/curl-7.26.0_win-proxy.patch
@@ -22,23 +22,22 @@
  #include urldata.h
  #include netrc.h
  
-@@ -4111,6 +4115,22 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
+@@ -4111,6 +4115,21 @@ static bool check_noproxy(const char* name, const char* 
no_proxy)
return FALSE;
  }
  
 +#ifdef WIN32
 +static char* wstrToCstr( LPWSTR wStr )
 +{
-+int bufSize;
-+char* out = NULL;
-+if( wStr != NULL )
-+{
-+bufSize = WideCharToMultiByte(
-+CP_ACP,  0, wStr, -1, NULL, 0, NULL, NULL );
-+out = ( char* )malloc( bufSize * sizeof(char));
-+WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, NULL );
-+}
-+return out;
++  int bufSize;
++  char* out = NULL;
++  if(wStr != NULL) {
++bufSize = WideCharToMultiByte(
++  CP_ACP,  0, wStr, -1, NULL, 0, NULL, NULL );
++out = ( char* )malloc( bufSize * sizeof(char));
++WideCharToMultiByte( CP_ACP, 0, wStr, -1, out, bufSize, NULL, NULL );
++  }
++  return out;
 +}
 +#endif
 +

-- 
To view, visit https://gerrit.libreoffice.org/1801
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifbd7dcc3fa8d18192d2ac2fc5faca1b0637c93c2
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[PATCH] Change in core[libreoffice-4-0]: curl: fixed crashers in curl patch for system proxy

2013-01-21 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1802

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/02/1802/1

curl: fixed crashers in curl patch for system proxy

Change-Id: Ieb8e782eab9c9bde1d2099d8cc32eceef9f2
---
M curl/curl-7.26.0_win-proxy.patch
1 file changed, 21 insertions(+), 17 deletions(-)



diff --git a/curl/curl-7.26.0_win-proxy.patch b/curl/curl-7.26.0_win-proxy.patch
index 61badc7..810438f 100644
--- a/curl/curl-7.26.0_win-proxy.patch
+++ b/curl/curl-7.26.0_win-proxy.patch
@@ -52,7 +52,7 @@
  
  #ifndef CURL_DISABLE_HTTP
/* If proxy was not specified, we check for default proxy environment
-@@ -4138,7 +4158,58 @@ static char *detect_proxy(struct connectdata *conn)
+@@ -4138,7 +4158,62 @@ static char *detect_proxy(struct connectdata *conn)
 * For compatibility, the all-uppercase versions of these variables are
 * checked if the lowercase versions don't exist.
 */
@@ -71,29 +71,33 @@
 +  ieNoProxy = wstrToCstr(ieProxyConfig-lpszProxyBypass);
 +
 +  /* Convert the ieNoProxy into a proper no_proxy value */
-+  no_proxy = strdup(ieNoProxy);
-+  pos = strpbrk(no_proxy, ; );
-+  while(NULL != pos) {
-+no_proxy[pos-no_proxy] = ',';
++  if(NULL != no_proxy) {
++no_proxy = strdup(ieNoProxy);
 +pos = strpbrk(no_proxy, ; );
++while(NULL != pos) {
++  no_proxy[pos-no_proxy] = ',';
++  pos = strpbrk(no_proxy, ; );
++}
 +  }
 +
 +  if(!check_noproxy(conn-host.name, no_proxy)) {
 +/* Look for the http proxy setting */
 +char* tok;
 +
-+tok = strtok(ieProxy, ;);
-+if(strchr(tok, '=') == NULL) {
-+  proxy = strdup(ieProxy);
-+}
-+else {
-+  do {
-+if(strncmp(tok, http=, 5) == 0) {
-+  /* We found HTTP proxy value, then use it */
-+  proxy = strdup( tok + 5 );
-+}
++if (NULL != ieProxy) {
++  tok = strtok(ieProxy, ;);
++  if(strchr(tok, '=') == NULL) {
++proxy = strdup(ieProxy);
 +  }
-+  while(NULL != strtok(NULL, ;));
++  else {
++do {
++  if(strncmp(tok, http=, 5) == 0) {
++/* We found HTTP proxy value, then use it */
++proxy = strdup( tok + 5 );
++  }
++}
++while(NULL != strtok(NULL, ;));
++  }
 +}
 +  }
 +
@@ -112,7 +116,7 @@
char proxy_env[128];
  
no_proxy=curl_getenv(no_proxy);
-@@ -4189,9 +4259,9 @@ static char *detect_proxy(struct connectdata *conn)
+@@ -4189,9 +4264,9 @@ static char *detect_proxy(struct connectdata *conn)
  }
} /* if(!check_noproxy(conn-host.name, no_proxy)) - it wasn't specified
 non-proxy */

-- 
To view, visit https://gerrit.libreoffice.org/1802
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieb8e782eab9c9bde1d2099d8cc32eceef9f2
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0]: Template Manager: show names on several lines

2013-01-16 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1715

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/15/1715/1

Template Manager: show names on several lines

Change-Id: I6990bb6ac52889de36f3ec2cce33e4dc2f3da2ea
---
M sfx2/inc/sfx2/thumbnailviewitem.hxx
M sfx2/source/control/templatecontaineritem.cxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/control/templateviewitem.cxx
M sfx2/source/control/thumbnailviewitem.cxx
5 files changed, 70 insertions(+), 76 deletions(-)



diff --git a/sfx2/inc/sfx2/thumbnailviewitem.hxx 
b/sfx2/inc/sfx2/thumbnailviewitem.hxx
index 9609667..18b7eae 100644
--- a/sfx2/inc/sfx2/thumbnailviewitem.hxx
+++ b/sfx2/inc/sfx2/thumbnailviewitem.hxx
@@ -22,6 +22,7 @@
 
 #include basegfx/vector/b2dvector.hxx
 #include drawinglayer/attribute/fontattribute.hxx
+#include drawinglayer/primitive2d/baseprimitive2d.hxx
 #include osl/mutex.hxx
 #include vcl/bitmapex.hxx
 #include sfx2/dllapi.h
@@ -106,6 +107,7 @@
 
 virtual void Paint (drawinglayer::processor2d::BaseProcessor2D *pProcessor,
 const ThumbnailItemAttributes *pAttrs);
+void addTextPrimitives (const rtl::OUString rText, const 
ThumbnailItemAttributes *pAttrs, Point aPos, 
drawinglayer::primitive2d::Primitive2DSequence rSeq);
 
 static drawinglayer::primitive2d::PolygonHairlinePrimitive2D*
 createBorderLine (const basegfx::B2DPolygon rPolygon);
diff --git a/sfx2/source/control/templatecontaineritem.cxx 
b/sfx2/source/control/templatecontaineritem.cxx
index 25fa5fe..febd87f 100644
--- a/sfx2/source/control/templatecontaineritem.cxx
+++ b/sfx2/source/control/templatecontaineritem.cxx
@@ -84,8 +84,6 @@
 Size aThumbSize( ( maThumbnailArea.getWidth() - 3 * nThumbPadding ) / 2, ( 
maThumbnailArea.getHeight() - 3* nThumbPadding ) / 2 );
 
 // Draw thumbnail
-Point aPos = maPrev1Pos;
-
 for (int i=0; i4; ++i)
 {
 long nPosX = 0;
@@ -153,21 +151,7 @@
 }
 }
 
-// Draw centered text below thumbnail
-aPos = maTextPos;
-
-// Create the text primitive
-basegfx::B2DHomMatrix aTextMatrix( createScaleTranslateB2DHomMatrix(
-pAttrs-aFontSize.getX(), pAttrs-aFontSize.getY(),
-double( aPos.X() ), double( aPos.Y() ) ) );
-
-aSeq[nCount++] = Primitive2DReference(
-new TextSimplePortionPrimitive2D(aTextMatrix,
- maTitle,0,maTitle.getLength(),
- std::vector double ( ),
- pAttrs-aFontAttr,
- 
com::sun::star::lang::Locale(),
- Color(COL_BLACK).getBColor() 
) );
+addTextPrimitives(maTitle, pAttrs, maTextPos, aSeq);
 
 pProcessor-process(aSeq);
 }
diff --git a/sfx2/source/control/templatelocalview.cxx 
b/sfx2/source/control/templatelocalview.cxx
index 7956101..76516d7 100644
--- a/sfx2/source/control/templatelocalview.cxx
+++ b/sfx2/source/control/templatelocalview.cxx
@@ -51,12 +51,6 @@
 {
 OUString aRegionName(mpDocTemplates-GetFullRegionName(i));
 
-if ((sal_uInt32)aRegionName.getLength()  mpItemAttrs-nMaxTextLenght)
-{
-aRegionName = aRegionName.copy(0,mpItemAttrs-nMaxTextLenght-3);
-aRegionName += ...;
-}
-
 TemplateContainerItem* pItem = new TemplateContainerItem( *this );
 pItem-mnId = i+1;
 pItem-maTitle = aRegionName;
@@ -68,12 +62,6 @@
 {
 OUString aName = mpDocTemplates-GetName(i,j);
 OUString aURL = mpDocTemplates-GetPath(i,j);
-
-if ((sal_uInt32)aName.getLength()  mpItemAttrs-nMaxTextLenght)
-{
-aName = aName.copy(0,mpItemAttrs-nMaxTextLenght-3);
-aName += ...;
-}
 
 TemplateItemProperties aProperties;;
 aProperties.nId = j+1;
@@ -181,12 +169,6 @@
 return false;
 
 OUString aRegionName = rName;
-
-if ((sal_uInt32)aRegionName.getLength()  mpItemAttrs-nMaxTextLenght)
-{
-aRegionName = aRegionName.copy(0,mpItemAttrs-nMaxTextLenght-3);
-aRegionName += ...;
-}
 
 TemplateContainerItem* pItem = new TemplateContainerItem( *this );
 pItem-mnId = nRegionId+1;
diff --git a/sfx2/source/control/templateviewitem.cxx 
b/sfx2/source/control/templateviewitem.cxx
index 51a2862..9f06ebb 100644
--- a/sfx2/source/control/templateviewitem.cxx
+++ b/sfx2/source/control/templateviewitem.cxx
@@ -108,34 +108,11 @@
 // draw thumbnail borders
 aSeq[3] = Primitive2DReference(createBorderLine(aBounds));
 
-// Draw centered text below thumbnail
-
-// Create the text primitive
-basegfx::B2DHomMatrix aTitleMatrix( createScaleTranslateB2DHomMatrix(
-pAttrs-aFontSize.getX(), pAttrs-aFontSize.getY(),
-

[PATCH] Change in core[libreoffice-4-0]: Template Manager: delete popup menus in the proper order

2013-01-15 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1689

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/89/1689/1

Template Manager: delete popup menus in the proper order

Change-Id: Iafccfaa572b945d8d75d926ccc1935320b9d70d9
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 388f0cb..f1e69f5 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -209,9 +209,9 @@
 delete mpSearchView;
 delete maView;
 delete mpOnlineView;
+delete mpTemplateDefaultMenu;
 delete mpActionMenu;
 delete mpRepositoryMenu;
-delete mpTemplateDefaultMenu;
 delete mpToolbars;
 }
 

-- 
To view, visit https://gerrit.libreoffice.org/1689
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iafccfaa572b945d8d75d926ccc1935320b9d70d9
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0]: Template Manager: removed All category

2013-01-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1674

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/74/1674/1

Template Manager: removed All category

Change-Id: I073b9601c82a464f705df30cfd7e760c212eb7ca
---
M sfx2/source/doc/templatedlg.cxx
M sfx2/source/doc/templatedlg.hrc
M sfx2/source/doc/templatedlg.src
3 files changed, 0 insertions(+), 9 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index bff2dc1..7bd3741 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -253,9 +253,6 @@
 case FILTER_DRAWS:
 eFilter = FILTER_APP_DRAW;
 break;
-default:
-case FILTER_ALL:
-;
 }
 mpCurView-filterTemplatesByApp(eFilter);
 return 0;
diff --git a/sfx2/source/doc/templatedlg.hrc b/sfx2/source/doc/templatedlg.hrc
index 02c6702..672575c 100644
--- a/sfx2/source/doc/templatedlg.hrc
+++ b/sfx2/source/doc/templatedlg.hrc
@@ -7,7 +7,6 @@
  */
 
 #define TAB_CONTROL 1
-#define FILTER_ALL  2
 #define FILTER_DOCS 3
 #define FILTER_PRESENTATIONS4
 #define FILTER_SHEETS   5
diff --git a/sfx2/source/doc/templatedlg.src b/sfx2/source/doc/templatedlg.src
index 2480862..a351431 100644
--- a/sfx2/source/doc/templatedlg.src
+++ b/sfx2/source/doc/templatedlg.src
@@ -121,11 +121,6 @@
 {
 PageItem
 {
-Identifier = FILTER_ALL;
-Text [ en-US ] = All;
-};
-PageItem
-{
 Identifier = FILTER_DOCS;
 Text [ en-US ] = Documents;
 };

-- 
To view, visit https://gerrit.libreoffice.org/1674
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I073b9601c82a464f705df30cfd7e760c212eb7ca
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0]: Template Manager: filter the results when opening the dialog

2013-01-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1675

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/75/1675/1

Template Manager: filter the results when opening the dialog

Conflicts:
sfx2/source/control/templateabstractview.cxx

Change-Id: If6cc5f6d5e502c1fc5da210179263be63fc8c8a6
---
M sfx2/source/control/templateabstractview.cxx
M sfx2/source/doc/templatedlg.cxx
2 files changed, 21 insertions(+), 2 deletions(-)



diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index f8f20f5..182d833 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -102,7 +102,7 @@
 : ThumbnailView(pParent,nWinStyle,bDisableTransientChildren),
   mpItemView(new TemplateView(this)),
   mbFilteredResults(false),
-  meFilterOption(FILTER_APP_NONE)
+  meFilterOption(FILTER_APP_WRITER)
 {
 
mpItemView-setItemStateHdl(LINK(this,TemplateAbstractView,OverlayItemStateHdl));
 }
@@ -111,7 +111,7 @@
 : ThumbnailView(pParent,rResId,bDisableTransientChildren),
   mpItemView(new TemplateView(this)),
   mbFilteredResults(false),
-  meFilterOption(FILTER_APP_NONE)
+  meFilterOption(FILTER_APP_WRITER)
 {
 
mpItemView-setItemStateHdl(LINK(this,TemplateAbstractView,OverlayItemStateHdl));
 }
@@ -174,6 +174,23 @@
 }
 }
 
+void TemplateAbstractView::showOverlay (bool bVisible)
+{
+mpItemView-Show(bVisible);
+
+// Clear items is the overlay is closed.
+if (!bVisible)
+{
+// Check if the folder view needs to be filtered
+if (mbFilteredResults)
+{
+filterItems(ViewFilter_Application(meFilterOption));
+}
+
+mpItemView-Clear();
+}
+}
+
 void TemplateAbstractView::filterTemplatesByKeyword(const OUString rKeyword)
 {
 if (mpItemView-IsVisible())
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 7bd3741..388f0cb 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -190,6 +190,8 @@
 maView-Populate();
 maView-Show();
 
+mpCurView-filterTemplatesByApp(FILTER_APP_WRITER);
+
 FreeResource();
 }
 

-- 
To view, visit https://gerrit.libreoffice.org/1675
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If6cc5f6d5e502c1fc5da210179263be63fc8c8a6
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0]: Template Manager: show folders with 4 previews inside a roun...

2013-01-14 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1678

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/78/1678/1

Template Manager: show folders with 4 previews inside a rounded rect

The previous display had the inconvenient to be misleading if there was
one or no template inside a folder.

Change-Id: I16faa2556ca02380bd4dac0d821ecac45b98b8c3
---
M sfx2/inc/sfx2/templatecontaineritem.hxx
M sfx2/source/control/templateabstractview.cxx
M sfx2/source/control/templatecontaineritem.cxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/control/thumbnailviewitem.cxx
5 files changed, 131 insertions(+), 63 deletions(-)



diff --git a/sfx2/inc/sfx2/templatecontaineritem.hxx 
b/sfx2/inc/sfx2/templatecontaineritem.hxx
index 2c2d18b..4768604 100644
--- a/sfx2/inc/sfx2/templatecontaineritem.hxx
+++ b/sfx2/inc/sfx2/templatecontaineritem.hxx
@@ -18,19 +18,26 @@
 public:
 
 BitmapEx maPreview2;
+BitmapEx maPreview3;
+BitmapEx maPreview4;
 std::vectorTemplateItemProperties maTemplates;
 
 TemplateContainerItem (ThumbnailView rView);
 
 virtual ~TemplateContainerItem ();
 
-const Point getPrev2Pos () const { return maPrev2Pos; }
-
 virtual void Paint (drawinglayer::processor2d::BaseProcessor2D *pProcessor,
 const ThumbnailItemAttributes *pAttrs);
+
+virtual void calculateItemsPosition (const long nThumbnailHeight, const 
long nDisplayHeight,
+ const long nPadding, sal_uInt32 
nMaxTextLenght,
+ const ThumbnailItemAttributes 
*pAttrs);
+
+bool HasMissingPreview( );
+
 private:
 
-Point maPrev2Pos;
+Rectangle maThumbnailArea;
 };
 
 #endif // TEMPLATEFOLDERVIEWITEM_HXX
diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index f8f20f5..4439725 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -65,8 +65,10 @@
 // Clear thumbnails
 pContainerItem-maPreview1.Clear();
 pContainerItem-maPreview2.Clear();
+pContainerItem-maPreview3.Clear();
+pContainerItem-maPreview4.Clear();
 
-for (size_t i = 0, n = rTemplates.size(); i  n; ++i)
+for (size_t i = 0, n = rTemplates.size(); i  n  
pContainerItem-HasMissingPreview(); ++i)
 {
 if (isValid(rTemplates[i].aPath))
 {
@@ -83,6 +85,18 @@

TEMPLATE_THUMBNAIL_MAX_WIDTH*0.75,

TEMPLATE_THUMBNAIL_MAX_HEIGHT*0.75);
 }
+else if ( pContainerItem-maPreview3.IsEmpty() )
+{
+pContainerItem-maPreview3 = 
TemplateAbstractView::scaleImg(rTemplates[i].aThumbnail,
+   
TEMPLATE_THUMBNAIL_MAX_WIDTH*0.75,
+   
TEMPLATE_THUMBNAIL_MAX_HEIGHT*0.75);
+}
+else if ( pContainerItem-maPreview4.IsEmpty() )
+{
+pContainerItem-maPreview4 = 
TemplateAbstractView::scaleImg(rTemplates[i].aThumbnail,
+   
TEMPLATE_THUMBNAIL_MAX_WIDTH*0.75,
+   
TEMPLATE_THUMBNAIL_MAX_HEIGHT*0.75);
+}
 }
 }
 
diff --git a/sfx2/source/control/templatecontaineritem.cxx 
b/sfx2/source/control/templatecontaineritem.cxx
index d63ce8a..25fa5fe 100644
--- a/sfx2/source/control/templatecontaineritem.cxx
+++ b/sfx2/source/control/templatecontaineritem.cxx
@@ -8,6 +8,7 @@
  */
 
 #include sfx2/templatecontaineritem.hxx
+#include sfx2/templateabstractview.hxx
 
 #include basegfx/matrix/b2dhommatrixtools.hxx
 #include basegfx/polygon/b2dpolygon.hxx
@@ -35,16 +36,31 @@
 {
 }
 
+void TemplateContainerItem::calculateItemsPosition (const long 
nThumbnailHeight, const long nDisplayHeight,
+ const long nPadding, sal_uInt32 
nMaxTextLenght,
+ const ThumbnailItemAttributes *pAttrs)
+{
+ThumbnailViewItem::calculateItemsPosition( nThumbnailHeight, 
nDisplayHeight, nPadding, nMaxTextLenght, pAttrs);
+Point aPos (maDrawArea.getX() + nPadding, maDrawArea.getY() + nPadding);
+maThumbnailArea = Rectangle(aPos, Size(maDrawArea.GetWidth() - 2 * 
nPadding, nThumbnailHeight));
+}
+
 void TemplateContainerItem::Paint (drawinglayer::processor2d::BaseProcessor2D 
*pProcessor,
 const ThumbnailItemAttributes *pAttrs)
 {
 int nCount = 0;
-int nSeqSize = 2;
+int nSeqSize = 3;
 
 if (!maPreview1.IsEmpty())
 nSeqSize += 3;
 
 if 

[PATCH] Change in core[libreoffice-4-0]: Template Manager: don't deselect item after setting default ...

2013-01-09 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1611

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/11/1611/1

Template Manager: don't deselect item after setting default template

Change-Id: I860fb2c91d6843a1d5345ab2cf9270473f73f5bc
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 8 insertions(+), 10 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 282372d..bff2dc1 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -1069,19 +1069,17 @@
 
 void SfxTemplateManagerDlg::OnTemplateAsDefault ()
 {
-assert(!maSelTemplates.empty());
-
-const TemplateViewItem *pItem = static_castconst 
TemplateViewItem*(*(maSelTemplates.begin()));
-
-OUString aServiceName;
-if (lcl_getServiceName(pItem-getPath(),aServiceName))
+if (!maSelTemplates.empty())
 {
-SfxObjectFactory::SetStandardTemplate(aServiceName,pItem-getPath());
+const TemplateViewItem *pItem = static_castconst 
TemplateViewItem*(*(maSelTemplates.begin()));
 
-createDefaultTemplateMenu();
+OUString aServiceName;
+if (lcl_getServiceName(pItem-getPath(),aServiceName))
+{
+
SfxObjectFactory::SetStandardTemplate(aServiceName,pItem-getPath());
 
-// clear selection and display view/action toolbars
-maView-deselectOverlayItem(pItem-mnId);
+createDefaultTemplateMenu();
+}
 }
 }
 

-- 
To view, visit https://gerrit.libreoffice.org/1611
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I860fb2c91d6843a1d5345ab2cf9270473f73f5bc
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PATCH] Change in core[libreoffice-4-0]: Template Manager: double click on the whole item instead of ...

2013-01-09 Thread Bosdonnat Cedric (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1610

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/10/1610/1

Template Manager: double click on the whole item instead of only thumbnail

Change-Id: Ie07d41e5dcb09977d89752e724fcbabd6c5d7df1
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 0 insertions(+), 1 deletion(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index c3957f0..62324a7 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -517,7 +517,6 @@
 maItemStateHdl.Call(pItem);
 
 Rectangle aRect(pItem-getDrawArea());
-aRect.SetSize(Size(mnItemWidth,mnThumbnailHeight));
 
 if (aRect.IsInside(rMEvt.GetPosPixel()))
 OnItemDblClicked(pItem);

-- 
To view, visit https://gerrit.libreoffice.org/1610
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie07d41e5dcb09977d89752e724fcbabd6c5d7df1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] Modify the default Style list in Writer, give them a custom ...

2012-12-06 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1248

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/1248
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I780eaecc29c88493f09194399dfc00be1484c549
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Samuel Mehrbrodt s.mehrbr...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


[PUSHED] fdo#34897 numbers were wrong, code unnecessary

2012-11-28 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1165

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/1165
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0b1cac78e549c2126d7e45298629e28fc76ac95d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Lennard Wasserthal wassert...@nefkom.net
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Lennard Wasserthal wassert...@nefkom.net

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


[PUSHED] fdo#48317 - Support jumping to next/previous change

2012-11-27 Thread Bosdonnat Cedric (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1156

Approvals:
  Bosdonnat Cedric: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/1156
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I03d583bef4225409f69934f16db1854564c2db5f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Muhammad Haggag mhag...@gmail.com
Gerrit-Reviewer: Bosdonnat Cedric cedric.bosdon...@free.fr

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


  1   2   >