Kaydet (Commit) 0617454d authored tarafından Stefan Knorr (astron)'s avatar Stefan Knorr (astron)

Remove duplication of License/Credits information

A bit differently this time around:
+ remove Credits menu item from menu
+ remove License button from the About dialog

Change-Id: I07c1e115aa209ef1d69458f5b188d73caaae4294
üst 9cfa2b97
......@@ -103,7 +103,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -151,7 +151,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -63,7 +63,6 @@ using namespace ::com::sun::star;
enum AboutDialogButton
{
CREDITS_BUTTON,
LICENSE_BUTTON,
WEBSITE_BUTTON
};
......@@ -75,7 +74,6 @@ AboutDialog::AboutDialog( Window* pParent, const ResId& rId) :
aCopyrightTextShadow ( this, ResId( ABOUT_COPYRIGHT_TEXT, *rId.GetResMgr() ) ),
aLogoImage ( this, ResId( ABOUT_IMAGE_LOGO, *rId.GetResMgr() ) ),
aCreditsButton ( this, ResId( ABOUT_BTN_CREDITS, *rId.GetResMgr() ) ),
aLicenseButton ( this, ResId( ABOUT_BTN_LICENSE, *rId.GetResMgr() ) ),
aWebsiteButton ( this, ResId( ABOUT_BTN_WEBSITE, *rId.GetResMgr() ) ),
aCancelButton ( this, ResId( ABOUT_BTN_CANCEL, *rId.GetResMgr() ) ),
m_aVersionTextStr(ResId(ABOUT_STR_VERSION, *rId.GetResMgr()).toString().trim()),
......@@ -85,7 +83,6 @@ AboutDialog::AboutDialog( Window* pParent, const ResId& rId) :
m_aBasedDerivedTextStr(ResId(ABOUT_STR_BASED_DERIVED, *rId.GetResMgr())),
m_aWebsiteLinkStr(ResId( ABOUT_STR_LINK_WEBSITE, *rId.GetResMgr())),
m_aCreditsLinkStr(ResId( ABOUT_STR_LINK_CREDITS, *rId.GetResMgr())),
m_aLicenseLinkStr(ResId( ABOUT_STR_LINK_LICENSE, *rId.GetResMgr())),
m_sBuildStr(ResId(ABOUT_STR_BUILD, *rId.GetResMgr())),
m_aDescriptionTextStr(ResId(ABOUT_STR_DESCRIPTION, *rId.GetResMgr()))
{
......@@ -103,12 +100,10 @@ AboutDialog::AboutDialog( Window* pParent, const ResId& rId) :
// Allow the button to be identifiable once they are clicked
aCreditsButton.SetData( (void*)CREDITS_BUTTON );
aLicenseButton.SetData( (void*)LICENSE_BUTTON );
aWebsiteButton.SetData( (void*)WEBSITE_BUTTON );
// Connect all handlers
aCreditsButton.SetClickHdl( LINK( this, AboutDialog, HandleClick ) );
aLicenseButton.SetClickHdl( LINK( this, AboutDialog, HandleClick ) );
aWebsiteButton.SetClickHdl( LINK( this, AboutDialog, HandleClick ) );
aCancelButton.SetClickHdl( LINK( this, AboutDialog, CancelHdl ) );
......@@ -127,8 +122,6 @@ IMPL_LINK( AboutDialog, HandleClick, PushButton*, pButton )
AboutDialogButton* pDialogButton = (AboutDialogButton*)pButton->GetData();
if ( pDialogButton == (AboutDialogButton*)CREDITS_BUTTON )
sURL = m_aCreditsLinkStr;
else if ( pDialogButton == (AboutDialogButton*)LICENSE_BUTTON)
sURL = m_aLicenseLinkStr;
else if ( pDialogButton == (AboutDialogButton*)WEBSITE_BUTTON )
sURL = m_aWebsiteLinkStr;
......@@ -282,7 +275,6 @@ void AboutDialog::LayoutControls()
MoveControl(aCopyrightText, 0, aHeightDifference * 0.75 );
MoveControl(aCopyrightTextShadow, 0, aHeightDifference * 0.75 );
MoveControl(aCreditsButton, 0, aHeightDifference);
MoveControl(aLicenseButton, 0, aHeightDifference);
MoveControl(aWebsiteButton, 0, aHeightDifference);
MoveControl(aCancelButton, 0, aHeightDifference);
}
......@@ -317,15 +309,12 @@ void AboutDialog::LayoutButtons(sal_Int32 aDialogWidth, sal_Int32 aDialogBorder,
sal_Int32 aButtonHPadding = 4;
sal_Int32 aAdjacentButtonSpacing = 15;
sal_Int32 aCreditsButtonWidth = aCreditsButton.CalcMinimumSize().Width();
sal_Int32 aLicenseButtonWidth = aLicenseButton.CalcMinimumSize().Width();
sal_Int32 aWebsiteButtonWidth = aWebsiteButton.CalcMinimumSize().Width();
sal_Int32 aCancelButtonWidth = aCancelButton.CalcMinimumSize().Width();
sal_Int32 aLargestButtonWidth = 70;
if ( aLargestButtonWidth < aCreditsButtonWidth )
aLargestButtonWidth = aCreditsButtonWidth;
if ( aLargestButtonWidth < aLicenseButtonWidth )
aLargestButtonWidth = aLicenseButtonWidth;
if ( aLargestButtonWidth < aWebsiteButtonWidth )
aLargestButtonWidth = aWebsiteButtonWidth;
if ( aLargestButtonWidth < aCancelButtonWidth )
......@@ -335,7 +324,6 @@ void AboutDialog::LayoutButtons(sal_Int32 aDialogWidth, sal_Int32 aDialogBorder,
aButtonSize.Height() = aWebsiteButton.CalcMinimumSize().Height() + ( 2 * aButtonVPadding );
aCreditsButton.SetSizePixel( aButtonSize );
aLicenseButton.SetSizePixel( aButtonSize );
aWebsiteButton.SetSizePixel( aButtonSize );
aCancelButton.SetSizePixel( aButtonSize );
......@@ -351,9 +339,6 @@ void AboutDialog::LayoutButtons(sal_Int32 aDialogWidth, sal_Int32 aDialogBorder,
aCreditsButton.SetPosPixel( aButtonPos );
aButtonPos.X() += aButtonSize.Width() + aAdjacentButtonSpacing;
aLicenseButton.SetPosPixel( aButtonPos );
aButtonPos.X() += aButtonSize.Width() + aAdjacentButtonSpacing;
aWebsiteButton.SetPosPixel( aButtonPos );
......
......@@ -26,7 +26,6 @@
#define ABOUT_STR_BASED_DERIVED 6
#define ABOUT_STR_LINK_WEBSITE 7
#define ABOUT_STR_LINK_CREDITS 8
#define ABOUT_STR_LINK_LICENSE 9
#define ABOUT_BTN_CREDITS 10
#define ABOUT_DESCRIPTION_TEXT 11
#define ABOUT_COPYRIGHT_TEXT 12
......
......@@ -105,18 +105,10 @@ ModalDialog RID_DEFAULTABOUT
{
Text[ en-US ] = "http://www.libreoffice.org/about-us/credits/";
};
String ABOUT_STR_LINK_LICENSE
{
Text[ en-US ] = "http://www.libreoffice.org/download/license/";
};
PushButton ABOUT_BTN_CREDITS
{
Text [ en-US ] = "Credits";
};
PushButton ABOUT_BTN_LICENSE
{
Text [ en-US ] = "License";
};
PushButton ABOUT_BTN_WEBSITE
{
Text [ en-US ] = "Website";
......
......@@ -52,7 +52,6 @@ private:
MultiLineEdit aCopyrightTextShadow;
FixedImage aLogoImage;
PushButton aCreditsButton;
PushButton aLicenseButton;
PushButton aWebsiteButton;
CancelButton aCancelButton;
......@@ -64,7 +63,6 @@ private:
String m_aBasedDerivedTextStr;
String m_aWebsiteLinkStr;
String m_aCreditsLinkStr;
String m_aLicenseLinkStr;
rtl::OUString m_sBuildStr;
String m_aDescriptionTextStr;
......
......@@ -153,7 +153,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -94,7 +94,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -77,7 +77,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -78,7 +78,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -86,7 +86,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -77,7 +77,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -426,7 +426,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -469,7 +469,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -338,7 +338,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -314,7 +314,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -125,7 +125,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -463,7 +463,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -393,7 +393,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -458,7 +458,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -458,7 +458,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -470,7 +470,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
......@@ -459,7 +459,6 @@
<menu:menuitem menu:id=".uno:SendFeedback"/>
<menu:menuseparator/>
<menu:menuitem menu:id=".uno:ShowLicense"/>
<menu:menuitem menu:id=".uno:ShowCredits"/>
<menu:menuitem menu:id=".uno:About"/>
</menu:menupopup>
</menu:menu>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment