includes/contact.html - Issue 29519587: Issue 5534 - Create Contact Section for help.eyeo.com

Side by Side 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.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
(Empty)
	1 <section class="section-accent">

	2 <div class="wrapper-sm">

	3

	4 <h2 class="h2 ta-center underlined">

	5 {{contact[heading] Still looking for help?}}
	juliandoucette 2017/08/30 11:33:06 NIT/Suggest: ID contact-heading (I know... DRY but NIT/Suggest: ID contact-heading (I know... DRY but this ID is a little misleading?) ire 2017/09/25 08:47:35 Done. Show quoted text On 2017/08/30 11:33:06, juliandoucette wrote: > NIT/Suggest: ID contact-heading (I know... DRY but this ID is a little > misleading?) Done.
	6 </h2>

	7

	8 <div class="row">

	9 <section class="column one-half section">
	juliandoucette 2017/08/30 11:33:06 Note: I think we should be consistent about whethe Note: I think we should be consistent about whether we add .section before/after/inside rows and columns (I'm not saying that the way you did it here is wrong, I'm just thinking out loud. Feedback is welcome.) ire 2017/09/25 08:47:35 I think it depends on where we want the spacing. I Show quoted text On 2017/08/30 11:33:06, juliandoucette wrote: > Note: I think we should be consistent about whether we add .section > before/after/inside rows and columns > > (I'm not saying that the way you did it here is wrong, I'm just thinking out > loud. Feedback is welcome.) I think it depends on where we want the spacing. I don't know that we can have a rule that applies in all cases.
	10 <header class="underlined">

	11 <h3 class="h4">

	12 {{contact-social[heading] Get in touch}}

	13 </h3>

	14 <p>

	15 {{contact-social-description[paragraph] Drop us a line on Twitter or Facebook and get your question answered ASAP.}}

	16 </p>

	17 </header>

	18 <a href="https://www.facebook.com/adblockplus">

	19 <img class="heading-icon" src="/img/png/logo-facebook.png" srcset="/im g/svg/logo-facebook.svg 2x" alt="">
	ire 2017/08/18 12:34:30 I left a null alt attribute here since the text wi I left a null alt attribute here since the text will have essentially the same content juliandoucette 2017/08/30 11:33:06 NIT/Suggest: Describe the logo. It's a "rounded sq Show quoted text On 2017/08/18 12:34:30, ire wrote: > I left a null alt attribute here since the text will have essentially the same > content NIT/Suggest: Describe the logo. It's a "rounded square facebook hieroglyph". Blind people appreciate these details (in my experience working on a website for a group of blind people). Even when they are not functional. This is based on subjective experience; of course. We may be able to find contradictory evidence and/or philosophy. ire 2017/09/25 08:47:35 Done. I haven't heard any evidence on the contrary Show quoted text On 2017/08/30 11:33:06, juliandoucette wrote: > On 2017/08/18 12:34:30, ire wrote: > > I left a null alt attribute here since the text will have essentially the same > > content > > NIT/Suggest: Describe the logo. It's a "rounded square facebook hieroglyph". > Blind people appreciate these details (in my experience working on a website for > a group of blind people). Even when they are not functional. This is based on > subjective experience; of course. We may be able to find contradictory evidence > and/or philosophy. Done. I haven't heard any evidence on the contrary, and your suggestion sounds helpful.
	20 Facebook
	ire 2017/08/18 12:34:30 Do we translate this? Do we translate this? juliandoucette 2017/08/30 11:33:06 Good question. I don't think it matters. The only Show quoted text On 2017/08/18 12:34:30, ire wrote: > Do we translate this? Good question. I don't think it matters. The only thing that matters is whether we are consistent about translating things like it or not. We could translate it by default and rely on the CMS to <fix> it by whitelist. Or we could not translate it. I think it requires less thinking to translate everything. ire 2017/09/25 08:47:35 I agree with you that translating everything makes Show quoted text On 2017/08/30 11:33:06, juliandoucette wrote: > On 2017/08/18 12:34:30, ire wrote: > > Do we translate this? > > Good question. I don't think it matters. The only thing that matters is whether > we are consistent about translating things like it or not. We could translate it > by default and rely on the CMS to <fix> it by whitelist. Or we could not > translate it. I think it requires less thinking to translate everything. I agree with you that translating everything makes sense. Done.
	21 </a>

	22 <a href="https://twitter.com/AdblockPlus">

	23 <img class="heading-icon" src="/img/png/logo-twitter.png" srcset="/img /svg/logo-twitter.svg 2x" alt="">

	24 Twitter

	25 </a>

	26 <a href="mailto:support@adblockplus.org">

	27 <img class="heading-icon" src="/img/png/email-icon.png" srcset="/img/s vg/email-icon.svg 2x" alt="">

	28 Email

	29 </a>

	30 </section>

	31

	32 <section class="column one-half section">

	33 <header class="underlined">

	34 <h3 class="h4">

	35 {{contact-developer[heading] Looking for developer support?}}

	36 </h3>

	37 <p>

	38 {{contact-developer-description[paragraph] Find source code, release notes, advanced documentation and more.}}

	39 </p>

	40 </header>

	41 <a href="https://adblockplus.org/forum">

	42 {{contact-developer-support-link[link] Developer Support}}

	43 </a>

	44 </section>

	45 </div>

	46

	47 </div>

	48 </section>

OLD	NEW

« no previous file with comments | « no previous file | static/img/png/accent-bg.png » ('j') | static/scss/base/_utilities.scss » ('J')