Move uniqueness filter from getQualityOptions to setQualityMenu
This commit is contained in:
		
							
								
								
									
										5
									
								
								src/js/controls.js
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										5
									
								
								src/js/controls.js
									
									
									
									
										vendored
									
									
								
							@ -641,9 +641,10 @@ const controls = {
 | 
			
		||||
        const type = 'quality';
 | 
			
		||||
        const list = this.elements.settings.panes.quality.querySelector('ul');
 | 
			
		||||
 | 
			
		||||
        // Set options if passed and filter based on config
 | 
			
		||||
        // Set options if passed and filter based on uniqueness and config
 | 
			
		||||
        if (utils.is.array(options)) {
 | 
			
		||||
            this.options.quality = options.filter(quality => this.config.quality.options.includes(quality));
 | 
			
		||||
            this.options.quality = utils.dedupe(options)
 | 
			
		||||
                .filter(quality => this.config.quality.options.includes(quality));
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
        // Toggle the pane and tab
 | 
			
		||||
 | 
			
		||||
@ -20,12 +20,9 @@ const html5 = {
 | 
			
		||||
    // Get quality levels
 | 
			
		||||
    getQualityOptions() {
 | 
			
		||||
        // Get sizes from <source> elements
 | 
			
		||||
        const sizes = html5.getSources.call(this)
 | 
			
		||||
        return html5.getSources.call(this)
 | 
			
		||||
            .map(source => Number(source.getAttribute('size')))
 | 
			
		||||
            .filter(Boolean);
 | 
			
		||||
 | 
			
		||||
        // Reduce to unique list
 | 
			
		||||
        return utils.dedupe(sizes);
 | 
			
		||||
    },
 | 
			
		||||
 | 
			
		||||
    extend() {
 | 
			
		||||
 | 
			
		||||
		Reference in New Issue
	
	Block a user