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

Fix labels for button bar in contact or group details view. #656

Merged
merged 1 commit into from
Oct 29, 2018

Conversation

MarcoZehe
Copy link
Contributor

Give each button a distinguished label so they state what they do. Fixes #654.

Give each button a distinguished label so they state what they do. Fixes #654.
@dbrgn
Copy link
Contributor

dbrgn commented Oct 26, 2018

Thanks for the pull request! Note, if you are already preparing a pull request, there's no need to create a separate issue for it. The reason why we generally ask contributors to create issues before opening pull requests is that larger features/changes can be discussed before the issue creator spends too much time implementing something we'd like to do differently.

Accessibility fixes are always welcome!

@dbrgn dbrgn added the accessibility Accessibility related label Oct 26, 2018
@MarcoZehe
Copy link
Contributor Author

Thanks, will not create issues for these kinds of fixes in the future, but submit pull requests directly. :)

Copy link
Contributor

@dbrgn dbrgn left a comment

Choose a reason for hiding this comment

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

Thanks!

@dbrgn dbrgn merged commit 7569297 into threema-ch:master Oct 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Accessibility related
Development

Successfully merging this pull request may close these issues.

2 participants