-
Notifications
You must be signed in to change notification settings - Fork 69
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
Initial ability to view post/comment reports #1210
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, this looks so awesome! I don't really have anything to add to either the code or the UX (without really trying it in-depth myself).
One thing that did come to mind is the filter button. I wonder if we should change the icon slightly when there's a filter applied. The same could be for the modlog.
I think this looks great! And I like your ideas for future additions :) honestly I think my only thought would be to better highlight the reason something is reported - feels like an element that would get a custom color or pill treatment, I think? |
I think that's great! Looks very clean :) |
I'll go ahead and merge this in for the initial feature, and we can continue to add on to this as I mentioned in the original description 😄 |
Pull Request Description
This PR adds the ability to view post/comment reports. If a user is a moderator for a community, or an admin, a new option will show up on the drawer to open a page containing post and comment reports.
This is the first iteration of this, but I have some ideas for additions to this!
@micahmo feel free to take a look through the code, but no need to do a deep dive into it as a lot of new files have been added to support reports
@machinaeZER0 it would be great if you could provide any feedback on the overall UI/UX! I feel like I'm fairly happy with it. The only thing bothering me right now is the contents within the filter bottom sheet (it feels a bit awkward but that might just be me)
Issue Being Fixed
Issue Number: N/A
Screenshots / Recordings
03cd7a4b-8d05-4d87-bc74-5e286c82e935.mp4
Adding in some static screenshots to get a better view of the UI
Checklist
semanticLabel
s where applicable for accessibility?