← Back to team overview

kicad-developers team mailing list archive

Tweaks to about dialog

 

As mentioned in previous thread, I have made some slight changes to the
About dialog.

1. Wording / link tweaks
2. Removed references to outdated links (now better served from the
kicad-pcb.org site)
3. Removed "artists" tab - very outdated, 3D models are now community
contributed

I have also added myself as a developer credit - if this is an acceptable
indulgence.

Patch attached.

Regards,
From 500378b90d8f91ab25c4a0e1bcbc4e6910f6c6a9 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@xxxxxxxxx>
Date: Fri, 24 Nov 2017 00:29:30 +1100
Subject: [PATCH] Slight tweaks to About dialog

- Small wording improvements
- Hyperlinks now show URL
- Remove "artists" tab
- Remove links to outdated library sources
- Added myself as developer credit
---
 common/dialog_about/AboutDialog_main.cpp | 46 +++++++++++---------------------
 common/dialog_about/dialog_about.cpp     |  2 +-
 2 files changed, 17 insertions(+), 31 deletions(-)

diff --git a/common/dialog_about/AboutDialog_main.cpp b/common/dialog_about/AboutDialog_main.cpp
index e23d17c..4929b76 100644
--- a/common/dialog_about/AboutDialog_main.cpp
+++ b/common/dialog_about/AboutDialog_main.cpp
@@ -117,7 +117,7 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
 
     description << wxT( "<p>" )
                 << _( "The KiCad EDA Suite is a set of open source applications for the "
-                      "creation of electronic schematics and to design printed circuit boards." )
+                      "creation of electronic schematics and printed circuit boards." )
                 << wxT( "</p>" );
 
     description << wxT( "</p>" );
@@ -130,34 +130,17 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
     // bullet-ed list with some http links
     description << wxT( "<ul>" );
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "http://www.kicad-pcb.org"; ),
-                                  _( "The official KiCad website" ) )
+                << wxT( "The official KiCad website - " )
+                << HtmlHyperlink( "http://www.kicad-pcb.org"; )
                 << wxT( "</li>" );
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://launchpad.net/kicad"; ),
-                                  _( "Developer's website on Launchpad" ) )
+                << wxT( "Developer website on Launchpad - " )
+                << HtmlHyperlink( "https://launchpad.net/kicad"; )
                 << wxT("</li>" );
 
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://github.com/KiCad/"; ),
-                                  _( "Official repository for component and footprint libraries" ) )
-                << wxT( "</li>" );
-
-    description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://github.com/KiCad/Footprint_Wizards"; ),
-                                  _( "Footprint wizards info on our official repository" ) )
-                << wxT( "</li>" );
-    description << wxT( "</ul></p>" );
-
-    description << wxT( "<p><b><u>" )
-                << _( "Non official repositories" )
-                << wxT( "</b></u>" );
-
-    description << wxT( "<ul>" );
-
-    description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "http://smisioto.no-ip.org/elettronica/kicad/kicad-en.htm"; ),
-                                  _( "Additional component libraries repository (smisioto)" ) )
+                << wxT( "Official KiCad library repositories - " )
+                << HtmlHyperlink( "https://github.com/KiCad/"; )
                 << wxT( "</li>" );
 
     description << wxT( "</ul></p>" );
@@ -169,8 +152,9 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
     // bullet-ed list with some http links
     description << wxT( "<ul>" );
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://bugs.launchpad.net/kicad/+bugs?orderby=-id&start=0"; ),
-                                  _( "Report or examine bugs" ) )
+                << wxT( "Report or examine bugs - " )
+                << HtmlHyperlink( "https://bugs.launchpad.net/kicad/+bugs?orderby=-id&start=0";,
+                                  "https://bugs.launchpad.net/kicad"; )
                 << wxT( "</li>" );
     description << wxT( "</ul></p>" );
 
@@ -180,13 +164,13 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
 
     description << wxT( "<ul>" );
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://groups.yahoo.com/neo/groups/kicad-users/info"; ),
-                                  _( "KiCad user's group" ) )
+                << wxT( "KiCad forum - " )
+                << HtmlHyperlink( wxT( "https://forum.kicad.info"; ) )
                 << wxT( "</li>" );
 
     description << wxT( "<li>" )
-                << HtmlHyperlink( wxT( "https://forum.kicad.info"; ),
-                                  _( "KiCad forum" ) )
+                << wxT( "KiCad user's group - " )
+                << HtmlHyperlink( "https://groups.yahoo.com/neo/groups/kicad-users/info"; )
                 << wxT( "</li>" );
 
     description << wxT( "</ul></p>" );
@@ -409,6 +393,7 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
                                          wxEmptyString,
                                          OTHERS_IN_TRANSLATION ) );
 
+    /*
     // Programm credits for icons
     #define ICON_CONTRIBUTION _( "Icons by" )
     aInfo.AddArtist( new CONTRIBUTOR( wxT( "Iñigo Zuluaga" ),
@@ -434,6 +419,7 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
                                      wxT( "reniemarquet@xxxxxxxxxx" ),
                                      MODELS_3D_CONTRIBUTION,
                                      KiBitmapNew( three_d_xpm ) ) );
+    */
 
     // Programm credits for package developers.
     aInfo.AddPackager( new CONTRIBUTOR( wxT( "Jean-Samuel Reynaud" ),
diff --git a/common/dialog_about/dialog_about.cpp b/common/dialog_about/dialog_about.cpp
index 0e4d881..4ec0266 100644
--- a/common/dialog_about/dialog_about.cpp
+++ b/common/dialog_about/dialog_about.cpp
@@ -123,7 +123,7 @@ void DIALOG_ABOUT::createNotebooks()
     createNotebookPage( m_auiNotebook, _( "Developers" ) , m_picDevelopers, m_info.GetDevelopers() );
     createNotebookPage( m_auiNotebook, _( "Doc Writers" ), m_picDocWriters, m_info.GetDocWriters() );
 
-    createNotebookPageByCategory( m_auiNotebook, _( "Artists" ), m_picArtists, m_info.GetArtists() );
+    //createNotebookPageByCategory( m_auiNotebook, _( "Artists" ), m_picArtists, m_info.GetArtists() );
     createNotebookPageByCategory( m_auiNotebook, _( "Translators" ), m_picTranslators,
                                   m_info.GetTranslators() );
     createNotebookPageByCategory( m_auiNotebook, _( "Packagers" ), m_picPackagers,
-- 
2.7.4


Follow ups