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
update homepage, add html template #662
Conversation
I edited the copy and layout on the home page a little in addition to adding the html template. I found all the |
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.
Good changes 🥇
<script async defer src='https://api.mapbox.com/mapbox-assembly/v0.2.0/assembly.js'></script> | ||
</head> | ||
<body> | ||
<!-- page content goes here --> |
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.
How about Your page ...
<title><!-- Your title goes here --></title> | ||
<meta charset='utf-8'> | ||
<meta name='viewport' content='width=device-width, initial-scale=1'> | ||
<link rel='shortcut icon' href='/path/to/favicon' type='image/x-icon'> |
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.
Should we say href='Your favicon path goes here'
to make it more clear that this is another fill-in-the-black situation?
@@ -146,7 +164,7 @@ class Home extends React.Component { | |||
For dynamic styles, responsive classes (described above) provide some additional flexibility. | |||
But if you need even more, you should use custom CSS instead of a utility class. | |||
</p> | |||
<h3 className='mt24 txt-bold'> | |||
<h3 className='mt48 txt-bold'> | |||
<code className='txt-code'>is-active</code> applies active states |
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.
Did you intend to remove the code from this? Aren't you disoriented?
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 couldn't think of a non-awkward way to re-write this one!!!
No description provided.