Skip to content
This repository was archived by the owner on Nov 23, 2018. It is now read-only.

Add more touch icons and description where it use#232

Closed
frontenddeveloping wants to merge 2 commits intoh5bp:masterfrom
frontenddeveloping:issue-227_apple-touch-icons
Closed

Add more touch icons and description where it use#232
frontenddeveloping wants to merge 2 commits intoh5bp:masterfrom
frontenddeveloping:issue-227_apple-touch-icons

Conversation

@frontenddeveloping
Copy link
Contributor

Add more touch icons and compress new and some old in https://tinypng.com/
For information was used famous article about touch icons https://mathiasbynens.be/notes/touch-icons

@frontenddeveloping frontenddeveloping changed the title issue 227: add more touch icons and description where it use add more touch icons and description where it use Oct 14, 2014
@frontenddeveloping frontenddeveloping changed the title add more touch icons and description where it use Add more touch icons and description where it use Oct 14, 2014
index.html Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@arthurvr
Copy link
Member

Thanks for the PR!

I do not really like the all the comments in the code, and would like to replace this with the docs. Thoughts?

@frontenddeveloping
Copy link
Contributor Author

I think it will be better to left comments in code, who reads docs? :)
Not all know all about this icons. It will be easy to understand which icons I must remove if I male iOS standalone app for iOS7+.

@arthurvr arthurvr closed this in 38e1a6e Oct 15, 2014
@arthurvr
Copy link
Member

Merged! Thanks @frontenddeveloping

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants