Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#2075 closed change (fixed)

[adblockplus.org Anwiki to CMS migration] put meta tags inside the head tag

Reported by: saroyanm Assignee: kzar
Priority: P3 Milestone:
Module: Websites Keywords:
Cc: kzar, trev Blocked By:
Blocking: #2035 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/5440736082264064/

Description

Background

We are placing part of the meta tags (title, description) inside body tag, which can have impact on SEO.

What to change

Put meta tags for title and description inside head tag.

Change History (7)

comment:1 Changed 5 years ago by saroyanm

  • Blocking 2035 removed

comment:2 Changed 5 years ago by saroyanm

  • Blocked By 2035 added

comment:3 Changed 5 years ago by trev

  • Blocked By 2035 removed
  • Blocking 2035 added
  • Cc trev added

@saroyanm: I reverted your changes. The current adblockplus.org website actually has all the <meta> tags correctly in the page header. The migrated content currently leaves the <meta> tags in the page body however - so this is blocking #2035.

comment:4 Changed 5 years ago by kzar

  • Owner set to kzar

@trev Sorry that was my fault, I suggested he change this ticket to not be counted as blocking #2035.

comment:5 Changed 5 years ago by kzar

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

comment:6 Changed 5 years ago by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed 4 years ago by Ross

  • Verified working set

The meta tags are now in the head instead of the body as expected.

Tested on:
https://beta.adblockplus.org/

Tested with:

IE9.0.8112Windows 7
Chrome42.0.2311.135Windows 8.1
Note: See TracTickets for help on using tickets.