diff --git a/gui/app/components/folder/permission-admin.js b/gui/app/components/folder/permission-admin.js index b34f1544..1c0a3c5e 100644 --- a/gui/app/components/folder/permission-admin.js +++ b/gui/app/components/folder/permission-admin.js @@ -109,11 +109,6 @@ export default Component.extend(NotifierMixin, { } }); - this.get('folderService').savePermissions(folder.get('id'), payload).then(() => { - this.showNotification('Saved permissions'); - this.get('router').transitionTo('folder', folder.get('id'), folder.get('slug')); - }); - if (is.not.undefined(hasEveryone)) { folder.markAsPublic(); this.showNotification('Marked space as public'); @@ -126,6 +121,12 @@ export default Component.extend(NotifierMixin, { this.showNotification('Marked space as private'); } } + + this.get('folderService').savePermissions(folder.get('id'), payload).then(() => { + this.showNotification('Saved permissions'); + $('#space-permission-modal').modal('hide'); + $('#space-permission-modal').modal('dispose'); + }); } } }); diff --git a/gui/app/components/ui/ui-dialog.js b/gui/app/components/ui/ui-dialog.js index 24519fcc..c9a83c22 100644 --- a/gui/app/components/ui/ui-dialog.js +++ b/gui/app/components/ui/ui-dialog.js @@ -19,6 +19,7 @@ export default Component.extend({ title: 'Confirm', show: false, buttonType: 'btn-secondary', + size: '', // modal-lg, modal-sm didInsertElement() { this._super(...arguments); diff --git a/gui/app/styles/bootstrap.scss b/gui/app/styles/bootstrap.scss index 015ec751..dab4698c 100644 --- a/gui/app/styles/bootstrap.scss +++ b/gui/app/styles/bootstrap.scss @@ -98,5 +98,11 @@ $link-hover-decoration: none; @import "node_modules/bootstrap/scss/utilities"; @import "node_modules/bootstrap/scss/popover"; @import "node_modules/bootstrap/scss/tooltip"; -// @import "node_modules/bootstrap/scss/images"; +@import "node_modules/bootstrap/scss/tables"; // @import "node_modules/bootstrap/scss/navbar"; +// @import "node_modules/bootstrap/scss/images"; + + +.modal-80 { + max-width: 80% !important; +} diff --git a/gui/app/styles/view/folder/settings.scss b/gui/app/styles/view/folder/settings.scss index b1c31af3..651ce3f0 100644 --- a/gui/app/styles/view/folder/settings.scss +++ b/gui/app/styles/view/folder/settings.scss @@ -1,48 +1,4 @@ .space-settings { - > .panel { - @include content-container(); - @include ease-in(); - @extend .transition-all; - margin: 0 0 30px 0; - } - - .permissions-table { - padding: 0; - margin: 0 auto; - width: 90%; - - > .row { - padding: 8px 0; - - > .permission-name-cell { - font-size: 1.1rem; - background-color: $color-off-white; - padding: 10px 10px; - } - - > .permission-roles-cell { - margin: 15px 0 20px 40px; - - > .role-container { - white-space: nowrap; - display: inline-block; - margin-bottom: 10px; - - > .role-category { - color: $color-gray; - font-weight: bold; - display: inline-block; - } - } - - > label { - color: $color-gray; - font-weight: normal; - } - } - } - } - .category-table { padding: 0; margin: 0 0 0 20px; diff --git a/gui/app/styles/view/space.scss b/gui/app/styles/view/space.scss index 0916d244..7512ab60 100644 --- a/gui/app/styles/view/space.scss +++ b/gui/app/styles/view/space.scss @@ -118,4 +118,37 @@ } } +.space-admin { + > .permission-table { + // width: 100%; + // overflow-x: auto; + > tbody, > thead { + > tr, > th { + > td, > th { + margin: 0; + padding: 10px 15px; + text-align: center; + } + + > td:first-child { + text-align: left; + } + } + } + + > thead { + > tr { + > th { + background-color: $color-off-white; + color: $color-gray; + } + + > th:first-child { + background-color: $color-white !important; + border: none !important; + } + } + } + } +} diff --git a/gui/app/templates/components/folder/permission-admin.hbs b/gui/app/templates/components/folder/permission-admin.hbs index 48e84392..474b535d 100644 --- a/gui/app/templates/components/folder/permission-admin.hbs +++ b/gui/app/templates/components/folder/permission-admin.hbs @@ -1,49 +1,71 @@ -