#38783 closed defect (bug) (fixed)
Twenty Fourteen: Revert Video Header Implementation
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | |
Focuses: | Cc: |
Description
For 4.7, Twenty Fourteen got video headers because it made sense to test out implementation in another, existing theme, besides Twenty Seventeen. However, I don't think think the theme should ship with video headers because:
- the theme was never designed with the feature in mind.
- the theme has been around for awhile, and the feature could interfere with existing customizations by users.
- it's okay to have a new feature just be in the latest default theme.
A clean revert can't be done because the changes to the theme were made along with the initial commit for video headers.
Attachments (1)
Change History (4)
#1
@
9 years ago
Not opposed. If we don't revert, we at least need to go back and update the implementation to come back in line with the changes that have been made since.
#2
@
9 years ago
- Owner set to karmatosed
- Resolution set to fixed
- Status changed from new to closed
In 39226:
#3
@
9 years ago
I'm not necessarily opposed either, but I wouldn't call the implementation incomplete. It worked fully in my testing, and no bugs were reported. That may have changed with later adjustments.
the theme has been around for awhile, and the feature could interfere with existing customizations by users.
I don't think there would be any issues with this, as existing header images are still used in the same way, so the only changes would happen after adding a video.
Removes video headers for Twenty Fourteen.