[wp-trac] [WordPress Trac] #41489: Customize Preview: do not show `cursor: not-allowed` on audio/video track titles within playlists
WordPress Trac
noreply at wordpress.org
Mon Jul 31 02:52:31 UTC 2017
#41489: Customize Preview: do not show `cursor: not-allowed` on audio/video track
titles within playlists
------------------------------+----------------------------------------
Reporter: celloexpressions | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.9
Component: Customize | Version: 4.7
Severity: normal | Keywords: good-first-bug needs-patch
Focuses: ui |
------------------------------+----------------------------------------
In the customize preview, external links get a `cursor: not-allowed`. Core
audio playlists link individual tracks within the playlist list to the
individual audio file (.mp3, etc.). This is likely the case as a fallback
for no-js users. Since the customize preview requires JS, clicking these
links will not actually navigate the customize preview off of the site,
and they will instead operate the playlist. Users should be encouraged to
interact with playlists in the preview, as they may be adding them within
content via Customize Posts or in a widget via a plugin such as Featured
Audio.
Suggested solution: add something along the lines of `if (
element.hasClass( 'wp-playlist-caption' ) { return: false; }` somewhere in
`wp.customize.isLinkPreviewable()` in `wp-includes/js/customize-
preview.js`.
This needs an actual patch and more thorough testing. There may be other
similar situations where we shouldn't use `cursor: not-allowed` that could
be grouped into this ticket if desired.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/41489>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list