Opened 9 years ago
Closed 7 years ago
#1618 closed defect (bug) (fixed)
Enable oEmbed for Themes on Rosetta sites
Reported by: | dd32 | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Theme Directory | Keywords: | |
Cc: |
Description
Follows on from #1464 - I'm not sure if this has ever worked, or if recent changes broke it.
https://wordpress.org/themes/twentyfifteen/embed/ works well, however https://de.wordpress.org/themes/twentyfifteen/embed/ doesnt.
This is made more difficult because you can't rely upon the global WP_Query on Rosetta Theme directory sites at present (The theme doesn't really use WordPress templates for rosetta blogs) - This may change as the mode used to present the plugin directory at multiple url's has proved to be effective in testing so far.
Change History (3)
#2
@
9 years ago
https://de.wordpress.org/themes/twentyfifteen/embed/ now works
However, in the process of doing that I altered the embed template to use wp_trim_words( $description, 55)
directly instead of using the embed excerpt functions, that's because $post->post_content
isn't translated, only $theme->description
is.
I'm fairly certain that in doing it this way, I've broken some languages excerpt changes (https://ja.wordpress.org/themes/twentyfifteen/embed/ perhaps?) or broken words mid-way.
We should figure out how to alter the post_content
directly to have the translated data available everywhere.
In 2755: