Made view on Github into button

[GiacomoLaw]

I wasn't sure about the button color, so I made it black. Please ask if you want me to change something else.

You can view a demo of it here 😄

[msikma]

Bit unexpected but this is cool 😄

There's one major issue that prevents me from merging this: the button CSS was added to pokesprite.min.css - this file gets automatically generated (along with every other file inside build/), so if I ever made an update it would delete your button CSS. Since it's just a few lines of code, you can just include it as an inline CSS block in the header.

That's easily fixed, other than that I have the following suggestions:

• Move the button so that it's underneath the main image, since if it's centered at the bottom it looks a bit distant from the rest of the content.
• Move the overview.min.css file so that it's loaded before the <script> tag. This is my mistake, all CSS should be above the Javascript.
• Some CSS suggestions: set the button's font-size to 14px, line-height to 21px. Set the button's vertical-align to middle. Set the icon's font-size to 21px, margin-right to 4px, vertical-align to top.
• Modify the code so that the indentation is correct. (Maybe you use tabs. It should be spaces.)

I'll merge this if you are interested in changing these things.

[GiacomoLaw]

Okay, no problem. Will do soon!

[GiacomoLaw]

• [x] Font size - button
• [x] Line height - button

• [x] Vertical align - button

• [x] FA icon size - FA button

• [x] Margin right - FA button

• [x] Move button

• [x] Move css over js

[GiacomoLaw]

@msikma Done! 😃

[GiacomoLaw]

Updated. Is that fine :D

[msikma]

There's a few more changes I would like. But I'll just merge this later and then show you what I meant...

Hope you didn't mind all my comments too much. 😄

[GiacomoLaw]

No, it's fine! Sorry about that.

[msikma]

Merged it. Thanks. Made a few more minor changes: https://github.com/msikma/pokesprite/commit/7c3bd55d3a2fbbe6007334842021ffba233964c0

[GiacomoLaw]

No problem 😄