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

Use the system icon for sending #91

Merged
merged 2 commits into from Apr 16, 2019

Conversation

@ChristianPauly
Copy link
Collaborator

commented Apr 15, 2019

The current one looks too much like the telegram logo ;)

Use the system icon for sending
The current one looks too much like the telegram logo ;)
@UniversalSuperBox

This comment has been minimized.

Copy link
Member

commented Apr 16, 2019

Is the send icon used after removing this? If not, it can be deleted as well.

@ChristianPauly

This comment has been minimized.

Copy link
Collaborator Author

commented Apr 16, 2019

Is the send icon used after removing this? If not, it can be deleted as well.

Good point.
Here is a screenshot how it looks now:

screenshot20190416_170728783

Merge if you like ;)

@Flohack74 Flohack74 merged commit 2ca5eb7 into xenial Apr 16, 2019

2 checks passed

continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details

@UniversalSuperBox UniversalSuperBox deleted the ChristianPauly-patch-1 branch Apr 18, 2019

@UniversalSuperBox UniversalSuperBox added this to In progress in OTA-9 via automation Apr 18, 2019

@UniversalSuperBox UniversalSuperBox moved this from In progress to QA in OTA-9 Apr 18, 2019

@UniversalSuperBox UniversalSuperBox moved this from QA to Done in OTA-9 Apr 24, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3 participants
You can’t perform that action at this time.