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)" <heinzless...@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

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

Reply via email to