Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: includes/contact.html

Issue 29519587: Issue 5534 - Create Contact Section for help.eyeo.com (Closed) Base URL: https://hg.adblockplus.org/help.eyeo.com
Patch Set: Created Aug. 18, 2017, 12:30 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | static/img/png/accent-bg.png » ('j') | static/scss/base/_utilities.scss » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/contact.html
===================================================================
new file mode 100644
--- /dev/null
+++ b/includes/contact.html
@@ -0,0 +1,48 @@
+<section class="section-accent">
+ <div class="wrapper-sm">
+
+ <h2 class="h2 ta-center underlined">
+ {{contact[heading] Still looking for help?}}
juliandoucette 2017/08/30 11:33:06 NIT/Suggest: ID contact-heading (I know... DRY but
ire 2017/09/25 08:47:35 Done.
+ </h2>
+
+ <div class="row">
+ <section class="column one-half section">
juliandoucette 2017/08/30 11:33:06 Note: I think we should be consistent about whethe
ire 2017/09/25 08:47:35 I think it depends on where we want the spacing. I
+ <header class="underlined">
+ <h3 class="h4">
+ {{contact-social[heading] Get in touch}}
+ </h3>
+ <p>
+ {{contact-social-description[paragraph] Drop us a line on Twitter or Facebook and get your question answered ASAP.}}
+ </p>
+ </header>
+ <a href="https://www.facebook.com/adblockplus">
+ <img class="heading-icon" src="/img/png/logo-facebook.png" srcset="/img/svg/logo-facebook.svg 2x" alt="">
ire 2017/08/18 12:34:30 I left a null alt attribute here since the text wi
juliandoucette 2017/08/30 11:33:06 NIT/Suggest: Describe the logo. It's a "rounded sq
ire 2017/09/25 08:47:35 Done. I haven't heard any evidence on the contrary
+ Facebook
ire 2017/08/18 12:34:30 Do we translate this?
juliandoucette 2017/08/30 11:33:06 Good question. I don't think it matters. The only
ire 2017/09/25 08:47:35 I agree with you that translating everything makes
+ </a>
+ <a href="https://twitter.com/AdblockPlus">
+ <img class="heading-icon" src="/img/png/logo-twitter.png" srcset="/img/svg/logo-twitter.svg 2x" alt="">
+ Twitter
+ </a>
+ <a href="mailto:support@adblockplus.org">
+ <img class="heading-icon" src="/img/png/email-icon.png" srcset="/img/svg/email-icon.svg 2x" alt="">
+ Email
+ </a>
+ </section>
+
+ <section class="column one-half section">
+ <header class="underlined">
+ <h3 class="h4">
+ {{contact-developer[heading] Looking for developer support?}}
+ </h3>
+ <p>
+ {{contact-developer-description[paragraph] Find source code, release notes, advanced documentation and more.}}
+ </p>
+ </header>
+ <a href="https://adblockplus.org/forum">
+ {{contact-developer-support-link[link] Developer Support}}
+ </a>
+ </section>
+ </div>
+
+ </div>
+</section>
« no previous file with comments | « no previous file | static/img/png/accent-bg.png » ('j') | static/scss/base/_utilities.scss » ('J')

Powered by Google App Engine
This is Rietveld