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

6.2.44 Cannot delete a moderator/owner #698

Closed
bohwaz opened this issue Jul 18, 2019 · 7 comments · Fixed by #713
Closed

6.2.44 Cannot delete a moderator/owner #698

bohwaz opened this issue Jul 18, 2019 · 7 comments · Fixed by #713
Milestone

Comments

@bohwaz
Copy link

bohwaz commented Jul 18, 2019

Version

6.2.44

Installation method

RPM

Expected behavior

When checking the box and the clicking the delete icon, it should delete the moderator or owner.

Actual behavior

Nothing happens. Dev console tells me there is no event listener on the icon.

@ikedas
Copy link
Member

ikedas commented Jul 25, 2019

Hi @bohwaz,

After you checked checkboxes, did you click "Update" button below?

@ikedas
Copy link
Member

ikedas commented Jul 25, 2019

This is an example:

issue-698

@bohwaz
Copy link
Author

bohwaz commented Jul 27, 2019

No I clicked the "delete" icon above the checkbox, which changes the cursor to indicate that it is a clickable action ;)

If the "update" button means "delete the checked editors" it probably needs to be renamed :)

@ikedas
Copy link
Member

ikedas commented Jul 27, 2019

“Delete” icon makes checkboxes of all users checked. “Update” button makes deletion (and the other modifications) happen in reality. How may we rename them?

@ldidry
Copy link
Contributor

ldidry commented Jul 28, 2019

"Apply modifications", maybe?

@ikedas
Copy link
Member

ikedas commented Jul 29, 2019

I agree to @ldidry.

@bohwaz
Copy link
Author

bohwaz commented Aug 7, 2019

Thanks that's an improvement, but IMHO the button should also be below the owners list, as it's not straightforward that the button applies to the whole page and not only the "add owner" form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants