WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#35194 closed defect (bug) (fixed)

Remove embed discovery tags from HTML header of static home pages

Reported by: peterwilsoncc Owned by: pento
Priority: normal Milestone: 4.4.1
Component: Embeds Version: 4.4
Severity: normal Keywords: has-patch has-unit-tests fixed-major
Cc: Focuses:

Description

It's not possible to embed a static front page, see #34971.

Attached patch prevents the discovery headers from showing on the front page.

Attachments (2)

35194.diff​ (549 bytes ) - added by peterwilsoncc 11 years ago.
35194.2.diff​ (1.7 KB ) - added by peterwilsoncc 11 years ago.

Download all attachments as: .zip

Change History (9)

@peterwilsoncc
11 years ago

#1 @peterwilsoncc
11 years ago

  • Keywords has-patch added

#2 @johnbillion
11 years ago

  • Keywords needs-unit-tests added

+1.

There are some tests for this in Tests_oEmbed_Discovery that could be added to.

@peterwilsoncc
11 years ago

#3 @peterwilsoncc
11 years ago

  • Keywords has-unit-tests added; needs-unit-tests removed

Unit tests added in 35194.2.diff​.

Includes home page tests for both show_on_front settings.

#4 @pento
11 years ago

  • Owner set to pento
  • Status new → assigned

#5 @pento
11 years ago

  • Resolution → fixed
  • Status assigned → closed

In 36059:

Embeds: Don't show embed discovery link on a static front page.

There's currently no iframe content being generated for a static front page. Giving out a link to that isn't an ideal user experience.

Props peterwilsoncc.

Fixes #35194 for trunk.

#6 @pento
11 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

#7 @pento
11 years ago

  • Resolution → fixed
  • Status reopened → closed

In 36060:

Embeds: Don't show embed discovery link on a static front page.

There's currently no iframe content being generated for a static front page. Giving out a link to that isn't an ideal user experience.

Props peterwilsoncc.

Merge of [36059] to the 4.4 branch.

Fixes #35194.

Note: See TracTickets for help on using tickets.