Jetpack: (5P) Podcast Player: Themes' incompatibilities

[WunderBart]

Tested Podcast Player with a few themes and listed visual conflicts I noticed in each of them. I think we can treat this issue as a master thread to report all the theme styling issues that need to be taken care of.

TwentyEleven

• Podcast title doesn't inherit secondary color
• The cover image is squashed

TwentyTwelve

• Visited link has different color
• Cover image has border-radius

TwentyThirteen

• MediaElement elements have different colors: Won't be fixed https://github.com/Automattic/jetpack/issues/15478#issuecomment-617384963
• Visited link has a different color (#ac0404)
• Hovered link have different styles (underline, black color)

TwentyFourteen

• Visited/hovered link has adifferent color
• MediaElement have different colors & icons ⚠️: Won't be fixed https://github.com/Automattic/jetpack/issues/15478#issuecomment-617384963

TwentyFifteen

• Borders everywhere! :D

TwentySixteen

• As in TwentyFifteen - extra borders

TwentySeventeen

• Borders!
• Header title has top margin & different font-weight
• Podcast title color is not inherited from secondary

TwentyNineteen

• Lookin' good!

TwentyTwenty

• Lookin' good!

Varia

• Lookin' good!

Maywood

• Hovered link color is always black (front-end only)

[obenland]

Let's remove those li borders & link styles globally and _lightly_ explore what a CSS reset for the block might look like and which consequences that would entail

[jeryj]

https://github.com/Automattic/jetpack/pull/15504 fixes almost all of the issues with very minor CSS overrides overall.

The only issues it doesn't fix is the twenty fourteen and twenty thirteen media players. They both (but especially twenty fourteen) include CSS specific to the media elements player. We could increase the specificity and add a few more things to address twenty thirteen, but twenty fourteen would include many changes _only relevant to twenty fourteen_.

In my opinion, as both themes are 6 - 7 years old and would require so many specific changes just for them, that it's not worth addressing.

[obenland]

Nice job! Yes, let's not invest more time in those mejs styles.