You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ab...@apache.org on 2018/08/31 11:07:35 UTC
[ambari] branch trunk updated: AMBARI-24561 Update styles for
pre-upgrade modals. (ababiichuk)
This is an automated email from the ASF dual-hosted git repository.
ababiichuk pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ambari.git
The following commit(s) were added to refs/heads/trunk by this push:
new 6451f81 AMBARI-24561 Update styles for pre-upgrade modals. (ababiichuk)
6451f81 is described below
commit 6451f81783a11b9e159763d98df8a9691062ddc9
Author: ababiichuk <ab...@hortonworks.com>
AuthorDate: Wed Aug 29 15:41:55 2018 +0300
AMBARI-24561 Update styles for pre-upgrade modals. (ababiichuk)
---
.../main/admin/stack_and_upgrade_controller.js | 8 +-
ambari-web/app/styles/stack_versions.less | 26 ++--
.../common/modal_popups/cluster_check_dialog.hbs | 12 +-
.../stack_upgrade/upgrade_configs_merge_table.hbs | 4 +-
.../upgrade_configs_recommend_table.hbs | 4 +-
.../main/admin/stack_upgrade/upgrade_options.hbs | 138 ++++++++++-----------
.../app/templates/wizard/step8/step8_log_popup.hbs | 1 -
.../common/modal_popups/cluster_check_popup.js | 1 +
8 files changed, 90 insertions(+), 104 deletions(-)
diff --git a/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js b/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
index ac04c88..e76fef0 100644
--- a/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
+++ b/ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js
@@ -896,7 +896,7 @@ App.MainAdminStackAndUpgradeController = Em.Controller.extend(App.LocalStorage,
*/
showPreparingUpgradeIndicator: function () {
return App.ModalPopup.show({
- header: '',
+ header: Em.I18n.t('admin.stackUpgrade.dialog.prepareUpgrade.header'),
showFooter: false,
showCloseButton: false,
bodyClass: Em.View.extend({
@@ -911,12 +911,6 @@ App.MainAdminStackAndUpgradeController = Em.Controller.extend(App.LocalStorage,
progressBarClass: 'progress log_popup',
/**
- * Popup-message
- * @type {string}
- */
- message: Em.I18n.t('admin.stackUpgrade.dialog.prepareUpgrade.header'),
-
- /**
* Hide popup when upgrade wizard is open
* @method autoHide
*/
diff --git a/ambari-web/app/styles/stack_versions.less b/ambari-web/app/styles/stack_versions.less
index 8a88ac9..911475c 100644
--- a/ambari-web/app/styles/stack_versions.less
+++ b/ambari-web/app/styles/stack_versions.less
@@ -618,6 +618,8 @@
}
.img-thumbnail {
background-color: #e4e4e4;
+ text-align: center;
+ padding: 10px;
.method-name,.method-icon,.method-description {
color: #333;
}
@@ -630,25 +632,13 @@
.method-name {
font-size: 16px;
font-weight: bold;
- text-align: center;
margin-top: 2px;
}
.method-description {
margin-top: 5px;
- text-align: center;
}
}
- .ROLLING {
- .method-icon {
- margin-left: 88px;
- }
- }
- .NON_ROLLING, .HOST_ORDERED {
- .method-icon {
- margin-left: 94px;
- }
- }
.selected.ROLLING, .selected.NON_ROLLING {
background-color: #d3e7ca;
box-shadow: 0 8px 6px -6px #b3b3b3;
@@ -700,7 +690,6 @@
display: inline-block;
width: auto;
height: @spinner-small-height;
- padding-left: @spinner-small-width + 4px;
i {
font-size: 1em;
}
@@ -727,10 +716,13 @@
font-size: 16px;
}
}
- .tolerance-text .tolerance-option {
- margin: 5px 15px;
- .ember-checkbox {
- margin-top: 0;
+ .tolerance-text {
+ margin-bottom: 30px;
+ .tolerance-option {
+ margin: 5px 15px;
+ .ember-checkbox {
+ margin-top: 0;
+ }
}
}
.text {
diff --git a/ambari-web/app/templates/common/modal_popups/cluster_check_dialog.hbs b/ambari-web/app/templates/common/modal_popups/cluster_check_dialog.hbs
index 83a57a6..23f5276 100644
--- a/ambari-web/app/templates/common/modal_popups/cluster_check_dialog.hbs
+++ b/ambari-web/app/templates/common/modal_popups/cluster_check_dialog.hbs
@@ -24,9 +24,9 @@
<h4>{{view.failTitle}}</h4>
{{/if}}
{{#if view.failAlert}}
- <div class="alert alert-warning">
+ <p>
{{view.failAlert}}
- </div>
+ </p>
{{/if}}
<div class="limited-height-2">
{{#each item in view.fails}}
@@ -42,9 +42,9 @@
<h4>{{view.failTitle}}</h4>
{{/if}}
{{#if view.failAlert}}
- <div class="alert alert-warning">
+ <p>
{{view.failAlert}}
- </div>
+ </p>
{{/if}}
<div class="limited-height-2">
{{#each item in view.bypass}}
@@ -59,9 +59,9 @@
<h4>{{view.warningTitle}}</h4>
{{/if}}
{{#if view.warningAlert}}
- <div class="alert alert-warning">
+ <p>
{{view.warningAlert}}
- </div>
+ </p>
{{/if}}
<div class="limited-height-2">
{{#each item in view.warnings}}
diff --git a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_merge_table.hbs b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_merge_table.hbs
index b78ba5a..c9a17bd 100644
--- a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_merge_table.hbs
+++ b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_merge_table.hbs
@@ -20,9 +20,9 @@
<a {{action openConfigsInNewWindow view.configs target="App.router.mainAdminStackAndUpgradeController"}} {{translateAttr title="common.openNewWindow"}} class="pull-right open-in-new-window" href="#">
<i class="icon-external-link"></i> <span>{{t common.open}}</span>
</a>
-<div class="alert alert-warning">
+<p>
{{t popup.clusterCheck.Upgrade.configsMerge.alert}}
-</div>
+</p>
<table class="configs-table table table-hover">
<thead>
<tr>
diff --git a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_recommend_table.hbs b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_recommend_table.hbs
index 8df3327..b962cad 100644
--- a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_recommend_table.hbs
+++ b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_configs_recommend_table.hbs
@@ -20,9 +20,9 @@
<a {{action openConfigsInNewWindow view.configs target="App.router.mainAdminStackAndUpgradeController"}} {{translateAttr title="common.openNewWindow"}} class="pull-right open-in-new-window" href="#">
<i class="icon-external-link"></i> <span id="i18n-34">{{t common.open}}</span>
</a>
-<div class="alert alert-warning">
+<p>
{{t popup.clusterCheck.Upgrade.configsRecommend.alert}}
-</div>
+</p>
<div class="configs-table-header">
<table class="configs-table table table-striped">
<thead>
diff --git a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_options.hbs b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_options.hbs
index 63b8fa1..fff5f39 100644
--- a/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_options.hbs
+++ b/ambari-web/app/templates/main/admin/stack_upgrade/upgrade_options.hbs
@@ -17,72 +17,72 @@
}}
{{#if view.parentView.controller.isUpgradeTypesLoaded}}
- {{#if view.parentView.controller.getSupportedUpgradeError}}
- <i class="glyphicon glyphicon-remove"/>{{t admin.stackVersions.version.upgrade.upgradeOptions.error}}
- <pre>{{view.parentView.controller.getSupportedUpgradeError}}</pre>
- {{else}}
- <div id="upgrade-options-popup-content">
- <div class="text">{{{view.versionText}}}</div>
- {{#if view.upgradeShow}}
- <div class="text method-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.method}}</div>
- {{/if}}
- <div {{bindAttr class=":row :method-options view.isInUpgradeWizard:disabled"}}>
- {{#each method in view.upgradeMethods}}
- {{#unless method.cantBeStarted}}
- <div class="method-option col-md-6">
- <div {{bindAttr class="method.allowed::not-allowed method.allowed::not-allowed-by-version
+ {{#if view.parentView.controller.getSupportedUpgradeError}}
+ <i class="glyphicon glyphicon-remove"/>{{t admin.stackVersions.version.upgrade.upgradeOptions.error}}
+ <pre>{{view.parentView.controller.getSupportedUpgradeError}}</pre>
+ {{else}}
+ <div id="upgrade-options-popup-content">
+ <p class="text">{{{view.versionText}}}</p>
+ {{#if view.upgradeShow}}
+ <p class="text method-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.method}}</p>
+ {{/if}}
+ <div {{bindAttr class=":row :method-options view.isInUpgradeWizard:disabled"}}>
+ {{#each method in view.upgradeMethods}}
+ {{#unless method.cantBeStarted}}
+ <div class="method-option col-md-6">
+ <div {{bindAttr class="method.allowed::not-allowed method.allowed::not-allowed-by-version
method.isPrecheckFailed:not-allowed method.isPrecheckFailed:check-failed
method.selected:selected method.type :img-thumbnail
view.upgradeShow::default-cursor"}}
- {{action selectMethod method target="view"}}>
- <div {{bindAttr class="method.icon :method-icon"}}></div>
- <div class="method-name">{{method.displayName}}</div>
- <div class="method-description">{{{method.description}}}</div>
- {{#if view.showPreUpgradeChecks}}
- {{#if method.isCheckRequestInProgress}}
- <div class="method-precheck-message checking">
- {{view App.SpinnerView message="admin.stackVersions.version.upgrade.upgradeOptions.preCheck.msg.checking"}}
- </div>
- {{else}}
- <div {{bindAttr class=":method-precheck-message method.precheckResultsMessageClass"}}>
- <i {{bindAttr class="method.precheckResultsMessageIconClass"}}></i>
- <b>{{method.precheckResultsTitle}}</b>
- <a {{action runAction method target="view"}}>
- {{method.precheckResultsMessage}}
- </a>
- {{#if method.bypassedFailures}}
- <div
- class="alert-danger">{{t admin.stackVersions.version.upgrade.upgradeOptions.errors_bypassed}}</div>
- {{/if}}
- </div>
- {{/if}}
- {{/if}}
+ {{action selectMethod method target="view"}}>
+ <div {{bindAttr class="method.icon :method-icon"}}></div>
+ <div class="method-name">{{method.displayName}}</div>
+ <div class="method-description">{{{method.description}}}</div>
+ {{#if view.showPreUpgradeChecks}}
+ {{#if method.isCheckRequestInProgress}}
+ <div class="method-precheck-message checking">
+ {{view App.SpinnerView message="admin.stackVersions.version.upgrade.upgradeOptions.preCheck.msg.checking"}}
</div>
- </div>
- {{/unless}}
- {{/each}}
- </div>
- {{#if view.upgradeShow}}
- <div class="text tolerance-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.tolerance}}
- <i class="glyphicon glyphicon-question-sign failure-tolerance-tooltip" data-toggle="tooltip"></i>
- <div>
- {{view App.CheckboxView
- labelClassNames="tolerance-option"
- labelTranslate="admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option2"
- checkedBinding="view.parentView.skipSCFailures"}}
- </div>
- <div>
- {{view App.CheckboxView
- labelClassNames="tolerance-option"
- labelTranslate="admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option1"
- checkedBinding="view.parentView.skipComponentFailures"
- }}
+ {{else}}
+ <div {{bindAttr class=":method-precheck-message method.precheckResultsMessageClass"}}>
+ <i {{bindAttr class="method.precheckResultsMessageIconClass"}}></i>
+ <b>{{method.precheckResultsTitle}}</b>
+ <a {{action runAction method target="view"}}>
+ {{method.precheckResultsMessage}}
+ </a>
+ {{#if method.bypassedFailures}}
+ <div
+ class="alert-danger">{{t admin.stackVersions.version.upgrade.upgradeOptions.errors_bypassed}}</div>
+ {{/if}}
+ </div>
+ {{/if}}
+ {{/if}}
+ </div>
</div>
- </div>
- {{/if}}
- <div class="alert alert-warning">{{t admin.stackVersions.version.upgrade.alertsWarning}}</div>
+ {{/unless}}
+ {{/each}}
</div>
- {{/if}}
+ {{#if view.upgradeShow}}
+ <div class="text tolerance-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.tolerance}}
+ <i class="glyphicon glyphicon-question-sign failure-tolerance-tooltip" data-toggle="tooltip"></i>
+ <div>
+ {{view App.CheckboxView
+ labelClassNames="tolerance-option"
+ labelTranslate="admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option2"
+ checkedBinding="view.parentView.skipSCFailures"}}
+ </div>
+ <div>
+ {{view App.CheckboxView
+ labelClassNames="tolerance-option"
+ labelTranslate="admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option1"
+ checkedBinding="view.parentView.skipComponentFailures"
+ }}
+ </div>
+ </div>
+ {{/if}}
+ <div class="alert alert-warning">{{t admin.stackVersions.version.upgrade.alertsWarning}}</div>
+ </div>
+ {{/if}}
{{else}}
<div class="row">
<div class="col-md-1 col-md-offset-3">{{view App.SpinnerView}}</div>
@@ -90,12 +90,12 @@
{{t admin.stackVersions.version.upgrade.upgradeOptions.loading}}
</div>
</div>
- <div class="text tolerance-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.tolerance}}
- <i class="icon-question-sign failure-tolerance-tooltip" data-toggle="tooltip"></i>
- <label class="tolerance-option">{{view Ember.Checkbox checkedBinding="view.parentView.skipSCFailures"}}
- {{t admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option2}}</label>
- <label class="tolerance-option">{{view Ember.Checkbox checkedBinding="view.parentView.skipComponentFailures"}}
- {{t admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option1}}</label>
- </div>
- <div class="alert alert-warning">{{t admin.stackVersions.version.upgrade.alertsWarning}}</div>
- {{/if}}
+ <div class="text tolerance-text">{{t admin.stackVersions.version.upgrade.upgradeOptions.bodyMsg.tolerance}}
+ <i class="icon-question-sign failure-tolerance-tooltip" data-toggle="tooltip"></i>
+ <label class="tolerance-option">{{view Ember.Checkbox checkedBinding="view.parentView.skipSCFailures"}}
+ {{t admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option2}}</label>
+ <label class="tolerance-option">{{view Ember.Checkbox checkedBinding="view.parentView.skipComponentFailures"}}
+ {{t admin.stackVersions.version.upgrade.upgradeOptions.tolerance.option1}}</label>
+ </div>
+ <div class="alert alert-warning">{{t admin.stackVersions.version.upgrade.alertsWarning}}</div>
+{{/if}}
diff --git a/ambari-web/app/templates/wizard/step8/step8_log_popup.hbs b/ambari-web/app/templates/wizard/step8/step8_log_popup.hbs
index e16c785..49cfea7 100644
--- a/ambari-web/app/templates/wizard/step8/step8_log_popup.hbs
+++ b/ambari-web/app/templates/wizard/step8/step8_log_popup.hbs
@@ -15,7 +15,6 @@
* See the License for the specific language governing permissions and
* limitations under the License.
}}
-<p>{{view.message}}</p>
<div {{bindAttr class="view.progressBarClass"}}>
<div class="progress-bar progress-bar-striped active" {{bindAttr style="view.barWidth"}}>
</div>
diff --git a/ambari-web/app/views/common/modal_popups/cluster_check_popup.js b/ambari-web/app/views/common/modal_popups/cluster_check_popup.js
index bef3c90..23896b7 100644
--- a/ambari-web/app/views/common/modal_popups/cluster_check_popup.js
+++ b/ambari-web/app/views/common/modal_popups/cluster_check_popup.js
@@ -59,6 +59,7 @@ App.showClusterCheckPopup = function (data, popup, configs) {
secondary: secondary,
header: popup.header,
classNames: ['cluster-check-popup'],
+ modalDialogClasses: ['modal-xlg'],
bodyClass: Em.View.extend({
failTitle: popup.failTitle,
failAlert: popup.failAlert,