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

implemented mod widget customization #1031

Merged
merged 4 commits into from
Apr 21, 2024
Merged

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:
    - closes Mod Mail Reports Widget Customization #1028

Pull Request Information

This PR implements mod widget customization.

@EricBAndrews EricBAndrews requested a review from a team as a code owner April 21, 2024 16:31
@EricBAndrews EricBAndrews requested review from WestonHanners, mormaer and Sjmarf and removed request for a team April 21, 2024 16:31
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.

The "Reset" button in the widget customization page sets it to the default post layout.

(Also, my SwiftLint doesn't like the var x = switch y { syntax again... I really need to look into that)

@EricBAndrews EricBAndrews requested a review from Sjmarf April 21, 2024 17: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.

One design question, otherwise LGTM!

@EricBAndrews EricBAndrews force-pushed the eric/mod-widget-customization branch from 6b3d470 to 5d75bfb Compare April 21, 2024 21:03
@EricBAndrews EricBAndrews merged commit cfb9be0 into dev Apr 21, 2024
4 checks passed
@EricBAndrews EricBAndrews deleted the eric/mod-widget-customization branch April 21, 2024 21:29
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.

Mod Mail Reports Widget Customization
2 participants