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
Add icon #23
Add icon #23
Conversation
README.md
Outdated
@@ -1,3 +1,5 @@ | |||
![asset 1-8](https://user-images.githubusercontent.com/38810169/40872729-06168d0e-6675-11e8-9537-af0a0f133d11.png) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can add NotifyAvailability
instead of asset 1-8
and remove the header below. This by itself looks good enough
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i removed this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was actually a good header image. Can you add this image in the path images/readmeHeader.png
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just update the readme. I would also prefer the icon folder to be moved to a images
folder.
now check |
@@ -0,0 +1 @@ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is unnecessary. I was asking to move the entire icon
folder that you have created in the folder images
.
Also, can you add this image in the path Also, if you have the svg file you've used to create the icon, can you add that? In the future if I need to create some other resolution image for this, that would make it easier. |
No description provided.