Re: [Review-3-6?] New artwork for 3.6

Le 03/07/2012 19:47, Jean-Baptiste Faure a écrit :
> Le 03/07/2012 14:42, Stefan Knorr a écrit :
> > Hi again,
> >
> > the license restriction doesn't apply any more, as luckily Andrea
> > Soragna (sorry misspelled him in the commit messages) has sent us a
> > license statement [1]. Thanks to him!
> Hi,
>
> There is no Close button on the new About dialog. Is it intended ?
> I remember that for the previous change in this dialog, we had have the
> same problem. It was fixed later.
>
> Best regards.
> JBF
Hi,

Here is a one line patch which restore a line in about.cxx modified in a
previous patch. With these patch I get the Close button.

Best regards.
JBF

-- 
Seuls des formats ouverts peuvent assurer la pérennité de vos documents.

From fa0c1095b5caabd68036f2110f0ef1197720d2c5 Mon Sep 17 00:00:00 2001
From: Jean-Baptiste FAURE <jbf.faure@orange.fr>
Date: Tue, 3 Jul 2012 21:01:47 +0200
Subject: [PATCH] restore close button in About dialog

Change-Id: I32d528cf047e890815bfeaf9b5dc0516ef8ffccf
---
 cui/source/dialogs/about.cxx |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cui/source/dialogs/about.cxx b/cui/source/dialogs/about.cxx
index db5057f..e18a22c 100644
--- a/cui/source/dialogs/about.cxx
+++ b/cui/source/dialogs/about.cxx
@@ -291,7 +291,7 @@ void AboutDialog::LayoutButtons(sal_Int32 aDialogWidth, sal_Int32 aDialogBorder,
     aWebsiteButton.SetSizePixel( aButtonSize );
     aCancelButton.SetSizePixel( aButtonSize );
 
-    sal_Int32 aButtonSpacing = aDialogWidth  - ( aDialogBorder * 2 ) - ( aButtonSize.Width() * 3 ) 
- aAdjacentButtonSpacing ;
+    sal_Int32 aButtonSpacing = aDialogWidth  - ( aDialogBorder * 2 ) - ( aButtonSize.Width() * 4 ) 
- ( aAdjacentButtonSpacing * 2 );
     if (aButtonSpacing < aAdjacentButtonSpacing)
     {
         aButtonsWidth = aAdjacentButtonSpacing - aButtonSpacing;
-- 
1.7.5.4