Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix adding additional admins #77

Merged
merged 1 commit into from
Apr 2, 2019

Conversation

georgehrke
Copy link
Member

Before, clicking the plus in the admin list had no effect. (not sure what caused this, v-click-outside should not be triggered since it's inside an element with v-if)

This pull-request makes sure that v-click-outside is not triggered by the click on the plus button.

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested & works :)

src/Admins.vue Outdated
@@ -47,6 +49,11 @@ export default {
isAdding: false,
isSavingChanges: false
}),
computed: {
additionalAdminPlaceholderLabel() {
return t('privacy', 'Add outside admin')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Add external admin/person" might sound a bit better but this is just a nitpick

src/Admins.vue Show resolved Hide resolved
@rullzer rullzer mentioned this pull request Mar 26, 2019
9 tasks
@georgehrke
Copy link
Member Author

let me quickly rebase and update the string as you suggested.

Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke georgehrke force-pushed the bugfix/noid/fix_adding_additional_admins branch from 3670985 to 538b344 Compare March 27, 2019 10:28
@ChristophWurst ChristophWurst merged commit 6055ea5 into master Apr 2, 2019
@ChristophWurst ChristophWurst deleted the bugfix/noid/fix_adding_additional_admins branch April 2, 2019 06:18
@MorrisJobke MorrisJobke mentioned this pull request Apr 3, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants