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

Eric/moderation color #1006

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Eric/moderation color #1006

merged 3 commits into from
Apr 12, 2024

Conversation

EricBAndrews
Copy link
Member

Checklist

  • I have read CONTRIBUTING.md
  • I have described what this PR contains
  • If this PR alters the UI, I have attached pictures/videos
  • This PR addresses one or more open issues that were assigned to me:
    - Slack

Pull Request Information

This PR standardizes moderation color following #1001. It changes the color from .green to Colors.moderation for:

  • Moderation feed header
  • Community pin icon
  • Moderator flair
  • Moderation settings

It leaves moderation actions that are positive/negative (e.g., remove/restore) as red/green.

@EricBAndrews EricBAndrews requested a review from a team as a code owner April 12, 2024 16:41
@EricBAndrews EricBAndrews requested review from WestonHanners, JakeShirley and Sjmarf and removed request for a team April 12, 2024 16:41
@EricBAndrews EricBAndrews enabled auto-merge (squash) April 12, 2024 16:52
Copy link
Member

@Sjmarf Sjmarf left a comment

Choose a reason for hiding this comment

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

Modlog appoint icon

IMG_0387

@EricBAndrews EricBAndrews requested a review from Sjmarf April 12, 2024 19:00
Copy link
Member

@Sjmarf Sjmarf left a comment

Choose a reason for hiding this comment

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

Nice 👍

@EricBAndrews EricBAndrews merged commit adf21df into dev Apr 12, 2024
4 checks passed
@EricBAndrews EricBAndrews deleted the eric/moderation-color branch April 12, 2024 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants