Opened on 10/10/2016 at 04:59:29 PM
Closed on 07/27/2017 at 05:11:27 PM
#4514 closed defect (fixed)
Font readability on acceptableads.com
| Reported by: | juliandoucette | Assignee: | ire |
|---|---|---|---|
| Priority: | P2 | Milestone: | |
| Module: | Websites | Keywords: | |
| Cc: | christiane, p.pastourmatzis, martin, saroyanm, jeen, athornburgh, ire | Blocked By: | |
| Blocking: | Platform: | Unknown / Cross platform | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): | |||
Description (last modified by juliandoucette)
Environment
- Desktop web browsers
How to reproduce
- See any page containing several consecutive paragraphs e.g. users
Observed behaviour
I find it hard to read main text on acceptableads.com. My eyes strain and I make more mistakes than usual.
I think this is because of:
- font family
- font weight
- line height
Expected behaviour
Increase:
- Body text line height increased from 1.2 to 1.5
- Button font size increased from 12px to 16px
- Button height increased to 44px (text inside to be centrally aligned)
- Footer text to be min 14px (It's12 px currently)
Attachments (3)
Change History (29)
Changed on 10/10/2016 at 04:59:41 PM by juliandoucette
comment:3 Changed on 10/26/2016 at 09:43:46 AM by juliandoucette
- Cc jeen added
comment:4 Changed on 10/26/2016 at 10:52:04 AM by jeen
The font family has been chosen for its wide language support, is extremely versatile for the range of weights it offers and renders well at small sizes.
I think simple increasing the line-height of the text (both body text and title text) from 120% to 150% would improve readability a lot.
@christiane: would you be alright with this change?
comment:5 Changed on 11/09/2016 at 11:44:57 AM by juliandoucette
From @christiane:
Up all font styles by 2px. Up h1 by 4px and increase line height to 150%
Changed on 11/09/2016 at 11:45:31 AM by juliandoucette
comment:6 Changed on 11/09/2016 at 11:45:51 AM by juliandoucette
- Priority changed from Unknown to P2
- Ready set
comment:7 Changed on 02/04/2017 at 04:58:55 PM by juliandoucette
- Owner set to juliandoucette
comment:8 Changed on 02/04/2017 at 05:01:06 PM by juliandoucette
- Cc athornburgh added
comment:9 Changed on 02/04/2017 at 05:01:27 PM by juliandoucette
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:10 Changed on 02/16/2017 at 02:26:40 PM by juliandoucette
- Priority changed from P2 to P3
- Ready unset
@saroyanm I'm reducing the priority of this change based on a discussion with @athornburgh and because I think that the solution that is in review right now needs some more work.
Changed on 02/16/2017 at 02:27:40 PM by juliandoucette
Patch from code review 29374562.
comment:11 Changed on 02/16/2017 at 02:30:22 PM by juliandoucette
- Review URL(s) modified (diff)
I think we should also take another look at the spacing in sections within the scope of this issue e.g. the accent section on the users page.
comment:12 Changed on 02/16/2017 at 02:30:36 PM by juliandoucette
- Status changed from reviewing to reopened
comment:13 Changed on 04/13/2017 at 11:41:04 AM by jeen
Following up on this. I would like the following updates made (not sure if this can all go in one ticket):
- Body text line height increased from 120% to 150%
- Button font size increased from 12px to 16px
- Button height increased to 44px (text inside to be centrally aligned)
- Footer text to be min 14px (It's12 px currently)
comment:14 Changed on 05/31/2017 at 02:33:24 PM by juliandoucette
- Owner juliandoucette deleted
comment:15 Changed on 05/31/2017 at 02:33:57 PM by juliandoucette
- Priority changed from P3 to P2
comment:16 Changed on 06/20/2017 at 08:32:50 AM by ire
- Cc iaderinokun added
I think the suggestions made by @jeen here work.
@juliandoucette Is there more you would like to do?
comment:17 Changed on 07/10/2017 at 06:36:29 PM by juliandoucette
@ire I think what Jeen suggested will work. If not, let's address it in another issue.
comment:18 Changed on 07/11/2017 at 06:06:47 AM by christiane
where exactly is the problem here?
comment:19 Changed on 07/11/2017 at 11:19:22 AM by juliandoucette
- Description modified (diff)
where exactly is the problem here?
I've updated the description. Sorry for the confusion christiane. You suggested a solution to this problem a long time ago but it hasn't been implemented yet because of other priorities.
comment:20 Changed on 07/13/2017 at 08:06:59 AM by ire
- Cc ire added; iaderinokun removed
comment:21 follow-up: ↓ 22 Changed on 07/24/2017 at 08:42:44 AM by ire
@juliandoucette: @jlow: Can we begin implementing this?
comment:22 in reply to: ↑ 21 Changed on 07/24/2017 at 09:49:27 AM by jeen
comment:23 Changed on 07/24/2017 at 01:12:31 PM by ire
- Owner set to ire
- Ready set
comment:24 Changed on 07/25/2017 at 01:38:11 PM by ire
- Review URL(s) modified (diff)
- Status changed from reopened to reviewing
comment:25 Changed on 07/27/2017 at 05:10:38 PM by abpbot
A commit referencing this issue has landed:
Issue 4514 - Font readability on acceptableads.com
comment:26 Changed on 07/27/2017 at 05:11:27 PM by ire
- Resolution set to fixed
- Status changed from reviewing to closed
Has there been any movement on this Jeen?