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

Moderation feature bugs #2369

Closed
2 tasks done
VolkerJunginger opened this issue Mar 22, 2024 · 4 comments
Closed
2 tasks done

Moderation feature bugs #2369

VolkerJunginger opened this issue Mar 22, 2024 · 4 comments

Comments

@VolkerJunginger
Copy link
Contributor

VolkerJunginger commented Mar 22, 2024

Your use case

The user-screen in the moderation

  • Always show all users in the edit admin list

    • iOS does not show yourself
  • Pending members in edit admin & moderation list

    • Android shows pending members; iOS does not
    • It must be possible to change the role of a pending member
  • Show role of user & show "not selectable" checkbox

    • Show sections for the users role: Admin, Moderator, Member
    • Decorate the pending members: USERNAME (Pending)
    • Show a checkbox that is not selectable
      @amshakal
  • iOS Use the same string for search in all memberlists.

    • In the memberlist seem to be another string that in the moderation or invite flow: Search for someone
  • iOS Memberlist:

    • As an admin I should not be able to ban other admins
    • I should not be able to ban myself

Android:Roles & Permissions

  • lines for headlines are missing

Have you considered any alternatives?

No response

Additional context

No response

Android

  1. T-Task
    jmartinesp

iOS

@jmartinesp
Copy link

@VolkerJunginger just to confirm, on Android should we do anything else to the separators in the 'change permissions for actions' screens? There are separators between the different sections, and the headlines are bolder (medium vs regular) than the other options.

We could change the headlines in the Compound components to be slightly larger or make then bold if they don't look different enough from the options.

Screenshot

@amshakal
Copy link

amshakal commented Apr 8, 2024

I dont think we should change our list component, we are using material standard components and styles and to a regular android user, thse should feel fine. I had a chat with Aaron and we can monitor if people complain about this list item in the future and iterate on the component accordingly.

@jmartinesp
Copy link

Should we close this now?

@amshakal
Copy link

Yes

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

No branches or pull requests

5 participants