Skip to content

General: merge again commits that were deleted from master by force push#3563

Merged
zinigor merged 8 commits intomasterfrom
before-force-push-one
Mar 22, 2016
Merged

General: merge again commits that were deleted from master by force push#3563
zinigor merged 8 commits intomasterfrom
before-force-push-one

Conversation

@jeherve
Copy link
Copy Markdown
Member

@jeherve jeherve commented Mar 17, 2016

Merge again commits that were deleted from master by force push
#3497 - Vimeo: set default width if $content_width isn't set
#3489 - Stats: update Stats script to be valid XHTML
#3461 - Contact Form: fix fatal error when no post meta exists for forms
#3445 - Move stats data building to new function
#3438 - Remove YouTube alignment CSS in feeds

@jeherve jeherve added Bug When a feature is broken and / or not performing as intended General [Status] Needs Review This PR is ready for review. labels Mar 17, 2016
@jeherve jeherve added this to the 3.10 milestone Mar 17, 2016
@jeherve jeherve added [Status] In Progress and removed [Status] Needs Review This PR is ready for review. labels Mar 17, 2016
jeherve and others added 5 commits March 17, 2016 14:49
Fixes #3495

If the theme doesn't set `$content_width`, we set a default, `640`, like for YouTube.
Previously the conditional would give a wrong result when there is no content_width, which was breaking the tests.
In XHTML, attribute minimization is forbidden.
`async` and `defer` must be defined as `<script async='async' defer='defer'>`.

Suggested here:
https://wordpress.org/support/topic/doesnt-work-with-xhtml-2
When creating a very simple form, no data is saved in post meta when the form is submitted.
This PR creates an enpty $post_meta_data array to avoid errors when there is no post meta data.

Fixes #3460
@jeherve jeherve force-pushed the before-force-push-one branch from 8ea9c4c to 0beaf3b Compare March 17, 2016 13:50
@jeherve jeherve added [Status] Needs Review This PR is ready for review. and removed [Status] In Progress labels Mar 17, 2016
Philip John added 2 commits March 17, 2016 14:53
We wrap YouTube videos in a span for some CSS alignment stuff. We don't need that in feeds, and it could muck up other plugins so let's remove it.
@jeherve jeherve added [Status] In Progress and removed [Status] Needs Review This PR is ready for review. labels Mar 17, 2016
This will allow reuse of the stats data needed to create the stats
pixel, in contexts like AMP.
@jeherve jeherve added [Status] Needs Review This PR is ready for review. and removed [Status] In Progress labels Mar 17, 2016
@dereksmart dereksmart added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Mar 17, 2016
@zinigor zinigor modified the milestones: 3.9.5, 3.10 Mar 18, 2016
zinigor added a commit that referenced this pull request Mar 22, 2016
General: merge again commits that were deleted from master by force push
@zinigor zinigor merged commit 713fd23 into master Mar 22, 2016
@zinigor zinigor deleted the before-force-push-one branch March 22, 2016 09:42
@zinigor zinigor removed the [Status] Ready to Merge Go ahead, you can push that green button! label Mar 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug When a feature is broken and / or not performing as intended General Touches WP.com Files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants