Opened 2 years ago

Closed 22 months ago

#5472 closed change (invalid)

Improvements to media coverage section on eyeo.com

Reported by: ire Assignee:
Priority: P4 Milestone:
Module: Websites Keywords:
Cc: juliandoucette, wspee, jeen Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

In 4912, we updated the media coverage links on eyeo.com homepage. In that ticket, it was also specified that the links should open in a new tab. Therefore, we need to make it clear to all users that the media coverage links will open in a new tab.

  • Users navigating with a keyboard
  • Users navigation with a mouse
  • Users navigating with a screenreader
  • Users navigating with a touch device

What to change

There are some improvements we can make

  1. Place a statement that links will open in a new window before the logos
  2. Add a white background to the image via CSS (so that the alt text has a white background too if the image doesn't load)

See 4912 codereview for context

Attachments (1)

Screen Shot 2018-02-19 at 11.16.31 am.png (47.7 KB) - added by ire 22 months ago.
If image fails to load

Download all attachments as: .zip

Change History (5)

comment:1 Changed 2 years ago by ire

  • Cc jeen added

Suggestions by @juliandoucette on No. 1

  1. "Click a publisher to open an article in a new window"
  2. "Click a publisher to see what they have to say about us (Opens a new window)"
  3. "Click a publisher to find out more (opens a new window)"

@jeen Do you like any of these or can you suggest a different phrase?

comment:2 Changed 2 years ago by jeen

@ juliandoucette @ire

I like how it has been executed currently: "(Links open in a new window)"

I think it is simple and to the point.

comment:3 Changed 2 years ago by juliandoucette

  • Priority changed from P3 to P4

Changed 22 months ago by ire

If image fails to load

comment:4 Changed 22 months ago by ire

  • Resolution set to invalid
  • Status changed from new to closed

I'm closing this ticket because:

Place a statement that links will open in a new window before the logos

Jeen is fine with the current statement

Add a white background to the image via CSS (so that the alt text has a white background too if the image doesn't load)

There is already a white background. If the image fails to load, the text is perfectly legible at the moment. See screenshot

Note: See TracTickets for help on using tickets.