question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Quality menu items "selected" style is not displayed upon selection.

See original GitHub issue

Expected behaviour

When a quality is selected, it should have a dedicated style, and the quality menu should get back to the main menu

Actual behaviour

The quality is not selected (in terms of visual feedback) and the menu stays in the same state

Environment

  • Browser: All
  • Operating System:All

Steps to reproduce

Change the quality with a standard HTML5 video


I assume this is because of the way menu events are handled (see https://github.com/sampotts/plyr/blob/master/src/js/controls.js#L595). Maybe the related hidden input selector looks for the raw value, while quality values are appended with a p (see https://github.com/sampotts/plyr/blob/master/src/js/controls.js#L524)? It then fails to find the radio input.

Moreover, once the input checked attribute is set to true, the span supposed to show that it is the selected value still has its hidden attribute (scss here:https://github.com/sampotts/plyr/blob/master/src/sass/components/menus.scss#L167)

I would have written a quick PR to fix that, but I lack the time to do so and I don’t know where/how you would prefer to fix it.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:5 (3 by maintainers)

github_iconTop GitHub Comments

2reactions
sampottscommented, May 5, 2018

When selecting an option, it’ll now take you back to the home tab pane.

0reactions
fridaycommented, May 3, 2018

Please don’t lump together separate suggestions or issues. If you want them done, create separate issues for separate issues/suggestions, and if not just don’t bring them up.

As for the main issue, you seem to be on the case yourself so I’ll leave you to it. 👍

Read more comments on GitHub >

github_iconTop Results From Across the Web

Selected MenuItem not displayed when selected from Select
When an option is selected, its not displaying in the select box. componentDidMount = (e) => { axios.get('https://jsonplaceholder.typicode.com/ ...
Read more >
Striking a Balance Between Native and Custom Select Elements
Here's the plan! We're going to build a styled select element. Not just the outside, but the inside too. Total styling control.
Read more >
Examine and edit CSS — Firefox Source Docs documentation
By default, this view only shows values that have been explicitly set by the page: to see all values, click the “Browser styles”...
Read more >
Custom select menu - CSS only - YouTube
Check out Scrimba's new career path: https://scrimba.com/path/gfrontend?utm_campaign=kevinpowell_launchAlso, a big thank you to David Lower ...
Read more >
How to Style WordPress Navigation Menus - WPBeginner
When you click on the highlighted navigation menu, it will show you ... Here you can select the background color, gradient, image, and...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found