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

Hi Jean-Baptiste, Thorsten,

> 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.

Hm, no, that seems like a mishap that happened because my earlier change
to the About dialogue and the menu is not in 3.6. The attached one-line
change should help, although I'd somewhat prefer if we could have the
duplication-removal patch in 3.6, too.
Otherwise, we have both menu items and About dialogue buttons for
License and Credits – while the former each lead to a local document,
the latter lead to an internet resource.

Astron.


From 9673e9cc0530196e3a66297b1237480c01a191ee Mon Sep 17 00:00:00 2001
From: "Stefan Knorr (astron)" <heinzlesspam@gmail.com>
Date: Tue, 3 Jul 2012 21:19:54 +0200
Subject: [PATCH] Make sure About can accommodate 4 buttons, not just 3

Change-Id: Id5fc92f1ef2e8810ec6603bbcebf454b9dbdedbb
---
 cui/source/dialogs/about.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cui/source/dialogs/about.cxx b/cui/source/dialogs/about.cxx
index db5057f..681e792 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.9.5