From d0f5c2b074ab0776bf9605ccbdbe0475c72c1041 Mon Sep 17 00:00:00 2001
From: kzantow <kzantow@gmail.com>
Date: Wed, 13 Apr 2016 10:22:09 -0400
Subject: [PATCH 1/2] JENKINS-34137 - better handling of the close button: not
 allowed while plugins installing or in certain other places, close now
 directs to setup complete panel, with message about skipping admin user
 creation

---
 war/src/main/js/pluginSetupWizardGui.js            | 4 ++--
 war/src/main/js/templates/pluginSelectionPanel.hbs | 2 +-
 war/src/main/js/templates/welcomePanel.hbs         | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/war/src/main/js/pluginSetupWizardGui.js b/war/src/main/js/pluginSetupWizardGui.js
index 4177cf4eca..94342188e2 100644
--- a/war/src/main/js/pluginSetupWizardGui.js
+++ b/war/src/main/js/pluginSetupWizardGui.js
@@ -207,7 +207,7 @@ var createPluginSetupWizard = function(appendTarget) {
 				decorate($container);
 				
 				var $modalHeader = $container.find('.modal-header');
-				if($modalHeader.length > 0) {
+				if($modalHeader.length > 0 && $modalHeader.is('.closeable')) {
 					$modalHeader.prepend(
 						'<button type="button" class="close" aria-label="Close"><span aria-hidden="true">&times;</span></button>');
 				}
@@ -815,7 +815,7 @@ var createPluginSetupWizard = function(appendTarget) {
 		'.plugin-select-recommended': function() { selectedPluginNames = pluginManager.recommendedPluginNames(); refreshPluginSelectionPanel(); },
 		'.plugin-show-selected': toggleSelectedSearch,
 		'.select-category': selectCategory,
-		'.close': closeInstaller,
+		'.close': skipFirstUser,
 		'.resume-installation': resumeInstallation,
 		'.install-done-restart': restartJenkins,
 		'.save-first-user:not([disabled])': saveFirstUser,
diff --git a/war/src/main/js/templates/pluginSelectionPanel.hbs b/war/src/main/js/templates/pluginSelectionPanel.hbs
index addd570e56..12a31294f7 100644
--- a/war/src/main/js/templates/pluginSelectionPanel.hbs
+++ b/war/src/main/js/templates/pluginSelectionPanel.hbs
@@ -1,4 +1,4 @@
-<div class="modal-header">
+<div class="modal-header closeable">
 	<h4 class="modal-title">{{translations.installWizard_installCustom_title}}</h4>
 </div>
 <div class="modal-body plugin-selector">
diff --git a/war/src/main/js/templates/welcomePanel.hbs b/war/src/main/js/templates/welcomePanel.hbs
index 5aa5742756..ccc08b98bc 100644
--- a/war/src/main/js/templates/welcomePanel.hbs
+++ b/war/src/main/js/templates/welcomePanel.hbs
@@ -1,4 +1,4 @@
-<div class="modal-header">
+<div class="modal-header closeable">
 	<h4 class="modal-title">{{translations.installWizard_welcomePanel_title}}</h4>
 </div>
 <div class="modal-body">

From 9622bbedbb80e2d710fe6ac6a536eeae871a0f2f Mon Sep 17 00:00:00 2001
From: kzantow <kzantow@gmail.com>
Date: Wed, 13 Apr 2016 10:55:40 -0400
Subject: [PATCH 2/2] JENKINS-33936 JENKINS-33937 - add links to wiki for
 plugins during setup wizard

---
 core/src/main/java/hudson/PluginManager.java                 | 2 ++
 .../resources/jenkins/install/pluginSetupWizard.properties   | 1 +
 war/src/main/js/templates/pluginSelectionPanel.hbs           | 5 +++--
 war/src/main/less/pluginSetupWizard.less                     | 4 ++++
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/hudson/PluginManager.java b/core/src/main/java/hudson/PluginManager.java
index 4e60191e6e..1f52b85678 100644
--- a/core/src/main/java/hudson/PluginManager.java
+++ b/core/src/main/java/hudson/PluginManager.java
@@ -1006,6 +1006,7 @@ public abstract class PluginManager extends AbstractModelObject implements OnMas
             pluginInfo.put("bundled", plugin.isBundled);
             pluginInfo.put("deleted", plugin.isDeleted());
             pluginInfo.put("downgradable", plugin.isDowngradable());
+            pluginInfo.put("website", plugin.getUrl());
             List<Dependency> dependencies = plugin.getDependencies();
             if (dependencies != null && !dependencies.isEmpty()) {
                 Map<String, String> dependencyMap = new HashMap<>();
@@ -1030,6 +1031,7 @@ public abstract class PluginManager extends AbstractModelObject implements OnMas
                 pluginInfo.put("excerpt", plugin.excerpt);
                 pluginInfo.put("site", site.getId());
                 pluginInfo.put("dependencies", plugin.dependencies);
+                pluginInfo.put("website", plugin.wiki);
                 response.add(pluginInfo);
             }
         }
diff --git a/core/src/main/resources/jenkins/install/pluginSetupWizard.properties b/core/src/main/resources/jenkins/install/pluginSetupWizard.properties
index a6fd62b24d..9996979c48 100644
--- a/core/src/main/resources/jenkins/install/pluginSetupWizard.properties
+++ b/core/src/main/resources/jenkins/install/pluginSetupWizard.properties
@@ -50,4 +50,5 @@ installWizard_configureProxy_save=Save and Continue
 installWizard_skipPluginInstallations=Skip Plugin Installations
 installWizard_installIncomplete_dependenciesLabel=Dependencies
 installWizard_installingConsole_dependencyIndicatorNote=** - required dependency
+installWizard_websiteLinkLabel=Website
 installWizard_retry=Retry
diff --git a/war/src/main/js/templates/pluginSelectionPanel.hbs b/war/src/main/js/templates/pluginSelectionPanel.hbs
index addd570e56..b5d295b025 100644
--- a/war/src/main/js/templates/pluginSelectionPanel.hbs
+++ b/war/src/main/js/templates/pluginSelectionPanel.hbs
@@ -40,6 +40,7 @@
 						<span class="title">
 							<input type="checkbox" id="chk-{{plugin.name}}" name="{{plugin.name}}" value="{{searchTerm}}" {{#inSelectedPlugins plugin.name}}checked="checked"{{/inSelectedPlugins}}/>
 							{{plugin.title}}
+							<a href="{{plugin.website}}" target="_blank">{{../../translations.installWizard_websiteLinkLabel}}</a>
 						</span>
 						<span class="description">
 							{{{plugin.excerpt}}}
@@ -47,9 +48,9 @@
 						{{#hasDependencies plugin.name}}
 							<div class="dep-list">
 								{{#eachDependency ../plugin.name}}
-								<span class="dep badge">
+								<a class="dep badge" href="{{website}}" target="_blank">
 								{{title}}
-								</span>
+								</a>
 								{{/eachDependency}}
 							</div>
 						{{/hasDependencies}}
diff --git a/war/src/main/less/pluginSetupWizard.less b/war/src/main/less/pluginSetupWizard.less
index 38a84b874e..0a34166ed2 100644
--- a/war/src/main/less/pluginSetupWizard.less
+++ b/war/src/main/less/pluginSetupWizard.less
@@ -432,6 +432,10 @@
 						.badge {
 							padding: 4px 8px;
 							margin: 5px 3px 0 0;
+							
+							&:hover, &:active, &:focus {
+								color: #fff;
+							}
 						}
 					}