Skip to content
This repository was archived by the owner on Jan 8, 2019. It is now read-only.

feat: add mail contact #51

Merged
merged 2 commits into from
Dec 23, 2017
Merged

feat: add mail contact #51

merged 2 commits into from
Dec 23, 2017

Conversation

tilmx
Copy link
Member

@tilmx tilmx commented Dec 22, 2017

No description provided.

@marionebl
Copy link
Contributor

marionebl commented Dec 22, 2017

Deploy preview processing.

Built with commit d36b306

https://app.netlify.com/sites/determined-cray-3fd905/deploys/5a3e96d7df99534d32afb56b

lib/index.html Outdated
@@ -154,7 +154,8 @@ <h3>Prototype design with live data</h3>
<h2>Be part of shaping the future of design</h2>
<p>Help us taking Alva to the next level with your design expertise, product management insights or engineering skills. Our next features, bugs and the roadmap are fully open. Feel free to add issues, start pull requests or let us discuss what should be next.</p>

<a class="button" href="https://github.com/meetalva/alva/projects/3" target="_blank">Explore our next steps on Github</a>
<a class="button" href="https://github.com/meetalva/alva/projects/3" target="_blank">Explore our next steps on Github</a><br>
<a class="contribute-mail" href="mailto:[email protected]">or send us an Email</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

A title attribute would be nice

lib/index.html Outdated
@@ -206,6 +207,7 @@ <h4>Tilman Frick</h4>
</section>

<section class="section footer">
<a href="mailto:[email protected]">[email protected]</a><br><br>
Copy link
Contributor

Choose a reason for hiding this comment

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

A title attribute would be nice

@tilmx
Copy link
Member Author

tilmx commented Dec 23, 2017

@marionebl I just added a title tag to the mail links

@tilmx tilmx merged commit 6b70f99 into master Dec 23, 2017
@tilmx tilmx deleted the feat/add-mail branch December 23, 2017 18:08
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.

2 participants