Opened 4 years ago

Last modified 3 years ago

#4514 closed defect

Font readability on acceptableads.com — at Version 11

Reported by: juliandoucette Assignee: juliandoucette
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):

https://codereview.adblockplus.org/29497585/

Description (last modified by juliandoucette)

Environment

https://acceptableads.com

How to reproduce

  1. Find the attached example at https://acceptableads.com/en/users.

Observed behaviour

I find it hard to read main text on https://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

I think that we should optimize our font styles for readability.

Change History (14)

Changed 4 years ago by juliandoucette

comment:1 Changed 4 years ago by juliandoucette

  • Description modified (diff)

comment:2 Changed 4 years ago by juliandoucette

  • Description modified (diff)

comment:3 Changed 4 years ago by juliandoucette

  • Cc jeen added

Has there been any movement on this Jeen?

comment:4 Changed 4 years ago 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 4 years ago by juliandoucette

From @christiane:

Up all font styles by 2px. Up h1 by 4px and increase line height to 150%

Changed 4 years ago by juliandoucette

comment:6 Changed 4 years ago by juliandoucette

  • Priority changed from Unknown to P2
  • Ready set

comment:7 Changed 3 years ago by juliandoucette

  • Owner set to juliandoucette

comment:8 Changed 3 years ago by juliandoucette

  • Cc athornburgh added

comment:9 Changed 3 years ago by juliandoucette

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:10 Changed 3 years ago 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 3 years ago by juliandoucette

Patch from code review 29374562.

comment:11 Changed 3 years ago 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.

Note: See TracTickets for help on using tickets.