Skip to content

Commit

Permalink
styling notification (#225)
Browse files Browse the repository at this point in the history
* styling notification

making network create non-blocking

adding error notifications

error notification

styling

Correction Animation

removing comments

regenerating bundle

* fixing icons

* comments fixed

comments fixed

regenerating bundle

* network delete fix
  • Loading branch information
shampur authored and vishal-j committed Dec 9, 2016
1 parent f64f0e8 commit b66bbca
Show file tree
Hide file tree
Showing 55 changed files with 1,633 additions and 1,563 deletions.
4 changes: 0 additions & 4 deletions app/applicationgroups/applicationgroupcreate.html
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,5 @@
</div>
</div>
</div>

<ctv-error *ngIf="showServerError" header="Error creating application group"
[error]="serverErrorMessage">
</ctv-error>
</div>
</form>
6 changes: 2 additions & 4 deletions app/applicationgroups/applicationgroupcreatectrl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ export class ApplicationGroupCreateComponent {

function resetForm() {
crudHelperService.stopLoader(applicationGroupCreateCtrl);
crudHelperService.hideServerError(applicationGroupCreateCtrl);
applicationGroupCreateCtrl.applicationGroup = {
groupName: '', // For Group Name
networkName: '', // For Network Name
Expand All @@ -61,7 +60,6 @@ export class ApplicationGroupCreateComponent {
createApplicationGroup(validform: boolean) {
var applicationGroupCreateCtrl = this;
if (validform) {
applicationGroupCreateCtrl.crudHelperService.hideServerError(applicationGroupCreateCtrl);
applicationGroupCreateCtrl.crudHelperService.startLoader(applicationGroupCreateCtrl);
applicationGroupCreateCtrl.applicationGroup.networkName =
applicationGroupCreateCtrl.selectedNetwork;
Expand All @@ -76,11 +74,11 @@ export class ApplicationGroupCreateComponent {
applicationGroupCreateCtrl.applicationGroupsModel.create(applicationGroupCreateCtrl.applicationGroup,undefined).then(
function successCallback(result) {
applicationGroupCreateCtrl.crudHelperService.stopLoader(applicationGroupCreateCtrl);
applicationGroupCreateCtrl.crudHelperService.showNotification("Application Group Created", result.key.toString());
applicationGroupCreateCtrl.crudHelperService.showNotification("Application group: Created", result.key.toString());
applicationGroupCreateCtrl.returnToApplicationGroup();
}, function errorCallback(result) {
applicationGroupCreateCtrl.crudHelperService.stopLoader(applicationGroupCreateCtrl);
applicationGroupCreateCtrl.crudHelperService.showServerError(applicationGroupCreateCtrl, result);
applicationGroupCreateCtrl.crudHelperService.showServerError("Application group: Create failed", result);
});
}
}
Expand Down
8 changes: 0 additions & 8 deletions app/applicationgroups/applicationgroupdetails.html
Original file line number Diff line number Diff line change
Expand Up @@ -66,14 +66,6 @@
</button>
</div>
</div>
<ctv-error *ngIf="((showServerError) && (mode === 'edit'))"
header="Error saving application group"
[error]="serverErrorMessage">
</ctv-error>
<ctv-error *ngIf="((showServerError) && (mode === 'details'))"
header="Error deleting application group"
[error]="serverErrorMessage">
</ctv-error>
</div>
</div>

Expand Down
12 changes: 5 additions & 7 deletions app/applicationgroups/applicationgroupdetailsctrl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ export class ApplicationGroupDetailsComponent implements OnInit{
}

applicationGroupDetailsCtrl.crudHelperService.startLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.hideServerError(applicationGroupDetailsCtrl);

applicationGroupDetailsCtrl.applicationGroupsModel.getModelByKey(activatedRoute.snapshot.params['key'], false, 'key')
.then(function (group) {
Expand Down Expand Up @@ -82,32 +81,31 @@ export class ApplicationGroupDetailsComponent implements OnInit{

deleteApplicationGroup() {
var applicationGroupDetailsCtrl = this;
applicationGroupDetailsCtrl.crudHelperService.hideServerError(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.startLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.applicationGroupsModel.delete(applicationGroupDetailsCtrl.applicationGroup).then(
function successCallback(result) {
applicationGroupDetailsCtrl.crudHelperService.stopLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.showNotification("Application Group Deleted", result.toString());
applicationGroupDetailsCtrl.crudHelperService.showNotification("Application group: Deleted", result.toString());
applicationGroupDetailsCtrl.returnToApplicationGroup();
}, function errorCallback(result) {
applicationGroupDetailsCtrl.crudHelperService.stopLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.showServerError(applicationGroupDetailsCtrl, result);
applicationGroupDetailsCtrl.crudHelperService.showServerError("Application group: Delete failed", result);
});
}

saveApplicationGroup() {
var applicationGroupDetailsCtrl = this;
applicationGroupDetailsCtrl.crudHelperService.hideServerError(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.startLoader(applicationGroupDetailsCtrl);

applicationGroupDetailsCtrl.applicationGroupsModel.save(applicationGroupDetailsCtrl.applicationGroup).then(
function successCallback(result) {
applicationGroupDetailsCtrl.crudHelperService.stopLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.showNotification("Application Group Updated", result.key.toString());
applicationGroupDetailsCtrl.crudHelperService.showNotification("Application group: Updated", result.key.toString());
applicationGroupDetailsCtrl.returnToApplicationGroupDetails();
}, function errorCallback(result) {
applicationGroupDetailsCtrl.crudHelperService.stopLoader(applicationGroupDetailsCtrl);
applicationGroupDetailsCtrl.crudHelperService.showServerError(applicationGroupDetailsCtrl, result);
applicationGroupDetailsCtrl.crudHelperService.showServerError("Application group: Update failed", result);

});
}
}
6 changes: 2 additions & 4 deletions app/appprofiles/appprofilecreate.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ export class AppProfileCreateComponent {

function resetForm() {
crudHelperService.stopLoader(component);
crudHelperService.hideServerError(component);
component.newAppProfile = {
key: '',
appProfileName: '',
Expand All @@ -45,20 +44,19 @@ export class AppProfileCreateComponent {
var component = this;
if (formvalid) {
this.crudHelperService.startLoader(this);
this.crudHelperService.hideServerError(this);
component.newAppProfile.key = this.appProfilesModel.generateKey(this.newAppProfile);
this.appProfilesModel.create(component.newAppProfile, undefined)
.then((result) => {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
component.crudHelperService.showNotification("Application Profile Created", result.key.toString());
component.crudHelperService.showNotification("Application profile: Created", result.key.toString());
});
component.returnToAppProfiles();
}, (error) => {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
});
component.crudHelperService.showServerError(component, error);
component.crudHelperService.showServerError("Application profile: Create failed", error);
});
}
}
Expand Down
3 changes: 0 additions & 3 deletions app/appprofiles/appprofilecreate.html
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,5 @@
</div>
</div>
</div>

<ctv-error *ngIf="showServerError" header="Error creating application profile" [error]="serverErrorMessage">
</ctv-error>
</div>
</form>
11 changes: 4 additions & 7 deletions app/appprofiles/appprofiledetails.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ export class AppProfileDetailsComponent {
}

component.crudHelperService.stopLoader(component);
component.crudHelperService.hideServerError(component);

component.appProfilesModel.getModelByKey(activatedRoute.snapshot.params['key'], false, 'key')
.then(function (appProfile) {
Expand Down Expand Up @@ -58,41 +57,39 @@ export class AppProfileDetailsComponent {

deleteAppProfile() {
var component = this;
component.crudHelperService.hideServerError(component);
component.crudHelperService.startLoader(component);
component.appProfilesModel.delete(component.appProfile).then(
function successCallback(result) {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
component.crudHelperService.showNotification("Application Profile Deleted", result);
component.crudHelperService.showNotification("Application profile: Deleted", result);
});
component.returnToAppProfile();
}, function errorCallback(result) {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
});
component.crudHelperService.showServerError(component, result);
component.crudHelperService.showServerError("Application profile: Delete failed", result);
});
}

saveAppProfile(formvalid: boolean) {
var component = this;
if (formvalid) {
component.crudHelperService.hideServerError(component);
component.crudHelperService.startLoader(component);

component.appProfilesModel.save(component.appProfile).then(
function successCallback(result) {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
component.crudHelperService.showNotification("Application Profile Updated", result.key.toString());
component.crudHelperService.showNotification("Application profile: Updated", result.key.toString());
});
component.returnToAppProfileDetails();
}, function errorCallback(result) {
component.ngZone.run(() => {
component.crudHelperService.stopLoader(component);
});
component.crudHelperService.showServerError(component, result);
component.crudHelperService.showServerError("Application profile: Update failed", result);
});
}
}
Expand Down
9 changes: 1 addition & 8 deletions app/appprofiles/appprofiledetails.html
Original file line number Diff line number Diff line change
Expand Up @@ -76,16 +76,9 @@ <h4 class="ui header" style="margin-top: 40px">Application Groups</h4>
</button>
</div>
</div>
<ctv-error *ngIf="((showServerError) && (mode === 'edit'))"
header="Error saving application profile"
[error]="serverErrorMessage">
</ctv-error>
</form>
<ctv-error *ngIf="((showServerError) && (mode === 'details'))"
header="Error deleting application profile"
[error]="serverErrorMessage">
</ctv-error>
</div>

</div>

</div>
20 changes: 15 additions & 5 deletions app/bower_components/semantic-ui/dist/components/icon.css
Original file line number Diff line number Diff line change
Expand Up @@ -507,12 +507,16 @@ i.icons .inverted.corner.icon {
}
.icon.info .path2:before {
content: "\e93f";
margin-left: -1em;
position: absolute;
right: 3px;
top: 3px;
color: #ffffff;
}
.icon.info .path3:before {
content: "\e940";
margin-left: -1em;
position: absolute;
top: 3px;
right: 3px;
color: #ffffff;
}
.icon.major .path1:before {
Expand All @@ -535,12 +539,16 @@ i.icons .inverted.corner.icon {
}
.icon.minor .path2:before {
content: "\e945";
margin-left: -1em;
position: absolute;
right: 4px;
top: 4px;
color: #ffffff;
}
.icon.minor .path3:before {
content: "\e946";
margin-left: -1em;
position: absolute;
top: 4px;
right: 4px;
color: #ffffff;
}
.icon.pending .path1:before {
Expand Down Expand Up @@ -568,7 +576,9 @@ i.icons .inverted.corner.icon {
}
.icon.ready .path2:before {
content: "\e94c";
margin-left: -1em;
position: absolute;
right: 3px;
top: 4px;
color: #ffffff;
}
.icon.soaking:before {
Expand Down
Loading

0 comments on commit b66bbca

Please sign in to comment.