Jetpack: (2P) Podcast Player: Don't render list for one episode only

Created on 15 May 2020  路  7Comments  路  Source: Automattic/jetpack

Is your feature request related to a problem? Please describe.

The Podcast Player block looks awkward with just one episode:

Screenshot 2020-05-15 11 51 22

The title is already in the heading and the media controls are already bound to that single episode - no need to click it first.

Describe the solution you'd like

Turn the heading into a link as a fallback and not render the list at all:

Screenshot 2020-05-15 11 51 43

[Block] Podcast Player [Type] Enhancement

Most helpful comment

Turn the heading into a link as a fallback

Based on new input from @jancavan we'll change this to the following:

Another idea is to not make the episode title a link, but add a link icon next to it that鈥檒l link to the episode; and this is applies to each episode, whether it鈥檚 a single one or there鈥檚 multiple.

Screen Shot 2020-06-02 at 9 37 56 AM

All 7 comments

Just noting that we'll loose the only place where we're linking out to the episode if we remove it.
Related, should the current title at the top maybe become a link in general?

Just noting that we'll loose the only place where we're linking out to the episode if we remove it.

@frontdevde Just to clarify, do you mean clicking on in to play the episode?

@jancavan If you hover over the list item you'll see in the bottom left of the browser that it links to the episode. You can command + click it to go to the episode in a new window.

Oh I see, thanks. It's not even obvious that it actually does that.

Turn the heading into a link as a fallback

Based on new input from @jancavan we'll change this to the following:

Another idea is to not make the episode title a link, but add a link icon next to it that鈥檒l link to the episode; and this is applies to each episode, whether it鈥檚 a single one or there鈥檚 multiple.

Screen Shot 2020-06-02 at 9 37 56 AM

Just to expand on the above ^:

  • Add a link icon next to the episode title, this will link to out to the episode in a new tab.
  • Episode title remains as is (ie: we don't turn it into a link to prevent users from inadvertently clicking on it due to size, only for it to link to the same episode)
  • Apply the same behavior to each episode. This way, we attain consistency regardless of how many episodes are listed in the track list.
  • In the mockup, I'm using gridicons-link.svg from Gridicons, but we also have link icons in Material and @wordpress/icons which look pretty much the same. Let me ask what we should be using because we don't have clear guidelines. I'll get back to you. @frontdevde

@frontdevde Let's just use link from @wordpress/icons since this is a block within Gutenberg.

Was this page helpful?
0 / 5 - 0 ratings