9:45 p.m.
Tested on Fedora 20.
Crístian Viana (1):
Do not show a success msg when creating a template
ui/js/src/kimchi.template_add_main.js | 1 -
1 file changed, 1 deletion(-)
--
1.8.5.3
9:45 p.m.
When the user creates a new template, a green success message is
displayed on top of the screen. This behavior is not common in Kimchi;
when an operation performs successfully, nothing is shown. Actually, the
templates page is the only one that does that.
In order to make Kimchi more consistent, remove the code that displays a
success message when the user creates a new template.
Signed-off-by: Crístian Viana <vianac(a)linux.vnet.ibm.com>
---
ui/js/src/kimchi.template_add_main.js | 1 -
1 file changed, 1 deletion(-)
diff --git a/ui/js/src/kimchi.template_add_main.js
b/ui/js/src/kimchi.template_add_main.js
index 5fd6532..dbb3952 100644
--- a/ui/js/src/kimchi.template_add_main.js
+++ b/ui/js/src/kimchi.template_add_main.js
@@ -365,7 +365,6 @@ kimchi.template_add_main = function() {
};
kimchi.createTemplate(data, function() {
successNum++;
- kimchi.message.success(i18n['KCHTMPL6001M'] + ': ' +
isoInfo.name);
$('input[value="' + isoInfo.isoId +
'"]').prop('checked', false);
$('.check-all>input').prop('checked', false);
kimchi.doListTemplates();
--
1.8.5.3
9:39 a.m.
Reviewed-by: ShaoHe Feng <shaohef(a)linux.vnet.ibm.com>
not seen this green message on my host.
But agree we should keep consistent.
On 03/08/2014 04:45 AM, Crístian Viana wrote:
When the user creates a new template, a green success message is
displayed on top of the screen. This behavior is not common in Kimchi;
when an operation performs successfully, nothing is shown. Actually, the
templates page is the only one that does that.
In order to make Kimchi more consistent, remove the code that displays a
success message when the user creates a new template.
Signed-off-by: Crístian Viana <vianac(a)linux.vnet.ibm.com>
---
ui/js/src/kimchi.template_add_main.js | 1 -
1 file changed, 1 deletion(-)
diff --git a/ui/js/src/kimchi.template_add_main.js
b/ui/js/src/kimchi.template_add_main.js
index 5fd6532..dbb3952 100644
--- a/ui/js/src/kimchi.template_add_main.js
+++ b/ui/js/src/kimchi.template_add_main.js
@@ -365,7 +365,6 @@ kimchi.template_add_main = function() {
};
kimchi.createTemplate(data, function() {
successNum++;
- kimchi.message.success(i18n['KCHTMPL6001M'] + ': ' +
isoInfo.name);
$('input[value="' + isoInfo.isoId +
'"]').prop('checked', false);
$('.check-all>input').prop('checked', false);
kimchi.doListTemplates();
--
Thanks and best regards!
Sheldon Feng(冯少合)<shaohef(a)linux.vnet.ibm.com>
IBM Linux Technology Center
2:42 p.m.
On 03/10/2014 09:24 PM, Crístian Viana wrote:
--
Thanks and best regards!
Sheldon Feng(冯少合)<shaohef(a)linux.vnet.ibm.com>
IBM Linux Technology Center
Am 10-03-2014 05:39, schrieb Sheldon:
> not seen this green message on my host.
Try to create a new template. Doesn't it happen on your system?? It
does on mine every time... (and there's the code there, afterall).
I have
tried.
yes. I see the code.
So Reviewed. :-)
3721
days inactive
3724
days old
5 comments
3 participants
participants (3)
-
Aline Manera -
Crístian Viana -
Sheldon