-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
[IMPROVE] Block user button #13627
Comments
May I do this? |
@bharadwajaditya sorry, I already have made changes to my local branch, just need to push. Thanks again |
You changed the logo in the for loop which will change for all the actions. I corrected that by changing only for block user. |
@bhardwajaditya Thanks, you are right. But please remember this, when a PR is opened on an issue, you should not open a PR yourself. You would have suggested changes to the old PR. Hope this will help you in the long run. Cheers 👍 |
Sorry, I am new to open source contributions. Will keep this in mind in future. Thanks for your help. |
Description:
This is UI/UX related bug. Block User button is a
microphone
right now, which should not be as far as I think. It would be better if we change the icon.Steps to reproduce:
Expected behavior:
Actual behavior:
Server Setup Information:
Additional context
I will fix this.
The text was updated successfully, but these errors were encountered: