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.

Sign up for GitHub

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

Jump to bottom

Quarantine moved to the menu #2276

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Quarantine moved to the menu #2276

wants to merge 9 commits into from
+11 −28

Conversation

br4n-d0n
Copy link

  • Moved Quarantine for better accessibility because it was hidden under Settings
  • Fixed some icons and rearranged to accommodate for Quarantine

Quarantine moved to the menu
ac04332 
- Moved Quarantine for better accessibility because it was hidden under Settings
- Fixed some icons and rearranged to accommodate for Quarantine
@br4n-d0n
Copy link
Author

I'm guessing it's quite busy and there are other more important things to address, but I wanted to know if @nguyenkims, @acasajus, or @cquintana92 could at least take a look at this considering the minimal amount of changes to the files.

@nguyenkims
Copy link
Contributor

As quarantine is a rare action (very few users have bounced emails), I think the menu might not be the best place for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acasajus acasajus Awaiting requested review from acasajus acasajus is a code owner

@cquintana92 cquintana92 Awaiting requested review from cquintana92 cquintana92 is a code owner

@nguyenkims nguyenkims Awaiting requested review from nguyenkims nguyenkims is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@br4n-d0n @nguyenkims