Opened 22 months ago

Last modified 22 months ago

#5963 closed defect

Fix alignment of product icon next to article title on help.eyeo.com — at Version 6

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

https://codereview.adblockplus.org/29601607

Description (last modified by ire)

How to reproduce

  1. Clone help.eyeo.com repository
  2. Go to an article page with a long title, e.g. http://localhost:5000/en/adblockplus/is-adblock-plus-the-same-thing-as-adblock
  3. Resize browser to mobile size

...

Observed behaviour

See that the icon is aligned to the left and pushes the whole title. (see attachment)

Expected behaviour

Remove the icon on mobile only

Change History (7)

comment:1 Changed 22 months ago by ire

  • Description modified (diff)

comment:2 Changed 22 months ago by ire

My suggestions:

  1. We move the icon to above the title
  2. We remove the icon altogether (on mobile only)

@jeen What do you think?

comment:3 Changed 22 months ago by ire

  • Description modified (diff)

comment:4 Changed 22 months ago by ire

  • Type changed from change to defect

comment:5 follow-up: Changed 22 months ago by jeen

I say we remove the icon altogether for mobile - since the icon already appears in the breadcrumb anyway

comment:6 in reply to: ↑ 5 Changed 22 months ago by ire

  • Description modified (diff)
  • Owner set to ire
  • Ready set

Replying to jeen:

I say we remove the icon altogether for mobile - since the icon already appears in the breadcrumb anyway

Alright. Thanks!

Note: See TracTickets for help on using tickets.