Skip to content

Commit 1a709f0

Browse files
committed
[UX] Added loading text for install/update/delete
Signed-off-by: Akash Ravi <akashkravi@gmail.com>
1 parent b9c1198 commit 1a709f0

File tree

3 files changed

+18
-3
lines changed

3 files changed

+18
-3
lines changed

extension/js/methods.js

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,13 @@ function update_packages(project) {
6767
}
6868
else
6969
common.confirm("Update Packages", html, "Confirm", function () {
70+
$('#loadingtext').text("Updating Packages");
71+
$('#loadingview').show();
7072
api.update_packages(packages, project, function () {
71-
scripts.package_view(project)
73+
$('#loadingview').hide();
74+
scripts.package_view(project);
7275
}, function () {
76+
$('#loadingview').hide();
7377
common.display_msg("Server Error", "The selected packages could not be updated. Please try again.");
7478
});
7579
});
@@ -100,9 +104,13 @@ function delete_packages(project) {
100104
html.append("<li>" + element + "</li>");
101105
}
102106
common.confirm("Delete Packages", html, "Confirm", function () {
107+
$('#loadingtext').text("Removing Packages");
108+
$('#loadingview').show();
103109
api.delete_packages(packages, project, function () {
104-
scripts.package_view(project)
110+
$('#loadingview').hide();
111+
scripts.package_view(project);
105112
}, function () {
113+
$('#loadingview').hide();
106114
common.display_msg("Server Error", "The selected packages could not be removed. Please try again.");
107115
});
108116
});
@@ -126,9 +134,13 @@ function install_packages(project) {
126134
html.append("<li>" + element + "</li>");
127135
}
128136
common.confirm("Install Packages", html, "Confirm", function () {
137+
$('#loadingtext').text("Removing Packages");
138+
$('#loadingview').show();
129139
api.install_packages(toInstall, project, function () {
130-
scripts.package_view(project)
140+
$('#loadingview').hide();
141+
scripts.package_view(project);
131142
}, function () {
143+
$('#loadingview').hide();
132144
common.display_msg("Server Error", "The selected packages could not be installed. Please try again.");
133145
});
134146
});

extension/js/scripts.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ function package_view(dir) {
1212
$('#loadingview').show();
1313
$('#installed-packages').empty();
1414
$('#to-install').empty();
15+
$('#loadingtext').text("Loading Environment Information");
1516

1617
api.get_info(dir, function (info) {
1718
var data = info.packages;

extension/js/templates/sidebar.html

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@
1010
</div>
1111
<br>
1212
<div id="loadingview">
13+
<br>
1314
<h2 id="loadingtext">Loading</h2>
15+
<br>
1416
</div>
1517
<div id="packageview">
1618
<div class="main-content">

0 commit comments

Comments
 (0)